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

fix(ci): Remove useless jobs on agent6 context #33481

Merged
merged 5 commits into from
Jan 28, 2025

Conversation

chouetz
Copy link
Member

@chouetz chouetz commented Jan 28, 2025

What does this PR do?

Remove useless jobs for the Agent 6:

  • argo tests, covered by new-e2e for security, not needed for otlp
  • kitchen_security, failing on amazonlinux2023
  • SMP test

https://datadoghq.atlassian.net/browse/ACIX-540
https://datadoghq.atlassian.net/browse/ACIX-541

Motivation

Describe how you validated your changes

Possible Drawbacks / Trade-offs

Additional Notes

Copy link
Contributor

@scottopell scottopell left a comment

Choose a reason for hiding this comment

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

looks good from SMP side, you can also remove the test/regression folder similar to the e2e removal https://github.com/DataDog/datadog-agent/blob/main/test/regression/README.md

@chouetz chouetz force-pushed the nschweitzer/remove_useless_tests branch from e1f5b64 to 3efc685 Compare January 28, 2025 16:16
@chouetz chouetz requested a review from a team as a code owner January 28, 2025 16:16
@chouetz chouetz requested a review from a team as a code owner January 28, 2025 16:21
@chouetz chouetz requested a review from jeremy-hanna January 28, 2025 16:21
@pr-commenter
Copy link

pr-commenter bot commented Jan 28, 2025

Test changes on VM

Use this command from test-infra-definitions to manually test this PR changes on a VM:

inv create-vm --pipeline-id=54211451 --os-family=ubuntu

@chouetz chouetz added changelog/no-changelog qa/no-code-change No code change in Agent code requiring validation labels Jan 28, 2025
@chouetz chouetz merged commit 5c57563 into 6.53.x Jan 28, 2025
144 of 148 checks passed
@chouetz chouetz deleted the nschweitzer/remove_useless_tests branch January 28, 2025 20:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants