Skip to content

Commit b396961

Browse files
authored
2 parents 899965d + 4c34444 commit b396961

File tree

1,347 files changed

+201300
-65029
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

1,347 files changed

+201300
-65029
lines changed

.github/workflows/README.md

+15
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,11 @@ When approved this pushes the PR to the testing pipeline,
5555
thus starting the cli integ test build.
5656
Owner: Core CDK team
5757

58+
### Initial Priority Assignment
59+
60+
[project-prioritization-assignment.yml](project-prioritization-assignment.yml): GitHub action for automatically adding PR's with priorities to the project priority board based on their labels.
61+
Owner: CDK Support team
62+
5863
## Issue Triggered
5964

6065
### Closed Issue Message
@@ -103,3 +108,13 @@ Owner: Core CDK team
103108

104109
[update-contributors.yml](update-contributors.yml): GitHub action that runs monthly to create a pull request for updating a CONTRIBUTORS file with the top contributors.
105110
Owner: Core CDK team
111+
112+
### R2 Priority Assignment
113+
114+
[project-prioritization-r2-assignment.yml](project-prioritization-r2-assignment.yml): GitHub action that runs every 6 hours to add PR's to the priority project board that satisfies R2 Priority.
115+
Owner: CDK Support team
116+
117+
### R5 Priority Assignment
118+
119+
[project-prioritization-r5-assignment.yml](project-prioritization-r5-assignment.yml): GitHub action that runs every day to add PR's to the priority project board that satisfies R5 Priority.
120+
Owner: CDK Support team

.github/workflows/github-merit-badger.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -17,4 +17,4 @@ jobs:
1717
badges: '[beginning-contributor,repeat-contributor,valued-contributor,admired-contributor,star-contributor,distinguished-contributor]'
1818
thresholds: '[0,3,6,13,25,50]'
1919
badge-type: 'achievement'
20-
ignore-usernames: '[rix0rrr,iliapolo,otaviomacedo,kaizencc,comcalvi,TheRealAmazonKendra,mrgrain,pahud,kellertk,ashishdhingra,HBobertz,sumupitchayan,colifran,khushail,moelasmar,paulhcsun,GavinZZ,aaythapa,xazhao,gracelu0,jfuss,shikha372,kirtishrinkhala,godwingrs22,bergjaak,IanKonlog,Leo10Gama,samson-keung,scorbiere,michelle-wangg,jiayiwang7,1kaileychen,saiyush,5d,iankhou,aws-cdk-automation,dependabot[bot],mergify[bot]]'
20+
ignore-usernames: '[rix0rrr,iliapolo,otaviomacedo,kaizencc,comcalvi,TheRealAmazonKendra,mrgrain,pahud,kellertk,ashishdhingra,HBobertz,colifran,khushail,moelasmar,paulhcsun,GavinZZ,aaythapa,xazhao,gracelu0,jfuss,shikha372,kirtishrinkhala,godwingrs22,bergjaak,IanKonlog,Leo10Gama,samson-keung,scorbiere,michelle-wangg,jiayiwang7,1kaileychen,saiyush,5d,iankhou,aws-cdk-automation,dependabot[bot],mergify[bot]]'

.github/workflows/pr-linter-trigger.yml .github/workflows/pr-linter-review-trigger.yml

+9
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,12 @@
1+
# Re-evaluate the PR linter after reviews. This is used to upgrade the label
2+
# of a PR to `needs-maintainer-review` after a trusted community members leaves
3+
# an approving review.
4+
#
5+
# Unprivileged workflow that runs in the context of the PR, when a review is changed.
6+
#
7+
# Save the PR number, and download it again in the PR Linter workflow which
8+
# needs to run in privileged `workflow_run` context (but then must restore the
9+
# PR context).
110
name: PR Linter Trigger
211

312
on:

.github/workflows/pr-linter.yml

