Skip to content

[pull] master from openshift:master#1748

Merged
pull[bot] merged 6 commits intoakram:masterfrom
openshift:master
Aug 7, 2024
Merged

[pull] master from openshift:master#1748
pull[bot] merged 6 commits intoakram:masterfrom
openshift:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Aug 7, 2024

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

sosiouxme and others added 6 commits August 5, 2024 13:52
This reverts commit f56cefc.

Removing the vsphere exclusion as it may obscure useful signal. The
affected vsphere tests have not failed since this change, not even to
flake.
The vertical scaling test will currently timeout on waiting for the CPMS
status.readyReplicas to scale up to 4 where as in practice that may not happen.
Ensuring that the cluster membership is 3 and checking that the old member has
been removed is enough of a signal that vertical scaling has successfully completed
OCPBUGS-37837: vertical scaling test should not rely on CPMS replicas
This test does not accuratly capture the bug it references, the matcher would match on correct reasons to mark a mc as degraded thus giving false positives. Also the test is not needed because if the condition the test is checking for were ever to arise the install would simply fail before we ever come to this state. The bug the test references also has tests so we should be fine to remove this check since it's not providing any value.

Signed-off-by: ehila <ehila@redhat.com>
TRT-1688: Revert "allow etcd log test to flake under vsphere"
OCPEDGE-1182: fix: remove un-needed test
@pull pull bot added the ⤵️ pull label Aug 7, 2024
@pull pull bot merged commit 6c91cc4 into akram:master Aug 7, 2024
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.

3 participants