-
Notifications
You must be signed in to change notification settings - Fork 57
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Changed integration tests user auth to private key
- Loading branch information
1 parent
3e95ca4
commit 6a0c614
Showing
8 changed files
with
236 additions
and
203 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,30 +1,31 @@ | ||
name: Environment cleanup | ||
|
||
on: | ||
workflow_dispatch: | ||
schedule: | ||
- cron: "0 10 * * *" | ||
|
||
jobs: | ||
tests: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
with: | ||
persist-credentials: false | ||
- name: Set up Python | ||
uses: actions/setup-python@v5 | ||
with: | ||
python-version: "3.11" | ||
- name: Install dependencies | ||
run: | | ||
python -m pip install --upgrade pip hatch | ||
python -m hatch env create e2e | ||
- name: Run cleaunp | ||
env: | ||
TERM: unknown | ||
SNOWFLAKE_CONNECTIONS_INTEGRATION_HOST: ${{ secrets.SNOWFLAKE_HOST }} | ||
SNOWFLAKE_CONNECTIONS_INTEGRATION_USER: ${{ secrets.SNOWFLAKE_USER }} | ||
SNOWFLAKE_CONNECTIONS_INTEGRATION_PASSWORD: ${{ secrets.SNOWFLAKE_PASSWORD }} | ||
SNOWFLAKE_CONNECTIONS_INTEGRATION_ACCOUNT: ${{ secrets.SNOWFLAKE_ACCOUNT }} | ||
run: python -m hatch run e2e:cleanup | ||
#name: Environment cleanup | ||
# | ||
#on: | ||
# workflow_dispatch: | ||
# schedule: | ||
# - cron: "0 10 * * *" | ||
# | ||
#jobs: | ||
# tests: | ||
# runs-on: ubuntu-latest | ||
# steps: | ||
# - uses: actions/checkout@v4 | ||
# with: | ||
# persist-credentials: false | ||
# - name: Set up Python | ||
# uses: actions/setup-python@v5 | ||
# with: | ||
# python-version: "3.11" | ||
# - name: Install dependencies | ||
# run: | | ||
# python -m pip install --upgrade pip hatch | ||
# python -m hatch env create e2e | ||
# - name: Run cleaunp | ||
# env: | ||
# TERM: unknown | ||
# SNOWFLAKE_CONNECTIONS_INTEGRATION_AUTHENTICATOR: SNOWFLAKE_JWT | ||
# SNOWFLAKE_CONNECTIONS_INTEGRATION_HOST: ${{ secrets.SNOWFLAKE_HOST }} | ||
# SNOWFLAKE_CONNECTIONS_INTEGRATION_USER: ${{ secrets.SNOWFLAKE_USER }} | ||
# SNOWFLAKE_CONNECTIONS_INTEGRATION_ACCOUNT: ${{ secrets.SNOWFLAKE_ACCOUNT }} | ||
# SNOWFLAKE_CONNECTIONS_INTEGRATION_PRIVATE_KEY_PATH: ${{ secrets.SNOWFLAKE_PRIVATE_KEY_PATH }} | ||
# run: python -m hatch run e2e:cleanup |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,27 +1,28 @@ | ||
name: "CLI Action testing" | ||
|
||
on: | ||
schedule: | ||
- cron: "0 0 * * *" | ||
workflow_dispatch: | ||
|
||
jobs: | ||
version: | ||
name: "Check Snowflake CLI version" | ||
runs-on: ubuntu-latest | ||
steps: | ||
- name: Checkout repo | ||
uses: actions/checkout@v4 | ||
with: | ||
persist-credentials: false | ||
- uses: Snowflake-Labs/snowflake-cli-action@v1 | ||
with: | ||
cli-version: "latest" | ||
default-config-file-path: "tests_integration/config/connection_configs.toml" | ||
- name: Test connection | ||
env: | ||
TERM: unknown | ||
SNOWFLAKE_CONNECTIONS_INTEGRATION_USER: ${{ secrets.SNOWFLAKE_USER }} | ||
SNOWFLAKE_CONNECTIONS_INTEGRATION_PASSWORD: ${{ secrets.SNOWFLAKE_PASSWORD }} | ||
SNOWFLAKE_CONNECTIONS_INTEGRATION_ACCOUNT: ${{ secrets.SNOWFLAKE_ACCOUNT }} | ||
run: snow connection test -c integration | grep Status | ||
#name: "CLI Action testing" | ||
# | ||
#on: | ||
# schedule: | ||
# - cron: "0 0 * * *" | ||
# workflow_dispatch: | ||
# | ||
#jobs: | ||
# version: | ||
# name: "Check Snowflake CLI version" | ||
# runs-on: ubuntu-latest | ||
# steps: | ||
# - name: Checkout repo | ||
# uses: actions/checkout@v4 | ||
# with: | ||
# persist-credentials: false | ||
# - uses: Snowflake-Labs/snowflake-cli-action@v1 | ||
# with: | ||
# cli-version: "latest" | ||
# default-config-file-path: "tests_integration/config/connection_configs.toml" | ||
# - name: Test connection | ||
# env: | ||
# TERM: unknown | ||
# SNOWFLAKE_CONNECTIONS_INTEGRATION_AUTHENTICATOR: SNOWFLAKE_JWT | ||
# SNOWFLAKE_CONNECTIONS_INTEGRATION_USER: ${{ secrets.SNOWFLAKE_USER }} | ||
# SNOWFLAKE_CONNECTIONS_INTEGRATION_ACCOUNT: ${{ secrets.SNOWFLAKE_ACCOUNT }} | ||
# SNOWFLAKE_CONNECTIONS_INTEGRATION_PRIVATE_KEY_PATH: ${{ secrets.SNOWFLAKE_PRIVATE_KEY_PATH }} | ||
# run: snow connection test -c integration | grep Status |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,127 +1,129 @@ | ||
name: E2E testing | ||
|
||
on: | ||
pull_request: | ||
branches: | ||
- "*" | ||
push: | ||
tags: | ||
- "v*" # Push events to matching v*, i.e. v1.0, v20.15.10 | ||
branches: | ||
- main | ||
repository_dispatch: | ||
types: [ok-to-test-command] | ||
schedule: | ||
- cron: "0 8 * * *" | ||
|
||
concurrency: | ||
group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }} | ||
cancel-in-progress: true | ||
|
||
jobs: | ||
define-matrix: | ||
uses: ./.github/workflows/matrix.yaml | ||
|
||
e2e-trusted: | ||
needs: define-matrix | ||
strategy: | ||
fail-fast: true | ||
matrix: | ||
os: ${{ fromJSON(needs.define-matrix.outputs.os) }} | ||
python-version: ${{ fromJSON(needs.define-matrix.outputs.python) }} | ||
runs-on: ${{ matrix.os }} | ||
if: github.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name == github.repository | ||
steps: | ||
- uses: actions/checkout@v4 | ||
with: | ||
persist-credentials: false | ||
- name: Set up Python | ||
uses: actions/setup-python@v5 | ||
with: | ||
python-version: ${{ matrix.python-version }} | ||
- name: Install dependencies | ||
run: | | ||
python -m pip install --upgrade pip hatch | ||
python -m hatch env create e2e | ||
- name: Run end to end tests | ||
env: | ||
TERM: unknown | ||
SNOWFLAKE_CONNECTIONS_INTEGRATION_HOST: ${{ secrets.SNOWFLAKE_HOST }} | ||
SNOWFLAKE_CONNECTIONS_INTEGRATION_USER: ${{ secrets.SNOWFLAKE_USER }} | ||
SNOWFLAKE_CONNECTIONS_INTEGRATION_PASSWORD: ${{ secrets.SNOWFLAKE_PASSWORD }} | ||
SNOWFLAKE_CONNECTIONS_INTEGRATION_ACCOUNT: ${{ secrets.SNOWFLAKE_ACCOUNT }} | ||
run: python -m hatch run e2e:test | ||
|
||
# Repo owner has commented /ok-to-test on a (fork-based) pull request | ||
e2e-fork: | ||
needs: define-matrix | ||
strategy: | ||
fail-fast: true | ||
matrix: | ||
os: ${{ fromJSON(needs.define-matrix.outputs.os) }} | ||
python-version: ${{ fromJSON(needs.define-matrix.outputs.python) }} | ||
runs-on: ${{ matrix.os }} | ||
permissions: | ||
pull-requests: write | ||
checks: write | ||
if: | | ||
github.event_name == 'repository_dispatch' && | ||
github.event.client_payload.slash_command.args.named.sha != '' && | ||
contains( | ||
github.event.client_payload.pull_request.head.sha, | ||
github.event.client_payload.slash_command.args.named.sha | ||
) | ||
steps: | ||
- uses: actions/checkout@v4 | ||
with: | ||
persist-credentials: false | ||
ref: 'refs/pull/${{ github.event.client_payload.pull_request.number }}/merge' | ||
- name: Set up Python | ||
uses: actions/setup-python@v5 | ||
with: | ||
python-version: ${{ matrix.python-version }} | ||
- name: Install dependencies | ||
run: | | ||
python -m pip install --upgrade pip hatch | ||
python -m hatch env create e2e | ||
- name: Run end to end tests | ||
env: | ||
TERM: unknown | ||
SNOWFLAKE_CONNECTIONS_INTEGRATION_HOST: ${{ secrets.SNOWFLAKE_HOST }} | ||
SNOWFLAKE_CONNECTIONS_INTEGRATION_USER: ${{ secrets.SNOWFLAKE_USER }} | ||
SNOWFLAKE_CONNECTIONS_INTEGRATION_PASSWORD: ${{ secrets.SNOWFLAKE_PASSWORD }} | ||
SNOWFLAKE_CONNECTIONS_INTEGRATION_ACCOUNT: ${{ secrets.SNOWFLAKE_ACCOUNT }} | ||
run: python -m hatch run e2e:test | ||
# Update check run called "e2e-fork" | ||
- uses: actions/github-script@v7 | ||
id: update-check-run | ||
if: ${{ always() }} | ||
env: | ||
number: ${{ github.event.client_payload.pull_request.number }} | ||
job: ${{ github.job }} | ||
# Conveniently, job.status maps to https://developer.github.com/v3/checks/runs/#update-a-check-run | ||
conclusion: ${{ job.status }} | ||
with: | ||
github-token: ${{ secrets.GITHUB_TOKEN }} | ||
script: | | ||
const { data: pull } = await github.rest.pulls.get({ | ||
...context.repo, | ||
pull_number: process.env.number | ||
}); | ||
const ref = pull.head.sha; | ||
const { data: checks } = await github.rest.checks.listForRef({ | ||
...context.repo, | ||
ref | ||
}); | ||
const check = checks.check_runs.filter(c => c.name === process.env.job); | ||
const { data: result } = await github.rest.checks.update({ | ||
...context.repo, | ||
check_run_id: check[0].id, | ||
status: 'completed', | ||
conclusion: process.env.conclusion | ||
}); | ||
return result; | ||
#name: E2E testing | ||
# | ||
#on: | ||
# pull_request: | ||
# branches: | ||
# - "*" | ||
# push: | ||
# tags: | ||
# - "v*" # Push events to matching v*, i.e. v1.0, v20.15.10 | ||
# branches: | ||
# - main | ||
# repository_dispatch: | ||
# types: [ok-to-test-command] | ||
# schedule: | ||
# - cron: "0 8 * * *" | ||
# | ||
#concurrency: | ||
# group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }} | ||
# cancel-in-progress: true | ||
# | ||
#jobs: | ||
# define-matrix: | ||
# uses: ./.github/workflows/matrix.yaml | ||
# | ||
# e2e-trusted: | ||
# needs: define-matrix | ||
# strategy: | ||
# fail-fast: true | ||
# matrix: | ||
# os: ${{ fromJSON(needs.define-matrix.outputs.os) }} | ||
# python-version: ${{ fromJSON(needs.define-matrix.outputs.python) }} | ||
# runs-on: ${{ matrix.os }} | ||
# if: github.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name == github.repository | ||
# steps: | ||
# - uses: actions/checkout@v4 | ||
# with: | ||
# persist-credentials: false | ||
# - name: Set up Python | ||
# uses: actions/setup-python@v5 | ||
# with: | ||
# python-version: ${{ matrix.python-version }} | ||
# - name: Install dependencies | ||
# run: | | ||
# python -m pip install --upgrade pip hatch | ||
# python -m hatch env create e2e | ||
# - name: Run end to end tests | ||
# env: | ||
# TERM: unknown | ||
# SNOWFLAKE_CONNECTIONS_INTEGRATION_AUTHENTICATOR: SNOWFLAKE_JWT | ||
# SNOWFLAKE_CONNECTIONS_INTEGRATION_HOST: ${{ secrets.SNOWFLAKE_HOST }} | ||
# SNOWFLAKE_CONNECTIONS_INTEGRATION_USER: ${{ secrets.SNOWFLAKE_USER }} | ||
# SNOWFLAKE_CONNECTIONS_INTEGRATION_ACCOUNT: ${{ secrets.SNOWFLAKE_ACCOUNT }} | ||
# SNOWFLAKE_CONNECTIONS_INTEGRATION_PRIVATE_KEY_PATH: ${{ secrets.SNOWFLAKE_PRIVATE_KEY_PATH }} | ||
# run: python -m hatch run e2e:test | ||
# | ||
# # Repo owner has commented /ok-to-test on a (fork-based) pull request | ||
# e2e-fork: | ||
# needs: define-matrix | ||
# strategy: | ||
# fail-fast: true | ||
# matrix: | ||
# os: ${{ fromJSON(needs.define-matrix.outputs.os) }} | ||
# python-version: ${{ fromJSON(needs.define-matrix.outputs.python) }} | ||
# runs-on: ${{ matrix.os }} | ||
# permissions: | ||
# pull-requests: write | ||
# checks: write | ||
# if: | | ||
# github.event_name == 'repository_dispatch' && | ||
# github.event.client_payload.slash_command.args.named.sha != '' && | ||
# contains( | ||
# github.event.client_payload.pull_request.head.sha, | ||
# github.event.client_payload.slash_command.args.named.sha | ||
# ) | ||
# steps: | ||
# - uses: actions/checkout@v4 | ||
# with: | ||
# persist-credentials: false | ||
# ref: 'refs/pull/${{ github.event.client_payload.pull_request.number }}/merge' | ||
# - name: Set up Python | ||
# uses: actions/setup-python@v5 | ||
# with: | ||
# python-version: ${{ matrix.python-version }} | ||
# - name: Install dependencies | ||
# run: | | ||
# python -m pip install --upgrade pip hatch | ||
# python -m hatch env create e2e | ||
# - name: Run end to end tests | ||
# env: | ||
# TERM: unknown | ||
# SNOWFLAKE_CONNECTIONS_INTEGRATION_AUTHENTICATOR: SNOWFLAKE_JWT | ||
# SNOWFLAKE_CONNECTIONS_INTEGRATION_HOST: ${{ secrets.SNOWFLAKE_HOST }} | ||
# SNOWFLAKE_CONNECTIONS_INTEGRATION_USER: ${{ secrets.SNOWFLAKE_USER }} | ||
# SNOWFLAKE_CONNECTIONS_INTEGRATION_ACCOUNT: ${{ secrets.SNOWFLAKE_ACCOUNT }} | ||
# SNOWFLAKE_CONNECTIONS_INTEGRATION_PRIVATE_KEY_PATH: ${{ secrets.SNOWFLAKE_PRIVATE_KEY_PATH }} | ||
# run: python -m hatch run e2e:test | ||
# # Update check run called "e2e-fork" | ||
# - uses: actions/github-script@v7 | ||
# id: update-check-run | ||
# if: ${{ always() }} | ||
# env: | ||
# number: ${{ github.event.client_payload.pull_request.number }} | ||
# job: ${{ github.job }} | ||
# # Conveniently, job.status maps to https://developer.github.com/v3/checks/runs/#update-a-check-run | ||
# conclusion: ${{ job.status }} | ||
# with: | ||
# github-token: ${{ secrets.GITHUB_TOKEN }} | ||
# script: | | ||
# const { data: pull } = await github.rest.pulls.get({ | ||
# ...context.repo, | ||
# pull_number: process.env.number | ||
# }); | ||
# const ref = pull.head.sha; | ||
# | ||
# const { data: checks } = await github.rest.checks.listForRef({ | ||
# ...context.repo, | ||
# ref | ||
# }); | ||
# | ||
# const check = checks.check_runs.filter(c => c.name === process.env.job); | ||
# | ||
# const { data: result } = await github.rest.checks.update({ | ||
# ...context.repo, | ||
# check_run_id: check[0].id, | ||
# status: 'completed', | ||
# conclusion: process.env.conclusion | ||
# }); | ||
# | ||
# return result; |
Oops, something went wrong.