Closed akrejcir closed 1 month ago
/cc @0xFelix @codingben
Yes, the comment is true, but the behavior is not correct in my opinion. It is very unexpected from the user's point of view. Even when user specifies the URL parameter, the operator will keep the resources from the internal bundle, ignoring the URL.
Can you add a test?
+1 unit test, was about also to comment, is it covered by unit tests?
I've added the test.
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: 0xFelix
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/cherry-pick release-v0.20
@akrejcir: new pull request created: #1035
What this PR does / why we need it: Previously, this order of operations would trigger a bug:
.spec.commonInstancetypes.url
to a URL - The operator will deploy resources from the URL.spec.commonInstancetypes.url
- The operator will deploy resources from internal bundle.spec.commonInstancetypes.url
to the same URL - The operator will ignore the resources and not deploy anythingRelease note: