-
Notifications
You must be signed in to change notification settings - Fork 17
Generator enhancements #674
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
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #674 +/- ##
==========================================
+ Coverage 83.40% 83.48% +0.07%
==========================================
Files 183 183
Lines 9005 9064 +59
Branches 1472 1487 +15
==========================================
+ Hits 7511 7567 +56
- Misses 1229 1232 +3
Partials 265 265 ☔ View full report in Codecov by Sentry. |
a33fea7
to
a28659c
Compare
a6316cd
to
c70924e
Compare
Updated existing migrations with fixes from workfloworchestrator/orchestrator-core#674
Signed-off-by: Mark90 <[email protected]>
Signed-off-by: Mark90 <[email protected]>
c70924e
to
c9928c0
Compare
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.
Bit too many nested for loops (and if statements) for my taste
product_block: - Fix imports of dependent product blocks in the same product - Detect invalid product block configuration: cyclic or multiple roots product: - Only attach the root product block to the product type migration: - Only relate the root product block to the product, instead of all blocks (product_product_blocks) - Fill depends_on_block_relations in correct order (product_block_relations) - Only create resource types for non-product block fields (product_block_resource_types) Signed-off-by: Mark90 <[email protected]>
Signed-off-by: Mark90 <[email protected]>
Signed-off-by: Mark90 <[email protected]>
- history: show current revision by default - upgrade: make revision a required argument Signed-off-by: Mark90 <[email protected]>
c9928c0
to
1cdc430
Compare
Generator enhancements
product_block
product
migration
product_product_blocks
)product_block_relations
)product_block_resource_types
)Please review the updated migrations in the diff, as well as in the example orchestrator workfloworchestrator/example-orchestrator#19
Other
db upgrade
mandatorydb history
command to indicate current revision by defaultCloses #651