add delete working cert strategy#807
add delete working cert strategy#807yanjiaxin534 wants to merge 12 commits intoeclipse-symphony:remote-agentfrom
Conversation
fix go mod redis queue delete contains get & fix cert manager bug add cert manager add nil check for log add double check for cert manager
yanjiaxin534
force-pushed
the
remote-agent-refact-workingcert
branch
from
4692f2c to
7d2a41a
Compare
| err = c.waitForCertificateReady(ctx, id, namespace, secretName) | ||
| if err != nil { | ||
| tLog.ErrorfCtx(ctx, "V (Targets) : onGetCert failed waiting for certificate - %s", err.Error()) | ||
| _, getErr := c.TargetsManager.StateProvider.Get(ctx, getRequest) |
There was a problem hiding this comment.
Since we have cert manager now, we should use cert manager also in targets-vendor
There was a problem hiding this comment.
Thank you, Yes, we should use cert manager in target vendor too. I fix it now
fix fix function
yanjiaxin534
force-pushed
the
remote-agent-refact-workingcert
branch
from
596e871 to
bb93566
Compare
|
|
||
| type CertManager struct { | ||
| managers.Manager | ||
| StateProvider states.IStateProvider |
There was a problem hiding this comment.
I don't think we should use IStateProvider and ISecretProvider here. It is better you need to define ICertProvider interface, it has some functions such as CreateCert, DeleteCert, RotateCert.
And for OSSCertManager implementation, let's call it OSSCMCertProvider, it should hold a kubeclient, instead of using IStateProvider and ISecretProvider.
If you want to switch to another way to manage cert, it is impossible you just change IStateProvider and ISecretProvider.
And let solution-manager to expose certprovider to vendor.
3 participants
Delete redis queue log
{"caller":"github.com/eclipse-symphony/symphony/api/pkg/apis/v1alpha1/managers/solution.(*SolutionManager).cleanupRemoteTargetResourcesAfterDeletion:324","diagnostics":{"correlationId":"","resourceId":"","traceContext":{"spanId":"39cb97f9f8b1ef58","traceId":"82c8d633331402da3eb3e8a3f33272c0"}},"instance":"symphony-api-86d764ffc-m6m9l","level":"info","msg":" M (Solution): successfully deleted Redis queue test-mqtt-schedule-windows-target-default for deleted target test-mqtt-schedule-windows-target","scope":"coa.runtime","time":"2025-09-05T09:31:25.076090323Z","type":"log","ver":"0.48.34"}
Unsubscribe topic log
{"caller":"github.com/eclipse-symphony/symphony/api/pkg/apis/v1alpha1/managers/solution.(*SolutionManager).cleanupRemoteTargetResourcesAfterDeletion:314","diagnostics":{"correlationId":"","resourceId":"","traceContext":{"spanId":"39cb97f9f8b1ef58","traceId":"82c8d633331402da3eb3e8a3f33272c0"}},"instance":"symphony-api-86d764ffc-m6m9l","level":"info","msg":" M (Solution): successfully unsubscribed from MQTT topic symphony/request/test-mqtt-schedule-windows-target for deleted target test-mqtt-schedule-windows-target","scope":"coa.runtime","time":"2025-09-05T09:31:25.069574951Z","type":"log","ver":"0.48.34"}
delete working cert log
{"caller":"github.com/eclipse-symphony/symphony/api/pkg/apis/v1alpha1/managers/cert.(*CertManager).DeleteWorkingCert:189","diagnostics":{"correlationId":"f84c5a3d-d18b-4805-b783-2d10ad670f45","resourceId":"","traceContext":{"spanId":"5c29b5dfc7145ae6","traceId":"95e10eecb525eb0cef3707320447f98c"}},"instance":"symphony-api-86d764ffc-m6m9l","level":"info","msg":"Deleting working cert for target test-mqtt-schedule-windows-target in namespace default","scope":"coa.runtime","time":"2025-09-05T09:31:05.146110303Z","type":"log","ver":"0.48.34"}
{"caller":"github.com/eclipse-symphony/symphony/api/pkg/apis/v1alpha1/providers/states/k8s.(*K8sStateProvider).Delete:463","diagnostics":{"correlationId":"f84c5a3d-d18b-4805-b783-2d10ad670f45","resourceId":"","traceContext":{"spanId":"76b76c42b6650d99","traceId":"95e10eecb525eb0cef3707320447f98c"}},"instance":"symphony-api-86d764ffc-m6m9l","level":"info","msg":" P (K8s State): delete state test-mqtt-schedule-windows-target in namespace default","scope":"coa.runtime","time":"2025-09-05T09:31:05.14614496Z","type":"log","ver":"0.48.34"}