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

Remove pr-test-libvirt job #1161

Merged
merged 1 commit into from
Jun 12, 2020

Conversation

pablochacin
Copy link
Contributor

Why is this PR needed?

This pipeline has duplicated logic with respect of pr-test.
pr-test now includes additional pr validation logic that makes no sense
to duplicate in the pr-test-libvirt job.

What does this PR do?

Removes the pr-test-libvirt job and the corresponding pipeline.

Anything else a reviewer needs to know?

The pr-test-libvirt must be manually removed from jenkins. Merging this PR will not automatically remove it.

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)

This pipeline has duplicated logic with respect of pr-test.
pr-test now includes additional pr validation logic that makes no sense
to duplicate in the pr-test-libvirt job.

The whole pr-test-libvirt pipeline must be reconsidered.

Signed-off-by: Pablo Chacin <pchacin@suse.com>
Copy link
Collaborator

@davidcassany davidcassany left a comment

Choose a reason for hiding this comment

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

LGTM

@pablochacin pablochacin merged commit 2068489 into SUSE:master Jun 12, 2020
mmnelemane pushed a commit to mmnelemane/skuba that referenced this pull request Jul 3, 2020
This pipeline has duplicated logic with respect of pr-test.
pr-test now includes additional pr validation logic that makes no sense
to duplicate in the pr-test-libvirt job.

The whole pr-test-libvirt pipeline must be reconsidered.

Signed-off-by: Pablo Chacin <pchacin@suse.com>
pablochacin added a commit to pablochacin/skuba that referenced this pull request Jul 9, 2020
This pipeline has duplicated logic with respect of pr-test.
pr-test now includes additional pr validation logic that makes no sense
to duplicate in the pr-test-libvirt job.

The whole pr-test-libvirt pipeline must be reconsidered.

Signed-off-by: Pablo Chacin <pchacin@suse.com>
(cherry picked from commit 2068489)
@pablochacin pablochacin deleted the remove-pr-test-libvirt-job branch July 10, 2020 10:25
mkoci-suse pushed a commit that referenced this pull request Feb 6, 2025
This pipeline has duplicated logic with respect of pr-test.
pr-test now includes additional pr validation logic that makes no sense
to duplicate in the pr-test-libvirt job.

The whole pr-test-libvirt pipeline must be reconsidered.

Signed-off-by: Pablo Chacin <pchacin@suse.com>
(cherry picked from commit 9aedd44)
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.

3 participants