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
Updates controller-runtime, remove WriteSVIDToDir #7214
Updates controller-runtime, remove WriteSVIDToDir #7214
Conversation
@@ -236,11 +236,12 @@ func (m *AppManager) WaitUntilJobState(isState func(*batchv1.Job, error) bool) ( | |||
|
|||
var lastJob *batchv1.Job | |||
|
|||
waitErr := wait.PollImmediate(PollInterval, PollTimeout, func() (bool, error) { |
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.
wait.PollImmediate
has been deprecated.
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #7214 +/- ##
==========================================
+ Coverage 62.20% 62.35% +0.15%
==========================================
Files 240 240
Lines 22112 22053 -59
==========================================
- Hits 13755 13752 -3
+ Misses 7210 7157 -53
+ Partials 1147 1144 -3 ☔ View full report in Codecov by Sentry. |
opts.ByObject = getTransformerFunctions(podSelector) | ||
if len(namespace) > 0 { | ||
opts.DefaultNamespaces = map[string]cache.Config{ | ||
namespace: {}, |
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.
Not use namespace, is it really useful?
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.
Yes, this sets the namespaces which should be watched, see here.
d92e219
to
45fdb31
Compare
54f77c8
to
bcc7f08
Compare
8b787cd
to
b353470
Compare
cc2542a
to
fd36f23
Compare
9c14eb0
to
e70d815
Compare
PR updates go modules, specifically controller-runtime to v0.16.3 so that the webhook server can be configured to use TLS options. This removes the need for the operator to write the SVID to disk and maintain it in memory. PR removes the WriteSVIDToDir option from security. Operator cache has been updated to use new controller-runtime types. Signed-off-by: joshvanl <me@joshvanl.dev>
Signed-off-by: joshvanl <me@joshvanl.dev>
Signed-off-by: joshvanl <me@joshvanl.dev>
Signed-off-by: joshvanl <me@joshvanl.dev>
v2 Signed-off-by: joshvanl <me@joshvanl.dev>
8beeee7
to
dd3f6b4
Compare
/test-version-skew |
Dapr Version Skew integration test (control-plane-master - 1.12.4)Commit ref: 5621210 ❌ Version Skew tests failedPlease check the logs for details on the error. |
Dapr Version Skew integration test (dapr-sidecar-master - 1.12.4)Commit ref: 5621210 ✅ Version Skew tests passed |
Dapr Version Skew e2e test (dapr-sidecar-master - 1.12.4)Commit ref: 5621210 ✅ Version Skew tests passed |
Dapr Version Skew e2e test (control-plane-master - 1.12.4)Commit ref: 5621210 ❌ Version Skew tests failedPlease check the logs for details on the error. |
/version-skew-test |
/test-version-skew |
Dapr Version Skew integration test (dapr-sidecar-master - 1.12.4)Commit ref: 880e34e ✅ Version Skew tests passed |
Dapr Version Skew integration test (control-plane-master - 1.12.4)Commit ref: 880e34e ✅ Version Skew tests passed |
Dapr Version Skew e2e test (dapr-sidecar-master - 1.12.4)Commit ref: 880e34e ❌ Version Skew tests failedPlease check the logs for details on the error. |
Dapr Version Skew e2e test (control-plane-master - 1.12.4)Commit ref: 880e34e ✅ Version Skew tests passed |
/test-version-skew |
Dapr Version Skew integration test (dapr-sidecar-master - 1.12.4)Commit ref: c899877 ✅ Version Skew tests passed |
Dapr Version Skew e2e test (dapr-sidecar-master - 1.12.4)Commit ref: c899877 ✅ Version Skew tests passed |
Dapr Version Skew integration test (control-plane-master - 1.12.4)Commit ref: c899877 ✅ Version Skew tests passed |
Dapr Version Skew e2e test (control-plane-master - 1.12.4)Commit ref: c899877 ✅ Version Skew tests passed |
PR updates go modules, specifically controller-runtime to v0.16.3 so that the webhook server can be configured to use TLS options. This removes the need for the operator to write the SVID to disk and maintain it in memory.
PR removes the WriteSVIDToDir option from security.
Operator cache has been updated to use new controller-runtime types.
Related: #5756