Repository navigation
Revert "SCS-0128: Remove testcase that needs instance creation with admin password" - #1250
Conversation
|
Have you re-created my original PR somewhere? We need to track this somewhere. |
|
@toothstone What do you mean by 'recreated the PR'? It was never gone, and it's referenced in this PR's description. |
toothstone
left a comment
toothstone
left a comment
There was a problem hiding this comment.
Approving to comply with you wish to discuss this in the SIG, I stand by my assessment that this test case should be dropped ;)
I've opened #1253 to re-apply my original PR in case this one here gets merged. In the future I would appreciate it if that wouldn't fall onto me.
|
Out of curiosity, why does this commit not need to be signed-off? |
|
What does it mean for SCS-compatible IaaS v6 that this MR was not merged at the time of its stabilization? Is this feature part of it or not? |
…dmin pas…" This reverts commit 732cf2b.
0c8c8ce to
9876c09
Compare
|
Wow. How was this not merged? It's pretty obvious from the SIG minutes that it was intended to be merged, and that the original PR had been merged by accident. |
3 participants
Reverts #1245
I mistakenly merged this too quickly. Most of the certified providers successfully run this check. This should first be discussed in the SIG Std/Cert.