-
Notifications
You must be signed in to change notification settings - Fork 1
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
Conversation
ee046a3
to
c7c9b5e
Compare
@@ -40,15 +40,15 @@ on: | |||
description: 'Config: Operator branch for config' | |||
type: string | |||
default: main | |||
core_branch: | |||
branch_core: |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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: |
There was a problem hiding this comment.
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: |
There was a problem hiding this comment.
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}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
neat
13e96df
to
292611b
Compare
292611b
to
cece400
Compare
Please review these two PRs together:
Checked working: