Skip to content

Commit e992f57

Browse files
authored
update: Update the Resource already exists condition reason (#177)
Description of changes: Currently the condition reason for the `Resource Already Exists` condition message instructs users to adopt the existing resource using the `AdoptResource` CRD, which we no longer implement. This change updates the message to instruct users to instead use the adoption by annotation feature to import an existing resource. By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
1 parent 910ed2a commit e992f57

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

pkg/condition/condition.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,9 @@ var (
2828
NotManagedMessage = "Resource already exists"
2929
NotManagedReason = "This resource already exists but is not managed by ACK. " +
3030
"To bring the resource under ACK management, you should explicitly adopt " +
31-
"the resource by creating a services.k8s.aws/AdoptedResource"
31+
"the resource by enabling the ResourceAdoption feature gate and populating " +
32+
"the `services.k8s.aws/adoption-policy` and `services.k8s.aws/adoption-fields` " +
33+
"annotations."
3234
UnknownSyncedMessage = "Unable to determine if desired resource state matches latest observed state"
3335
NotSyncedMessage = "Resource not synced"
3436
SyncedMessage = "Resource synced successfully"

0 commit comments

Comments
 (0)