Repository navigation
Re-apply "SCS-0128: Remove admin pass instance creation test" - #1253
toothstone wants to merge 1 commit into
Conversation
|
There was an agreement with @mbuechse that SCS-0128 will be amended to only require not failing any tests, so skipping tests is allowed. |
This was reverted in #1285, so I'll reopen this PR. There are valid security reasons to disallow instance creation with admin passwords, so SCS should not mandate this feature. |
|
You are right: on 2026-07-23 we decided to
And the latest change contradicts that, for the standard now says:
However, there is a priori no reason why the test case in question cannot be (made) one of those well-founded exceptions. Then again, it has to be discussed whether it should. The above-linked meeting minutes also state
I don't know what is meant here by 'is not a mandatory feature'. Mandatory -- mandated by whom? Maybe it was mandatory in OpenStack-powered Compute, maybe it wasn't. Regardless, I think we, the SCS community, should reconsider this matter and come to a (loose) consensus what we want:
So I'm adding this to the SIG agenda. |
I'm pretty sure this was an early version of the "correctness vs completeness" debate, before we had those terms for it.
|
|
Also, versioning the list of test cases would probably be helpful going forward, to avoid such ambiguity. |
…admin pas…" This reverts commit 0c8c8ce. Signed-off-by: Friedrich Zahn <[email protected]>
53de38c to
33651c4
Compare
We keep a list of exceptions in scs-0128. This list can be amended, and we (edit: CAN) do minor versions and keep a changelog. The admin-password thing was never on this list, and so it should be tested! |
Having the human-readable changelog is helpful to humans, but for the technical implementation of automated testing not so much. I'm currently doing "curl | tempest" with the test list, and that would be much nicer and more stable with a canonical URL per SCS-0128 version. |
|
I don't see the contradiction. We (can) have a stable URL per major version. That was the idea. |
Recreation of #1245 after it was erroneously merged and had to be reverted in #1250
Only applicable once #1250 is merged.