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

UID2-3783 Add steps for eks e2e test #117

Merged
merged 27 commits into from
Aug 15, 2024
Merged

Conversation

cYKatherine
Copy link
Contributor

@cYKatherine cYKatherine commented Aug 12, 2024

@cYKatherine cYKatherine self-assigned this Aug 12, 2024
@cYKatherine cYKatherine changed the title Add steps for eks e2e test UID2-3783 Add steps for eks e2e test Aug 12, 2024
@cYKatherine cYKatherine force-pushed the kcc-UID2-3783-e2e-test-for-eks branch 10 times, most recently from ee046a3 to c7c9b5e Compare August 13, 2024 05:52
@@ -40,15 +40,15 @@ on:
description: 'Config: Operator branch for config'
type: string
default: main
core_branch:
branch_core:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Have to merge these inputs together as github has a limit of 10 inputs. See https://github.com/IABTechLab/uid2-operator/pull/889/files

repository: IABTechLab/uid2-admin
path: uid2-admin

- name: Checkout uid2-shared-actions repo
uses: actions/checkout@v4
with:
ref: v2
ref: kcc-UID2-3783-e2e-test-for-eks
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Revert before merging

@@ -291,6 +316,10 @@ jobs:
echo "uid2_e2e_phone_support=false" >> ${GITHUB_OUTPUT}
fi

- name: Test
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is for debugging purpose, make it sleep for 10 mins so that I can check if deployment is successful. Will revert before merging.

description: The operator key
required: true

# outputs:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Leave this for future case

@@ -8,9 +8,6 @@ inputs:
bore_url_optout:
description: The bore URL for optout service
required: true
bore_url_localstack:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is not used, removed in GCP


jq_string_update ${SECRET_JSON_FILE} core_base_url "http://${BORE_URL_CORE}"
jq_string_update ${SECRET_JSON_FILE} optout_base_url "http://${BORE_URL_OPTOUT}"
jq_string_update ${SECRET_JSON_FILE} api_token "${OPERATOR_KEY}"
Copy link
Contributor

Choose a reason for hiding this comment

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

neat

@cYKatherine cYKatherine force-pushed the kcc-UID2-3783-e2e-test-for-eks branch from 13e96df to 292611b Compare August 14, 2024 22:02
@cYKatherine cYKatherine force-pushed the kcc-UID2-3783-e2e-test-for-eks branch from 292611b to cece400 Compare August 14, 2024 23:44
@cYKatherine cYKatherine merged commit 20f0900 into main Aug 15, 2024
3 checks passed
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.

3 participants