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

Config processing tests #1097

Merged
merged 108 commits into from
Apr 4, 2025
Merged

Config processing tests #1097

merged 108 commits into from
Apr 4, 2025

Conversation

rashmichandrashekar
Copy link
Contributor

@rashmichandrashekar rashmichandrashekar commented Mar 25, 2025

PR Description

Adding config processing tests for the scenarios listed here -
https://microsoft.sharepoint.com/teams/ContainerInsights/_layouts/15/Doc.aspx?sourcedoc={dc357060-d7e5-46a6-a599-88ec4fdee1de}&action=edit&wd=target%28Prometheus%20Metrics.one%7C29807f1f-70be-4902-b7f6-75b2540c23d3%2FConfigmap%20testing%20scenarios%7Cbae9a2e2-2b4e-431f-8fb6-6b0de29ae595%2F%29&wdorigin=NavigationUrl

Link to scheduled run - https://github-private.visualstudio.com/azure/_build/results?buildId=96095&view=results
comment: # (The below checklist is for PRs adding new features. If a box is not checked, add a reason why it's not needed.)

New Feature Checklist

  • List telemetry added about the feature.
  • Link to the one-pager about the feature.
  • List any tasks necessary for release (3P docs, AKS RP chart changes, etc.) after merging the PR.
  • Attach results of scale and perf testing.

Tests Checklist

  • Have end-to-end Ginkgo tests been run on your cluster and passed? To bootstrap your cluster to run the tests, follow these instructions.
    • Labels used when running the tests on your cluster:
      • operator
      • windows
      • arm64
      • arc-extension
      • fips
  • Have new tests been added? For features, have tests been added for this feature? For fixes, is there a test that could have caught this issue and could validate that the fix works?

@rashmichandrashekar rashmichandrashekar requested a review from a team as a code owner March 25, 2025 01:44
@rashmichandrashekar
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link

github-actions bot commented Apr 2, 2025

This PR is stale because it has been open 7 days with no activity. Remove stale label or comment or this will be closed in 5 days.

vishiy
vishiy previously approved these changes Apr 4, 2025
Copy link
Contributor

@vishiy vishiy left a comment

Choose a reason for hiding this comment

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

Thanks for adding config proc tests @rashmichandrashekar .

gracewehner
gracewehner previously approved these changes Apr 4, 2025
@rashmichandrashekar
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rashmichandrashekar rashmichandrashekar enabled auto-merge (squash) April 4, 2025 18:56
@rashmichandrashekar rashmichandrashekar merged commit 761e44c into main Apr 4, 2025
23 checks passed
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