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

[v0.10] Skip checks on logs in sharding end-to-end tests #2838

Merged

Conversation

weyfonk
Copy link
Contributor

@weyfonk weyfonk commented Sep 11, 2024

  • Skip checks on logs in sharding tests

This prevents test failures when tests are run more than once, without logs being cleared (eg. without deployments being scaled down and back up) between runs, or when debug logs are disabled

  • Add shard ID checks on bundle and bundle deployment

Sharding end-to-end tests now check that Fleet resources created from a GitRepo bear the same shard ID value, through the fleet.cattle.io/shard-ref label, as specified in the GitRepo itself.

User resources contained in bundle deployments are not labelled with the shard ID.

  • Fix check on configmap absence

When checking for a config map not being created, we must check that it consistently doesn't exist, instead of verifying that it eventually (at a unique point in time) doesn't exist.

Backport of #2832 to v0.10.
Refers to #2828.

* Skip checks on logs in sharding tests

This prevents test failures when:
* tests are run more than once, without logs being cleared (eg. without
  deployments being scaled down and back up) between runs
* debug logs are disabled

* Add shard ID checks on bundle and bundle deployment

Sharding end-to-end tests now check that Fleet resources created from a
`GitRepo` bear the same shard ID value, through the
`fleet.cattle.io/shard-ref` label, as specified in the `GitRepo` itself.

User resources contained in bundle deployments are not labelled with the
shard ID.

* Fix check on configmap absence

When checking for a config map not being created, we must check that it
_consistently_ doesn't exist, instead of verifying that it eventually
(at a unique point in time) doesn't exist.
@manno manno merged commit 4953fe7 into rancher:release/v0.10 Sep 13, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants