Skip to content

DO-2058: Expose IMS org ID to aio-app-deployment deploy hooks - #194

Merged
tmthrgd-aligent merged 1 commit into
mainfrom
fix/DO-2058_add_missing_aio_deploy_env_var
Nov 5, 2026
Merged

tmthrgd-aligent merged 1 commit into
mainfrom
fix/DO-2058_add_missing_aio_deploy_env_var

Conversation

@tmthrgd-aligent

Copy link
Copy Markdown
Contributor

aio-commerce-lib-app's post-app-deploy hook reads project.org.ims_org_id from the AIO config and sends it as the x-gw-ims-org-id header when triggering the app upgrade. In CI that key is only populated from the environment, and the deploy step set AIO_PROJECT_ORG_ID (the numeric org ID) but not the IMS org ID, so the hook failed with:

HTTP 401: cannot authorize request, reason: missing x-gw-ims-org-id header

Map the existing AIO_IMS_ORG_ID secret to AIO_PROJECT_ORG_IMS__ORG__ID (__ is a literal _ in aio-lib-core-config env mapping).

Description of the proposed changes

Screenshots (if applicable)

Other solutions considered (if any)

Notes to PR author

⚠️ Please make sure the changes adhere to the guidelines mentioned in our contribution guide.

Notes to reviewers

ℹ️ When you've finished leaving feedback, please add a final comment to the PR tagging the author, letting them know that you have finished leaving feedback

Time Tracking

ABC-xxx: code review, select project XXXX

aio-commerce-lib-app's post-app-deploy hook reads `project.org.ims_org_id`
from the AIO config and sends it as the `x-gw-ims-org-id` header when
triggering the app upgrade. In CI that key is only populated from the
environment, and the deploy step set AIO_PROJECT_ORG_ID (the numeric org
ID) but not the IMS org ID, so the hook failed with:

  HTTP 401: cannot authorize request, reason: missing x-gw-ims-org-id header

Map the existing AIO_IMS_ORG_ID secret to AIO_PROJECT_ORG_IMS__ORG__ID
(`__` is a literal `_` in aio-lib-core-config env mapping).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@tmthrgd-aligent
tmthrgd-aligent requested a review from a team as a code owner November 5, 2026 05:34
@tmthrgd-aligent

Copy link
Copy Markdown
Contributor Author

There may be some additional environment variables we need to set, but that will only be clear once this change has been merged so the command can progress further.

@aaronmedina-dev aaronmedina-dev left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@tmthrgd-aligent
tmthrgd-aligent merged commit 7466d4f into main Nov 5, 2026
3 checks passed
@tmthrgd-aligent
tmthrgd-aligent deleted the fix/DO-2058_add_missing_aio_deploy_env_var branch November 5, 2026 23:18
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