+28-29
Original file line numberDiff line numberDiff line change
@@ -11,11 +11,19 @@ on:
1111
- opened
1212
- synchronize
1313
- reopened
14+
15+
# Triggered from a separate job when a review is added
1416
workflow_run:
1517
workflows: [PR Linter Trigger]
1618
types:
1719
- completed
18-
status:
20+
21+
# Trigger when a status is updated (CodeBuild leads to statuses)
22+
status: {}
23+
24+
# Trigger when a check suite is completed (GitHub actions and CodeCov create checks)
25+
check_suite:
26+
types: [completed]
1927

2028
jobs:
2129
download-if-workflow-run:
@@ -26,39 +34,29 @@ jobs:
2634
# if conditions on all individual steps because subsequent jobs depend on this job
2735
# and we cannot skip it entirely
2836
steps:
29-
- name: 'Download artifact'
37+
- name: 'Download workflow_run artifact'
3038
if: github.event_name == 'workflow_run'
31-
uses: actions/github-script@v7
39+
uses: dawidd6/action-download-artifact@v7
3240
with:
33-
script: |
34-
let allArtifacts = await github.rest.actions.listWorkflowRunArtifacts({
35-
owner: context.repo.owner,
36-
repo: context.repo.repo,
37-
run_id: context.payload.workflow_run.id,
38-
});
39-
let matchArtifact = allArtifacts.data.artifacts.filter((artifact) => {
40-
return artifact.name == "pr_info"
41-
})[0];
42-
let download = await github.rest.actions.downloadArtifact({
43-
owner: context.repo.owner,
44-
repo: context.repo.repo,
45-
artifact_id: matchArtifact.id,
46-
archive_format: 'zip',
47-
});
48-
let fs = require('fs');
49-
fs.writeFileSync(`${process.env.GITHUB_WORKSPACE}/pr_info.zip`, Buffer.from(download.data));
50-
- name: 'Unzip artifact'
51-
if: github.event_name == 'workflow_run'
52-
run: unzip pr_info.zip
41+
run_id: ${{ github.event.workflow_run.id }}
42+
name: pr_info
43+
path: pr/
44+
search_artifacts: true
5345

54-
- name: 'Make GitHub output'
46+
- name: 'Determine PR info'
47+
# PR info comes from the artifact if downloaded, or GitHub context if not.
5548
if: github.event_name == 'workflow_run'
5649
id: 'pr_output'
5750
run: |
58-
echo "cat pr_number"
59-
echo "pr_number=$(cat pr_number)" >> "$GITHUB_OUTPUT"
60-
echo "cat pr_sha"
61-
echo "pr_sha=$(cat pr_sha)" >> "$GITHUB_OUTPUT"
51+
if [[ ! -f pr/pr_number ]]; then
52+
echo "${{ github.event.pull_request.number }}" > pr/pr_number
53+
fi
54+
if [[ ! -f pr/pr_sha ]]; then
55+
echo "${{ github.event.pull_request.head.sha }}" > pr/pr_sha
56+
fi
57+
cat pr/*
58+
echo "pr_number=$(cat pr/pr_number)" >> "$GITHUB_OUTPUT"
59+
echo "pr_sha=$(cat pr/pr_sha)" >> "$GITHUB_OUTPUT"
6260
6361
validate-pr:
6462
# Necessary to have sufficient permissions to write to the PR
@@ -67,6 +65,7 @@ jobs:
6765
pull-requests: write
6866
statuses: read
6967
issues: read
68+
checks: read
7069
runs-on: ubuntu-latest
7170
needs: download-if-workflow-run
7271
steps:
@@ -80,7 +79,7 @@ jobs:
8079
uses: ./tools/@aws-cdk/prlint
8180
env:
8281
GITHUB_TOKEN: ${{ secrets.PROJEN_GITHUB_TOKEN }}
83-
# PR_NUMBER and PR_SHA is empty if triggered by pull_request_target, since we already have that info
8482
PR_NUMBER: ${{ needs.download-if-workflow-run.outputs.pr_number }}
8583
PR_SHA: ${{ needs.download-if-workflow-run.outputs.pr_sha }}
84+
LINTER_LOGIN: ${{ vars.LINTER_LOGIN }}
8685
REPO_ROOT: ${{ github.workspace }}
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
name: PR Prioritization
2+
on:
3+
pull_request_target:
4+
types:
5+
- labeled
6+
- opened
7+
- reopened
8+
- synchronize
9+
- ready_for_review
10+
11+
jobs:
12+
prioritize:
13+
if: github.repository == 'aws/aws-cdk'
14+
runs-on: ubuntu-latest
15+
steps:
16+
- uses: actions/checkout@v4
17+
- name: Add PR to Project & Set Priority
18+
uses: actions/github-script@v7
19+
with:
20+
github-token: ${{ secrets.PROJEN_GITHUB_TOKEN }}
21+
script: |
22+
const script = require('./scripts/prioritization/assign-priority.js')
23+
await script({github, context})
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
name: PR Prioritization R2 Check
2+
on:
3+
schedule:
4+
- cron: '0 */6 * * 1-5' # Runs every 6 hours during weekdays
5+
workflow_dispatch: # Manual trigger
6+
7+
jobs:
8+
update_project_status:
9+
if: github.repository == 'aws/aws-cdk'
10+
runs-on: ubuntu-latest
11+
steps:
12+
- uses: actions/checkout@v4
13+
14+
- name: Check and assign R2 Priority to PRs
15+
uses: actions/github-script@v7
16+
with:
17+
github-token: ${{ secrets.PROJEN_GITHUB_TOKEN }}
18+
script: |
19+
const script = require('./scripts/prioritization/assign-r2-priority.js')
20+
await script({github})
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
name: PR Prioritization R5 Check
2+
on:
3+
schedule:
4+
- cron: '0 6 * * 1-5' # Runs at 6AM every day during weekdays
5+
workflow_dispatch: # Manual trigger
6+
7+
jobs:
8+
update_project_status:
9+
if: github.repository == 'aws/aws-cdk'
10+
runs-on: ubuntu-latest
11+
steps:
12+
- uses: actions/checkout@v4
13+
- name: Check and Assign R5 Priority to PRs
14+
uses: actions/github-script@v7
15+
with:
16+
github-token: ${{ secrets.PROJEN_GITHUB_TOKEN }}
17+
script: |
18+
const script = require('./scripts/prioritization/assign-r5-priority.js')
19+
await script({github})

.mergify.yml

+7-1
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ queue_rules:
1212
- -closed
1313
- "#approved-reviews-by>=1"
1414
- -approved-reviews-by~=author
15+
# This is important! It makes the PR Linter work.
1516
- "#changes-requested-reviews-by=0"
1617
- status-success~=AWS CodeBuild us-east-1
1718
- status-success=validate-pr
@@ -30,6 +31,7 @@ queue_rules:
3031
- -closed
3132
- "#approved-reviews-by>=1"
3233
- -approved-reviews-by~=author
34+
# This is important! It makes the PR Linter work.
3335
- "#changes-requested-reviews-by=0"
3436
- status-success~=AWS CodeBuild us-east-1
3537
- status-success=validate-pr
@@ -43,7 +45,7 @@ pull_request_rules:
4345
label:
4446
add: [ contribution/core ]
4547
conditions:
46-
- author~=^(rix0rrr|iliapolo|otaviomacedo|kaizencc|comcalvi|TheRealAmazonKendra|mrgrain|pahud|ashishdhingra|kellertk|HBobertz|sumupitchayan|colifran|moelasmar|paulhcsun|GavinZZ|aaythapa|xazhao|gracelu0|jfuss|shikha372|kirtishrinkhala|godwingrs22|bergjaak|samson-keung|IanKonlog|Leo10Gama|scorbiere|michelle-wangg|jiayiwang7|1kaileychen|saiyush|5d|iankhou)$
48+
- author~=^(rix0rrr|iliapolo|otaviomacedo|kaizencc|comcalvi|TheRealAmazonKendra|mrgrain|pahud|ashishdhingra|kellertk|HBobertz|colifran|moelasmar|paulhcsun|GavinZZ|aaythapa|xazhao|gracelu0|jfuss|shikha372|kirtishrinkhala|godwingrs22|bergjaak|samson-keung|IanKonlog|Leo10Gama|scorbiere|michelle-wangg|jiayiwang7|1kaileychen|saiyush|5d|iankhou)$
4749
- -label~="contribution/core"
4850
- name: automatic merge
4951
actions:
@@ -61,6 +63,7 @@ pull_request_rules:
6163
- author!=dependabot-preview[bot]
6264
- "#approved-reviews-by>=1"
6365
- -approved-reviews-by~=author
66+
# This is important! It makes the PR Linter work.
6467
- "#changes-requested-reviews-by=0"
6568
- status-success~=AWS CodeBuild us-east-1
6669
- status-success=validate-pr
@@ -81,6 +84,7 @@ pull_request_rules:
8184
- author!=dependabot-preview[bot]
8285
- "#approved-reviews-by>=2"
8386
- -approved-reviews-by~=author
87+
# This is important! It makes the PR Linter work.
8488
- "#changes-requested-reviews-by=0"
8589
- status-success~=AWS CodeBuild us-east-1
8690
- status-success=validate-pr
@@ -101,6 +105,7 @@ pull_request_rules:
101105
- author!=dependabot-preview[bot]
102106
- "#approved-reviews-by>=1"
103107
- -approved-reviews-by~=author
108+
# This is important! It makes the PR Linter work.
104109
- "#changes-requested-reviews-by=0"
105110
- status-success~=AWS CodeBuild us-east-1
106111
- status-success=validate-pr
@@ -140,6 +145,7 @@ pull_request_rules:
140145
- -closed
141146
- author~=dependabot
142147
- "#approved-reviews-by>=1"
148+
# This is important! It makes the PR Linter work.
143149
- "#changes-requested-reviews-by=0"
144150
- status-success~=AWS CodeBuild us-east-1
145151
- status-success=validate-pr

CHANGELOG.v2.alpha.md

+30
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,36 @@
22

33
All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
44

5+
6+
## [2.177.0-alpha.0](https://github.com/aws/aws-cdk/compare/v2.176.0-alpha.0...v2.177.0-alpha.0) (2025-01-24)
7+
8+
9+
### ⚠ BREAKING CHANGES TO EXPERIMENTAL FEATURES
10+
11+
* **glue-alpha:** Developers must refactor their existing Job
12+
instantiation method calls to choose the right job type and language,
13+
and use the new constants static values to define the associated Job
14+
configuration settings. See the RFC and/or new README for examples.
15+
16+
### Description of how you validated changes
17+
18+
Increased unit test coverage to > 90%, consulted with Glue service team
19+
on best practices and sane defaults, updated integration tests.
20+
21+
### Features
22+
23+
* **amplify-alpha:** throw `ValidationError` instead of untyped errors ([#33141](https://github.com/aws/aws-cdk/issues/33141)) ([a7cd9eb](https://github.com/aws/aws-cdk/commit/a7cd9ebc55f8fd70a469aea7dcf1c16919475982)), closes [#32569](https://github.com/aws/aws-cdk/issues/32569)
24+
25+
26+
### Bug Fixes
27+
28+
* **custom-resource-handlers:** do not allow unauthorized connection for iam OIDC connection (under feature flag) ([#32921](https://github.com/aws/aws-cdk/issues/32921)) ([3e4f377](https://github.com/aws/aws-cdk/commit/3e4f3773bfa48b75bf0adc7d53d46bbec7714a9e)), closes [#32920](https://github.com/aws/aws-cdk/issues/32920)
29+
30+
31+
### Code Refactoring
32+
33+
* **glue-alpha:** Refactored glue-alpha L2 CDK construct RFC 0497 ([#32521](https://github.com/aws/aws-cdk/issues/32521)) ([1a18dc9](https://github.com/aws/aws-cdk/commit/1a18dc951a3946430231b685bd3584f62055127c))
34+
535
## [2.176.0-alpha.0](https://github.com/aws/aws-cdk/compare/v2.175.1-alpha.0...v2.176.0-alpha.0) (2025-01-15)
636

737

0 commit comments

Comments
 (0)