-
-
Notifications
You must be signed in to change notification settings - Fork 333
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
Make config linking non-unique #3128
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #3128 +/- ##
==========================================
+ Coverage 34.82% 34.87% +0.04%
==========================================
Files 228 228
Lines 14755 14772 +17
==========================================
+ Hits 5139 5152 +13
- Misses 9238 9242 +4
Partials 378 378 ☔ View full report in Codecov by Sentry. |
we should not do that ... this is a hotfix at most!! @qwerty287 could you check why the dublication did ocured in the first place. or at lesat add a dedicated pipelineConfigID and retrive it based on that. this table is just for n-m relation mapping and such should never have a dublicate!!! |
Well, I know that this shouldn't occur, but in this case, it was 100% expected. Just add two yaml files with exactly the same content (yes, I know this isn't a common use case). This triggers the unique constraints and fails, but I think it should work without issues. |
This reverts commit 106508a.
This reverts commit 106508a.
I found the root cause so i created a fix and add tests for that #3215 |
closes #3093