Skip to content

Conversation

aditighag
Copy link
Member

No description provided.

@aditighag aditighag requested a review from pchaigno July 3, 2025 17:16
@aditighag aditighag added the release-note/ci This PR makes changes to the CI. label Jul 3, 2025
@aditighag aditighag requested review from a team as code owners July 3, 2025 17:16
@aditighag aditighag requested a review from tamilmani1989 July 3, 2025 17:16
@aditighag aditighag requested a review from a team as a code owner July 3, 2025 17:18
@aditighag aditighag requested a review from thorn3r July 3, 2025 17:18
@aditighag aditighag marked this pull request as draft July 3, 2025 17:19
@aditighag
Copy link
Member Author

/ci-aks

1 similar comment
@aditighag
Copy link
Member Author

/ci-aks

@aditighag aditighag force-pushed the pr/aditighag/enable-kpr-aks-workflow branch from 8a367c7 to 03805c2 Compare July 7, 2025 21:22
@aditighag
Copy link
Member Author

/ci-aks

@aditighag aditighag closed this Jul 7, 2025
@aditighag aditighag reopened this Jul 7, 2025
@aditighag
Copy link
Member Author

aditighag added 2 commits July 8, 2025 12:34
... as a pre-req for Egress gateway

Signed-off-by: Aditi Ghag <aditi@cilium.io>
... as a pre-req for Egress gateway

Signed-off-by: Aditi Ghag <aditi@cilium.io>
@aditighag aditighag force-pushed the pr/aditighag/enable-kpr-aks-workflow branch from 03805c2 to fc34346 Compare July 8, 2025 19:35
@aditighag aditighag marked this pull request as ready for review July 8, 2025 19:35
@aditighag aditighag removed request for a team and thorn3r July 8, 2025 19:35
@aditighag
Copy link
Member Author

/test

pchaigno
pchaigno previously approved these changes Jul 8, 2025
Copy link
Member

@pchaigno pchaigno left a comment

Choose a reason for hiding this comment

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

I think a single commit might have been fine 😅 but LGTM!

@pchaigno pchaigno enabled auto-merge July 8, 2025 21:32
@aditighag aditighag changed the title Enable KPR and BPF masquerading ci-aks: Enable KPR and BPF masquerading Jul 9, 2025
@aditighag
Copy link
Member Author

@tamilmani1989 Can you ack the PR? Thanks!

@aditighag aditighag dismissed pchaigno’s stale review July 10, 2025 16:11

Approved by Tom

@aditighag
Copy link
Member Author

Oops, I dismissed wrong review. 😓

Copy link
Member

@pchaigno pchaigno left a comment

Choose a reason for hiding this comment

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

🙂

@pchaigno pchaigno closed this Jul 10, 2025
auto-merge was automatically disabled July 10, 2025 16:17

Pull request was closed

@pchaigno pchaigno reopened this Jul 10, 2025
@pchaigno
Copy link
Member

Reopened to try and trigger the Mergeability workflow that was stuck.

@pchaigno pchaigno added this pull request to the merge queue Jul 10, 2025
Merged via the queue into cilium:main with commit a3249ec Jul 10, 2025
123 of 132 checks passed
@aditighag aditighag deleted the pr/aditighag/enable-kpr-aks-workflow branch July 10, 2025 16:50
@tamilmani1989
Copy link
Contributor

@aditighag sorry late to party. may i know motivation for this change?

@aditighag aditighag added affects/v1.18 This issue affects v1.18 branch and removed affects/v1.18 This issue affects v1.18 branch labels Aug 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants