Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Github action to set up snowcli in a CI/CD workflow #1

Merged
merged 29 commits into from
Apr 9, 2024

Conversation

sfc-gh-jvasquezrojas
Copy link
Collaborator

This action installs snowflake-cli-labs in an isolated environment using pipx, so it will not have conflicts with any other package. In addition there is an optional parameter to set up a default config file in .snowflake directory.

There is not a conventional way to test a composite github action because there is not a framework for that, instead some
usage examples were added to ensure a valid correctness and utility of the github action. A brief description of every example jobs and what they do are the following:

install_Snowcli_with_pip_and_then_with_pipx: This job installs snowcli version 2.1.0 with pipx and version 2.1.1 with pip, the workflow should show 2.1.0 when executes snow --version.

install_in_multiple_os: This job validates the correct installation of snowcli in any OS.

deploy_demo_native_app: This is a more complex that set up snowflake cli and creates a version for the applicaiton package defined in the source code.

action.yml Outdated Show resolved Hide resolved
config.toml Outdated Show resolved Hide resolved
config.toml Outdated Show resolved Hide resolved
install_snowcli.sh Outdated Show resolved Hide resolved
install_snowcli.sh Outdated Show resolved Hide resolved
install_snowcli.sh Outdated Show resolved Hide resolved
install_snowcli.sh Outdated Show resolved Hide resolved
test.sh Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@sfc-gh-jvasquezrojas sfc-gh-jvasquezrojas force-pushed the composite_snowcli_action_deploy branch from 8dd0dfb to 28fd622 Compare April 4, 2024 15:57
@sfc-gh-jvasquezrojas sfc-gh-jvasquezrojas force-pushed the composite_snowcli_action_deploy branch from 367f9ec to 5d5b423 Compare April 5, 2024 22:55
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
cli-version:
description: 'Snowflake CLI version'
required: false
default: ''
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm wondering if "latest" would be more meaningful, what do you think? It would add more information about what is the default behavior without checking scripts

README.md Outdated Show resolved Hide resolved
@sfc-gh-turbaszek sfc-gh-turbaszek merged commit bc942ec into main Apr 9, 2024
10 checks passed
@sfc-gh-turbaszek sfc-gh-turbaszek deleted the composite_snowcli_action_deploy branch April 9, 2024 06:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants