Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove Builder and Jenkins job for IPAM tests and Skip irrelevant tests from Flexible IPAM tests #6991

Merged
merged 1 commit into from
Mar 3, 2025

Conversation

KMAnju-2021
Copy link
Contributor

@KMAnju-2021 KMAnju-2021 commented Feb 12, 2025

  1. adding --skip mode-irrelevant flag for the Flexible IPAM tests in order to run fewer tests.
  2. Remove the Flexible-IPAM builder and job from jenkins as IPAM tests are now handled by GitHub workflows.

@KMAnju-2021 KMAnju-2021 changed the title add skip flag to ipam workflow and remove stale code add --skip mode-irrelevant flag and remove stale code Feb 12, 2025
@KMAnju-2021 KMAnju-2021 changed the title add --skip mode-irrelevant flag and remove stale code add --skip mode-irrelevant flag to ipam and remove stale code Feb 12, 2025
@rajnkamr rajnkamr changed the title add --skip mode-irrelevant flag to ipam and remove stale code Remove Builder and Jenkins job for IPAM tests and Skip irrelevant tests from Flexible IPAM tests Feb 12, 2025
Copy link
Contributor

@antoninbas antoninbas left a comment

Choose a reason for hiding this comment

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

LGTM, but I'll wait for @XinShuYang's review before merging

XinShuYang
XinShuYang previously approved these changes Feb 19, 2025
Copy link
Contributor

@XinShuYang XinShuYang left a comment

Choose a reason for hiding this comment

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

LGTM

@XinShuYang XinShuYang added the action/backport Indicates a PR that requires backports. label Feb 19, 2025
antoninbas
antoninbas previously approved these changes Feb 19, 2025
@antoninbas
Copy link
Contributor

/test-all

@antoninbas
Copy link
Contributor

@KMAnju-2021 sorry, I forgot to merge this PR, could you rebase?

@antoninbas
Copy link
Contributor

/test-all

@KMAnju-2021 KMAnju-2021 requested a review from antoninbas March 2, 2025 14:46
@antoninbas antoninbas merged commit b7e0c38 into antrea-io:main Mar 3, 2025
59 of 64 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action/backport Indicates a PR that requires backports.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants