Skip to content

Commit

Permalink
Moving the env vars to the deploy step, Update to Secrets instead of …
Browse files Browse the repository at this point in the history
…Vars for helix admin api keys (adobecom#16)

* Moving the env vars to the deploy step

* Update to Secrets instead of Vars for helix admin api keys

* Adding logs for finding Graybox files and Excel updates

---------

Co-authored-by: sukamat <sukamat@adobe.com>
  • Loading branch information
arshadparwaiz and sukamat authored May 21, 2024
1 parent 2fafd5d commit d8627d1
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 9 deletions.
17 changes: 8 additions & 9 deletions .github/workflows/deploy_stage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,13 @@ jobs:
os: ${{ matrix.os }}
version: 10.x.x
- name: Build
env:
AIO_RUNTIME_NAMESPACE: ${{ secrets.AIO_RUNTIME_NAMESPACE_STAGE }}
uses: adobe/aio-apps-action@3.3.0
with:
os: ${{ matrix.os }}
command: build
- name: Deploy
env:
AIO_RUNTIME_NAMESPACE: ${{ secrets.AIO_RUNTIME_NAMESPACE_STAGE }}
AIO_RUNTIME_AUTH: ${{ secrets.AIO_RUNTIME_AUTH_STAGE }}
Expand All @@ -44,7 +51,7 @@ jobs:
CERT_THUMB_PRINT: ${{ secrets.CERT_THUMB_PRINT }}
CLIENT_ID: ${{ secrets.CLIENT_ID }}
ENABLE_PREVEW: ${{ vars.ENABLE_PREVEW }}
HELIX_ADMIN_API_KEYS: ${{ vars.HELIX_ADMIN_API_KEYS }}
HELIX_ADMIN_API_KEYS: ${{ secrets.HELIX_ADMIN_API_KEYS }}
GRAYBOX_USER_GROUPS: ${{ vars.GRAYBOX_USER_GROUPS }}
GROUP_CHECK_URL: ${{ vars.GROUP_CHECK_URL }}
MAX_BULK_PREVIEW_CHECKS: ${{ vars.MAX_BULK_PREVIEW_CHECKS }}
Expand All @@ -54,14 +61,6 @@ jobs:
SPLUNK_HEC__HEC_TOKEN: ${{ secrets.SPLUNK_HEC_TOKEN_STAGE }}
TENANT_ID: ${{ secrets.TENANT_ID }}
uses: adobe/aio-apps-action@3.3.0
with:
os: ${{ matrix.os }}
command: build
- name: Deploy
env:
AIO_RUNTIME_NAMESPACE: ${{ secrets.AIO_RUNTIME_NAMESPACE_STAGE }}
AIO_RUNTIME_AUTH: ${{ secrets.AIO_RUNTIME_AUTH_STAGE }}
uses: adobe/aio-apps-action@3.3.0
with:
os: ${{ matrix.os }}
command: deploy
Expand Down
1 change: 1 addition & 0 deletions actions/graybox/promote-worker.js
Original file line number Diff line number Diff line change
Expand Up @@ -202,6 +202,7 @@ async function findAllGrayboxFiles({
const uri = `${baseURI}${gbFolders.shift()}:/children?$top=${MAX_CHILDREN}`;
// eslint-disable-next-line no-await-in-loop
const res = await fetchWithRetry(uri, options);
logger.info(`Find all Graybox files URI: ${uri} \nResponse: ${res.ok}`);
if (res.ok) {
// eslint-disable-next-line no-await-in-loop
const json = await res.json();
Expand Down
1 change: 1 addition & 0 deletions actions/sharepoint.js
Original file line number Diff line number Diff line change
Expand Up @@ -62,6 +62,7 @@ async function getAuthorizedRequestOption({ body = null, json = true, method = '
async function executeGQL(url, opts) {
const options = await getAuthorizedRequestOption(opts);
const res = await fetchWithRetry(url, options);
getAioLogger().info(`Executing Excel Update with URL: ${url} \n Response: ${JSON.stringify(res.status)}`);
if (!res.ok) {
throw new Error(`Failed to execute ${url}`);
}
Expand Down

0 comments on commit d8627d1

Please sign in to comment.