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

Action var tests off #2632

Merged
merged 32 commits into from
Aug 26, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
32 commits
Select commit Hold shift + click to select a range
37fda8e
Update test_aws_integration.yaml
BrianCashProf Aug 15, 2024
f151ac4
Update test_aws_integration.yaml
BrianCashProf Aug 15, 2024
4cd3fa0
Update test_aws_integration.yaml
BrianCashProf Aug 15, 2024
05ed23c
Update test_aws_integration.yaml
BrianCashProf Aug 15, 2024
8bfe2ff
Update test_azure_integration.yaml
BrianCashProf Aug 15, 2024
bb0b97d
Update test_do_integration.yaml
BrianCashProf Aug 15, 2024
ee65825
Update test_gcp_integration.yaml
BrianCashProf Aug 15, 2024
e05b40c
Update test_azure_integration.yaml
BrianCashProf Aug 15, 2024
e2a8ada
Update test_azure_integration.yaml
BrianCashProf Aug 15, 2024
a98fb6c
Update test_aws_integration.yaml
BrianCashProf Aug 15, 2024
c0d3ccf
Update test_do_integration.yaml
BrianCashProf Aug 15, 2024
d9e4885
Update test_gcp_integration.yaml
BrianCashProf Aug 15, 2024
ca97e76
Merge branch 'develop' into action-var-tests-off
BrianCashProf Aug 16, 2024
91f8373
Merge branch 'nebari-dev:develop' into action-var-tests-off
BrianCashProf Aug 23, 2024
bb9a06c
Update test_aws_integration.yaml
BrianCashProf Aug 23, 2024
574bdd8
Update test_aws_integration.yaml
BrianCashProf Aug 23, 2024
0c314b4
Update test_aws_integration.yaml
BrianCashProf Aug 23, 2024
3452b37
Update test_aws_integration.yaml
BrianCashProf Aug 23, 2024
763de93
Update test_aws_integration.yaml
BrianCashProf Aug 23, 2024
f34439b
Update test_aws_integration.yaml
BrianCashProf Aug 23, 2024
635356e
Update test_aws_integration.yaml
BrianCashProf Aug 23, 2024
b10fd45
Update test_aws_integration.yaml
BrianCashProf Aug 23, 2024
fb112a1
Update test_aws_integration.yaml
BrianCashProf Aug 23, 2024
4e1ad4d
Update test_aws_integration.yaml
BrianCashProf Aug 26, 2024
63b4111
Update test_azure_integration.yaml
BrianCashProf Aug 26, 2024
a2c7182
Update test_azure_integration.yaml
BrianCashProf Aug 26, 2024
8acb95c
Update test_do_integration.yaml
BrianCashProf Aug 26, 2024
ac2fc75
Update test_gcp_integration.yaml
BrianCashProf Aug 26, 2024
0a9a258
Update test_aws_integration.yaml
BrianCashProf Aug 26, 2024
1e4acd2
Update test_azure_integration.yaml
BrianCashProf Aug 26, 2024
5cd522c
Update test_do_integration.yaml
BrianCashProf Aug 26, 2024
ebad703
Update test_gcp_integration.yaml
BrianCashProf Aug 26, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 1 addition & 21 deletions .github/workflows/test_aws_integration.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -33,31 +33,11 @@ env:
NEBARI_GH_BRANCH: ${{ github.event.inputs.branch || 'develop' }}
NEBARI_IMAGE_TAG: ${{ github.event.inputs.image-tag || 'main' }}
TF_LOG: ${{ github.event.inputs.tf-log-level || 'info' }}∏
NO_PROVIDER_CREDENTIALS_aws: false

jobs:
# Used to skip cloud provider checks due to "jobs" not supporting {{ env }} variables contexts.
check-for-credentials_aws:
runs-on: ubuntu-latest
permissions:
id-token: write
contents: read
outputs:
provider_credentials_aws: ${{ steps.flag-check.outputs.provider_credentials_aws }}
steps:
- name: Check if user wants to run AWS integration based on credentials
id: flag-check
run: |
if [ "${{ env.NO_PROVIDER_CREDENTIALS_aws }}" == "true" ]; then
echo "::set-output name=provider_credentials_aws::0"
else
echo "::set-output name=provider_credentials_aws::1"
fi

test-aws-integration:
runs-on: ubuntu-latest
needs: check-for-credentials_aws
if: ${{ needs.check-for-credentials.outputs.provider_credentials_aws == '1' }}
if: ${{ vars.SKIP_AWS_INTEGRATION_TEST != 'true' }}
permissions:
id-token: write
contents: read
Expand Down
28 changes: 1 addition & 27 deletions .github/workflows/test_azure_integration.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -31,34 +31,11 @@ env:
NEBARI_GH_BRANCH: ${{ github.event.inputs.branch || 'develop' }}
NEBARI_IMAGE_TAG: ${{ github.event.inputs.image-tag || 'main' }}
TF_LOG: ${{ github.event.inputs.tf-log-level || 'info' }}
NO_PROVIDER_CREDENTIALS_azure: false

