Skip to content
This repository was archived by the owner on Feb 6, 2025. It is now read-only.

Remove addon upgrade test from pr pipeline #1299

Merged

Conversation

pablochacin
Copy link
Contributor

@pablochacin pablochacin commented Aug 3, 2020

Why is this PR needed?

The PR pipeline tests only check the changes introduced the PR do not break the ability to deploy a new cluster. The addon upgrade apply test does not comply with this objective.

Fixes https://github.com/SUSE/avant-garde/issues/1867

What does this PR do?

Removes the pr mark from the test_upgrade_addon_apply test to prevent its execution in the PR pipeline

Merge restrictions

(Please do not edit this)

We are in v4-maintenance phase, so we will restrict what can be merged to prevent unexpected surprises:

What can be merged (merge criteria):
    2 approvals:
        1 developer: code is fine
        1 QA: QA is fine
    there is a PR for updating documentation (or a statement that this is not needed)

Remove test that is not part of the minimal functionality to be tested
for approving a PR.

Signed-off-by: Pablo Chacin <pchacin@suse.com>
@jordimassaguerpla jordimassaguerpla merged commit 1637ade into SUSE:master Aug 3, 2020
mkoci-suse pushed a commit that referenced this pull request Feb 6, 2025
…om-pr

Remove addon upgrade test from pr pipeline
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants