-
Notifications
You must be signed in to change notification settings - Fork 137
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
Add Verification of OneAgent Version #3405
Conversation
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.
LGTM
fa49a41
to
59b5230
Compare
59b5230
to
6cfc3e4
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #3405 +/- ##
==========================================
+ Coverage 57.23% 57.25% +0.01%
==========================================
Files 344 344
Lines 19783 19791 +8
==========================================
+ Hits 11323 11331 +8
Misses 7222 7222
Partials 1238 1238
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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.
LGTM
Co-authored-by: adam.orcholski <adam.orcholski@dynatrace.com>
see K8S-10190
Description
The operator should reject OneAgent versions that are not conformant to major.minor.patch pattern subset of SemVer. e.g. 1.0.0
How can this be tested?
Apply a DK with OneAgent version set to an invalid version such as
latest
,raw
or1.296.0-raw
and to a valid one such as 1.296.0