Skip to content

Commit 4103e79

Browse files
authored
chore: use different approach to authenticate user in request-cli-integ-test (#26523)
The original attempt didn't work. The push ended up being tried by `github-actions[bot]` which (correctly) failed. Try a different way that should also work. ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
1 parent 85f3499 commit 4103e79

File tree

2 files changed

+25
-4
lines changed

2 files changed

+25
-4
lines changed

.github/workflows/README.md

+23-3
Original file line numberDiff line numberDiff line change
@@ -5,62 +5,82 @@ These workflows and actions are configured in the AWS CDK GitHub repository.
55
## Pull Request Triggered
66

77
### Auto Approve
8+
89
[auto-approve.yml](auto-approve.yml): Approves merging PRs with the
910
`auto-approve` label.
1011
Owner: Core CDK team
1112

1213
### PR Linter
14+
1315
[pr-linter.yml](pr-linter.yml): Runs `tools/@aws-cdk-prlint` on each PR to
1416
check for correctness.
1517
Owner: Core CDK team
1618

1719
### v2-main PR automation
20+
1821
[v2-pull-request.yml](v2-pull-request.yml): Runs `pkglint` on merge forward PRs
1922
and commits the results.
2023
Owner: Core CDK team
2124

2225
### Label Assigner
26+
2327
[issue-label-assign.yml](issue-label-assign.yml): Github action for automatically adding labels and/or setting assignees when an Issue or PR is opened or edited based on user-defined Area
2428
Owner: CDK support team
2529

2630
### PR Labeler
27-
[pr-labeler.yml](pr-labeler.yml): GitHub action for automatically porting triage labels from issues
31+
32+
[pr-labeler.yml](pr-labeler.yml): GitHub action for automatically porting triage labels from issues
2833
linked in the PR description to the PR.
2934
Owner: Core CDK team
3035

3136
### GitHub Merit Badger
37+
3238
[github-merit-badger.yml](github-merit-badger.yml): GitHub action that adds 'merit badges' to pull
3339
requests based on the users prior contributions to the CDK.
3440
Owner: Core CDK team
3541

42+
### Request CLI Integ Test
43+
44+
[request-cli-integ-test.yml](request-cli-integ-test.yml):
45+
Checks for relevant changes to the CLI code and requests a deployment to the `test-pipeline` environment.
46+
When approved this pushes the PR to the testing pipeline,
47+
thus starting the cli integ test build.
48+
Owner: Core CDK team
49+
3650
## Issue Triggered
3751

3852
### Closed Issue Message
53+
3954
[closed-issue-message.yml](closed-issue-message.yml): Adds a reminder message
4055
to issues that are closed.
4156
Owner: CDK support team
4257

4358
### Label Assigner
59+
4460
[issue-label-assign.yml](issue-label-assign.yml): Github action for automatically adding labels and/or setting assignees when an Issue or PR is opened or edited based on user-defined Area
4561
Owner: CDK support team
4662

4763
## Scheduled Actions
4864

4965
### Issue Lifecycle Handling
66+
5067
[close-stale-issues.yml](close-stale-issues.yml): Handles labeling issues and
5168
PRs with `closing-soon`, `response-requested`, etc.
5269
Owner: CDK support team
5370

5471
### Yarn Upgrader
72+
5573
[yarn-upgrade.yml](yarn-upgrade.yml): Upgrades yarn dependencies and creates a
5674
patch file for downloading.
5775
Owner: Core CDK team
5876

5977
### Issue Reprioritizer
78+
6079
[issue-reprioritization.yml](issue-reprioritization.yml): GitHub action that labels `p2`
6180
issues as `p1` if a certain level of community engagement is met.
6281
Owner: Core CDK team
6382

6483
### Issue Metrics
65-
[issue-metrics.yml](issue-metrics.yml): GitHub action that runs monthly to report on metrics for issues created last month.
66-
Owner: Core CDK team
84+
85+
[repo-metrics.yml](repo-metrics.yml): GitHub action that runs monthly to report on metrics for issues and PRs created last month.
86+
Owner: Core CDK team

.github/workflows/request-cli-integ-test.yml

+2-1
Original file line numberDiff line numberDiff line change
@@ -35,14 +35,15 @@ jobs:
3535
- name: Checkout
3636
uses: actions/checkout@v3
3737
with:
38+
token: ${{ secrets.PROJEN_GITHUB_TOKEN }}
3839
ref: ${{ github.event.pull_request.head.ref }}
3940
repository: ${{ github.event.pull_request.head.repo.full_name }}
4041
- name: Submit to test-pipeline
4142
# Needs to run with PROJEN_GITHUB_TOKEN because we need permissions to force push the branch
4243
run: |
4344
git config --global user.name 'aws-cdk-automation'
4445
git config --global user.email '[email protected]'
45-
git push --force --atomic https://oauth2:${{ secrets.PROJEN_GITHUB_TOKEN }}@github.com/${{ github.repository }}.git FETCH_HEAD:test-main-pipeline
46+
git push --force --atomic ${{ github.repositoryUrl }} FETCH_HEAD:test-main-pipeline
4647
- name: Explain next steps
4748
uses: thollander/actions-comment-pull-request@v2
4849
with:

0 commit comments

Comments
 (0)