-
Notifications
You must be signed in to change notification settings - Fork 57
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
Phase out the Repository model from the KATalogus #2984
Conversation
Fix the integration tests Remove repository references in Rocky
Test upgrading and downgrading with uniqueness issue Set empty not-nullable foreign key to repository to seeded "LOCAL" on downgrades
… in other modules. Signed-off-by: Donny Peeters <donny@bitestreams.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me overall. I just had one suggestion. I didn't dive too deep into this since, as you mentioned, these changes are mostly removal or relocation of code. The modified and new tests are comprehensive and should give us confidence to merge this, well done 👍
Signed-off-by: Donny Peeters <donny@bitestreams.com>
Set type to str again since the values were not compatible
Signed-off-by: Donny Peeters <donny@bitestreams.com>
Fix other references to the app module that has been removed Update the other relevant documentation on seeding the KATalogus database, which is no longer needed Signed-off-by: Donny Peeters <donny@bitestreams.com>
757d05d
to
aa9f613
Compare
Checklist for QA:
What works:Seems to look good. Haven't found anything obvious:
What doesn't work:
Bug or feature?:
|
* main: (78 commits) Translations update from Hosted Weblate (#3048) Translations update from Hosted Weblate (#3018) Fix empty consumes of boefjes will trigger tasks in scheduler (#3017) Fixes text in secondary menu on scan profile detail page (#3035) chore: Resolves css-issues found by sonarcloud (#3034) Add raw AuthToken SQL migration (#3009) Translations update from Hosted Weblate (#3012) Rewrite xtdb-cli.py with "click" (#2957) Phase out the Repository model from the KATalogus (#2984) Fix merge conflicts in weblate (#3007) Translations update from Hosted Weblate (#2996) Reports: Fix select all OOIs (#2909) Adding IPv6 support to documentation for Docker setups (#2813) Translations update from Hosted Weblate (#2930) User documentation for reports (#2898) Fix task api status code response for malformed id in the scheduler (#2953) Add drill trace option in dnssec boefje (#2979) Updated packages (#2972) Update granian and remove workaround for fixed bug (#2980) Fix typing in boefjes/normalizers (#2933) ...
* main: (40 commits) Translations update from Hosted Weblate (#3091) feat: 📝 add API titles (#3055) Fixed 2 small mistakes in documentation (#3089) Documentation - developer and helper functionality documentation for xtdb-cli tool (#3023) fix: 🔧 update db normalize setting (#2777) Translations update from Hosted Weblate (#3048) Translations update from Hosted Weblate (#3018) Fix empty consumes of boefjes will trigger tasks in scheduler (#3017) Fixes text in secondary menu on scan profile detail page (#3035) chore: Resolves css-issues found by sonarcloud (#3034) Add raw AuthToken SQL migration (#3009) Translations update from Hosted Weblate (#3012) Rewrite xtdb-cli.py with "click" (#2957) Phase out the Repository model from the KATalogus (#2984) Fix merge conflicts in weblate (#3007) Translations update from Hosted Weblate (#2996) Reports: Fix select all OOIs (#2909) Adding IPv6 support to documentation for Docker setups (#2813) Translations update from Hosted Weblate (#2930) User documentation for reports (#2898) ...
Changes
This PR phases out the unused Repository model, in favor (later PRs) of a generic oci_image url for the boefje images, see #2983 and #2881. Most changes are either the removal or relocation of code and files, so do not let the number of
Files changed
distract you.Issue link
Closes #2983
Demo
This is a transparent change and adds no functionality. We could almost call it a refactor.
But, what is important is that migrating this change happens without issues. There are potential caveats here however when users have actually added Repository models, although changes are low that these are actually being used for the plugins since the API code that these models can connected has actually already been deleted a long time ago in this PR. Therefore, probably 80% of users will have no issues, 19% will have to look into their katalogus database to find unused repository entries they can just delete because they did not know about or forgot them, and 1% that's either running the old API or built their own should be told to contact us to help them migrate whatever they are running behind that API to the new OCI framework we introduced.
Hence, try to create some data, enable plugins, add organizations and then checkout this PR to see if there are any migration issues!
Code Checklist
Communication
.env
changes files if required and changed the.env-dist
accordingly.Checklist for code reviewers:
Copy-paste the checklist from the docs/source/templates folder into your comment.
Checklist for QA:
Copy-paste the checklist from the docs/source/templates folder into your comment.