Skip to content
This repository has been archived by the owner on Apr 7, 2022. It is now read-only.

[1LP][RFR] Updating blockers for test_ssa_schedule #8188

Merged
merged 1 commit into from
Nov 26, 2018

Conversation

jan-zmeskal
Copy link
Contributor

Blocker

@jan-zmeskal jan-zmeskal added the blockers-only To be applied on PRs which contain blocker modifications ONLY label Nov 22, 2018
@@ -592,8 +592,7 @@ def test_ssa_compliance(local_setup_provider, ssa_compliance_profile, ssa_vm,
@pytest.mark.rhv3
@pytest.mark.tier(2)
@pytest.mark.long_running
@pytest.mark.meta(blockers=[BZ(1578792, forced_streams=['5.8', '5.9'],
unblock=lambda provider: vm_system_type != 'redhat')])
@pytest.mark.meta(blockers=[GH('ManageIQ/integration_tests:8157')])
Copy link
Contributor

Choose a reason for hiding this comment

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

@jan-zmeskal I can see this PR 8157 has been closed. can you please cross-check it..?

Copy link
Contributor

Choose a reason for hiding this comment

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

@sbulage, this is not PR but GH issue

Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry for Typo :)

Copy link
Contributor

Choose a reason for hiding this comment

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

@sbulage, good point. @jan-zmeskal, as far as I understand you have this issue only against RHV, right ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi @sbulage, as far as I can tell, #8157 is still open as of me writing this (Friday 23rd of November). Regarding @izapolsk question, it's true. Just now I realized that other providers won't experience this problem since they don't need to set up Engine Management Relationship. I'll unblock it for other providers.

@@ -592,8 +592,7 @@ def test_ssa_compliance(local_setup_provider, ssa_compliance_profile, ssa_vm,
@pytest.mark.rhv3
@pytest.mark.tier(2)
@pytest.mark.long_running
@pytest.mark.meta(blockers=[BZ(1578792, forced_streams=['5.8', '5.9'],
unblock=lambda provider: vm_system_type != 'redhat')])
@pytest.mark.meta(blockers=[GH('ManageIQ/integration_tests:8157')])
Copy link
Contributor

Choose a reason for hiding this comment

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

@sbulage, this is not PR but GH issue

@izapolsk
Copy link
Contributor

PRT failed obtaining appliances. I've restarted PRT. lets wait for its result.

@izapolsk izapolsk changed the title [RFR][NOTEST] Updating blockers for test_ssa_schedule [1LP][RFR][NOTEST] Updating blockers for test_ssa_schedule Nov 22, 2018
@sbulage
Copy link
Contributor

sbulage commented Nov 22, 2018

@izapolsk it should not block all tests, only RHEV tests should be blocked right .?

@izapolsk izapolsk changed the title [1LP][RFR][NOTEST] Updating blockers for test_ssa_schedule [1LP][RFR] Updating blockers for test_ssa_schedule Nov 22, 2018
@izapolsk izapolsk merged commit 88ac683 into ManageIQ:master Nov 26, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
blockers-only To be applied on PRs which contain blocker modifications ONLY
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants