-
Notifications
You must be signed in to change notification settings - Fork 103
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
Enable V2 by default #614
Enable V2 by default #614
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.
This pull request does not contain a valid label. Please add one of the following labels: bug, enhancement, refactoring, documentation, tooling
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.
No review necessary from docs on this one
@@ -0,0 +1,4 @@ | |||
# The following patches adds configure the DatadogAgents CRD to use v1alpha1 as the storage version. |
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.
# The following patches adds configure the DatadogAgents CRD to use v1alpha1 as the storage version. | |
# The following patches adds a configuration to the DatadogAgents CRD to use v1alpha1 as the storage version. |
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.
Fixed.
@@ -1,10 +1,10 @@ | |||
# The following patches adds configure the DatadogAgents CRD to use v2alpha1 as the storage version. | |||
# The following patches adds configure the DatadogAgents CRD to use v1alpha1 as the storage version. |
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.
# The following patches adds configure the DatadogAgents CRD to use v1alpha1 as the storage version. | |
# The following patches adds a configuration to the DatadogAgents CRD to use v1alpha1 as the storage version. |
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.
Fixed.
@@ -921,7 +921,7 @@ type DatadogAgentStatus struct { | |||
// DatadogAgent Deployment with the Datadog Operator. | |||
// +kubebuilder:object:root=true | |||
// +kubebuilder:subresource:status | |||
// +kubebuilder:unservedversion |
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.
should we add this flag to v1alpha1?
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.
Added and adapted manifests, patches, etc. 👍
docs/how-to-contribute.md
Outdated
@@ -32,18 +32,16 @@ make IMG=test/operator:test deploy | |||
|
|||
Note: `IMG` currently defaults to: `datadog/datadog-operator:latest` | |||
|
|||
## \[TMP\] how to test `v2alpha` | |||
## Test using `v2alpha` |
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.
should we rm this section header and merge the cert manager install cmd with the section above?
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.
Changed 👍
4ecb5b0
to
974d69a
Compare
Codecov Report
@@ Coverage Diff @@
## main #614 +/- ##
=======================================
Coverage 58.51% 58.51%
=======================================
Files 3 3
Lines 135 135
=======================================
Hits 79 79
Misses 43 43
Partials 13 13
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
Because it's default now.
974d69a
to
340bede
Compare
Just in case we need to test something with v1.
340bede
to
463975b
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.
Tested changes, all set.
What does this PR do?
Enables V2 by default. It's no longer necessary to run the commands with
KUSTOMIZE_CONFIG=config/test-v2
.I've added some configs to be able to test V1 in case we need it.