Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multiple flavor-manager-input.py scripts under /Tests #667

Open
martinmo opened this issue Jul 19, 2024 · 2 comments
Open

Multiple flavor-manager-input.py scripts under /Tests #667

martinmo opened this issue Jul 19, 2024 · 2 comments
Labels
bug Something isn't working SCS-VP10 Related to tender lot SCS-VP10

Comments

@martinmo
Copy link
Member

scs-0103-v1 mentions

The openstack-flavor-manager is able to create all standard, mandatory SCS flavors for you. It takes input that can be generated by flavor-manager-input.py.

(https://docs.scs.community/standards/scs-0103-v1-standard-flavors#operational-tooling)

There are two scripts with that name:

$ fd flavor-manager-input
Tests/iaas/flavor-naming/flavor-manager-input.py
Tests/iaas/standard-flavors/flavor-manager-input.py

Which one is to be used, or can one of it be renamed or even deleted?

@martinmo martinmo added the bug Something isn't working label Jul 19, 2024
@mbuechse
Copy link
Contributor

The standard in question is supposed to refer to the latter script. The directory standard-flavors corresponds to that standard, while flavor-naming (where the other script resides) corresponds to scs-0100-vN.

  • We could rename the directories using the scs-NNNN codes to make the correspondence more obvious.
  • We can at some point in the future rename or delete the former script. I think it's no longer being used, but kept for the sake of compatibility.

@mbuechse mbuechse added the SCS-VP10 Related to tender lot SCS-VP10 label Jul 19, 2024
@martinmo
Copy link
Member Author

We could rename the directories using the scs-NNNN codes to make the correspondence more obvious.

+1

(IMHO, this could actually be done for all the scripts in Tests/, but this would be a bigger change.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working SCS-VP10 Related to tender lot SCS-VP10
Projects
None yet
Development

No branches or pull requests

2 participants