jobs:
# Used to skip cloud provider checks due to "jobs" not supporting {{ env }} variables contexts
check-for-credentials:
runs-on: ubuntu-latest
permissions:
id-token: write
contents: read

outputs:
provider_credentials_azure: ${{ steps.flag-check.outputs.provider_credentials_azure }}

steps:

- name: Check if user wants to run Azure integration based on credentials
id: flag-check
run: |
if [ "${{ env.NO_PROVIDER_CREDENTIALS_azure }}" == "true" ]; then
echo "::set-output name=provider_credentials_azure::0"
else
echo "::set-output name=provider_credentials_azure::1"
fi

test-azure-integration:
runs-on: ubuntu-latest
needs: check-for-credentials
if: ${{ needs.check-for-credentials.outputs.provider_credentials_azure == '1' }}
if: ${{ vars.SKIP_AZURE_INTEGRATION_TEST != 'true' }}
permissions:
id-token: write
contents: read
Expand Down Expand Up @@ -110,6 +87,3 @@ jobs:
NEBARI_SECRET__default_images__dask_worker: "quay.io/nebari/nebari-dask-worker:${{ env.NEBARI_IMAGE_TAG }}"
ARM_CLIENT_ID: ${{ env.ARM_CLIENT_ID }}
ARM_TENANT_ID: ${{ env.ARM_TENANT_ID }}
ARM_SUBSCRIPTION_ID: ${{ env.ARM_SUBSCRIPTION_ID }}
ARM_USE_OIDC: "true"
CLOUDFLARE_TOKEN: ${{ env.CLOUDFLARE_TOKEN }}
23 changes: 1 addition & 22 deletions .github/workflows/test_do_integration.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -31,33 +31,12 @@ env:
NEBARI_GH_BRANCH: ${{ github.event.inputs.branch || 'develop' }}
NEBARI_IMAGE_TAG: ${{ github.event.inputs.image-tag || 'main' }}
TF_LOG: ${{ github.event.inputs.tf-log-level || 'info' }}
NO_PROVIDER_CREDENTIALS_do: false


jobs:
# Used to skip cloud provider checks due to "jobs" not supporting {{ env }} variables contexts
check-for-credentials:
runs-on: ubuntu-latest
permissions:
id-token: write
contents: read
outputs:
provider_credentials_do: ${{ steps.flag-check.outputs.provider_credentials_do }}
steps:
- name: Check if user wants to run Digital Ocean integration based on credentials
id: flag-check
run: |
if [ "${{ env.NO_PROVIDER_CREDENTIALS_do }}" == "true" ]; then
echo "::set-output name=provider_credentials_do::0"
else
echo "::set-output name=provider_credentials_do::1"
fi


test-do-integration:
runs-on: ubuntu-latest
needs: check-for-credentials
if: ${{ needs.check-for-credentials.outputs.provider_credentials_do == '1' }}
if: ${{ vars.SKIP_DO_INTEGRATION_TEST != 'true' }}
permissions:
id-token: write
contents: read
Expand Down
22 changes: 1 addition & 21 deletions .github/workflows/test_gcp_integration.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -31,31 +31,11 @@ env:
NEBARI_GH_BRANCH: ${{ github.event.inputs.branch || 'develop' }}
NEBARI_IMAGE_TAG: ${{ github.event.inputs.image-tag || 'main' }}
TF_LOG: ${{ github.event.inputs.tf-log-level || 'info' }}
NO_PROVIDER_CREDENTIALS_gcp: false

jobs:
# Used to skip cloud provider checks due to "jobs" not supporting {{ env }} variables contexts
check-for-credentials:
runs-on: ubuntu-latest
permissions:
id-token: write
contents: read
outputs:
provider_credentials_gcp: ${{ steps.flag-check.outputs.provider_credentials_gcp }}
steps:
- name: Check if user wants to run GCP integration based on credentials
id: flag-check
run: |
if [ "${{ env.NO_PROVIDER_CREDENTIALS_gcp }}" == "true" ]; then
echo "::set-output name=provider_credentials_gcp::0"
else
echo "::set-output name=provider_credentials_gcp::1"
fi

test-gcp-integration:
runs-on: ubuntu-latest
needs: check-for-credentials
if: ${{ needs.check-for-credentials.outputs.provider_credentials_gcp == '1' }}
if: ${{ vars.SKIP_GCP_INTEGRATION_TEST != 'true' }}
permissions:
id-token: write
contents: read
Expand Down