Skip to content

Introduce ability to unclutter Tests directory #921

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.

Sign up for GitHub

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

Jump to bottom
Open
mbuechse opened this issue May 19, 2025 · 1 comment · May be fixed by #928
Open

Introduce ability to unclutter Tests directory #921

mbuechse opened this issue May 19, 2025 · 1 comment · May be fixed by #928
Assignees
@mbuechse
Labels
SCS is standardized SCS is standardized

Comments

@mbuechse
Copy link
Contributor

mbuechse commented May 19, 2025 •
edited
Loading

Currently, we never delete anything from the certificate-scope spec files, and therefore, we also don't delete any old code paths. The ever growing amount of old, effectively unused code, however, makes refactorings increasingly costly.

As discussed in SIG Std/Cert of 2025-05-15, let us allow deleting really old stuff from scs-compatible-iaas.yaml and related files (since we have records of everything in git):

  • timeline entries that are no longer current (maybe when at least 6 months old?)
  • versions no longer referenced in the timeline (prior to v3)
  • modules no longer referenced by any version (e.g., old flavor naming)
  • code paths (cmd line switches) no longer referenced by any module (ditto)

To this end,

  • mention git in yaml file (comment)
  • mention “history in git” in scs-0003-v1
@mbuechse mbuechse self-assigned this May 19, 2025
@mbuechse mbuechse added the SCS is standardized SCS is standardized label May 19, 2025
@garloff
Copy link
Member

garloff commented May 21, 2025 •
edited
Loading

Purging old and no longer relevant things is beneficial, IMVHO.
Let's not be too aggressive about it, but your proposal looks well-balanced to me.

  • Things that are no longer valid (for at least 6 months).
  • Things that are no longer referenced.

mbuechse added a commit that referenced this issue May 21, 2025
@mbuechse
Introduce ability to prune old content from spec YAML
aa00678 
resolves #921

Signed-off-by: Matthias Büchse <[email protected]>
@mbuechse mbuechse linked a pull request May 21, 2025 that will close this issue
Open
@mbuechse mbuechse changed the title Unclutter Tests directory Introduce ability to unclutter Tests directory May 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mbuechse mbuechse

Labels
SCS is standardized SCS is standardized
Projects
Sovereign Cloud Stack
Status: Backlog
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Introduce ability to prune old content from spec YAML SovereignCloudStack/standards
2 participants
@mbuechse @garloff