community icon indicating copy to clipboard operation
community copied to clipboard

Implement `update` functionality for `VPCEndpoint` resource

Open brycahta opened this issue 2 years ago • 17 comments

Is your feature request related to a problem? ec2-controller supports Create/Delete functionality for VPCEndpoint resource, but not Update.

Describe the solution you'd like I should be able to modify/update an ACK-managed VPCEndpoint resource using its controller

Describe alternatives you've considered Updating resource myself using aws-sdk, cli, or console

brycahta avatar Mar 02 '22 20:03 brycahta

ModifyVpcEndpoint is an ignored operation in generator.yaml.

Note, resource will not be in ACK.ResourceSynced condition if users do not provide a policyDocument in the Spec because AWS will set a default policyDocument if not provided. The resulting diff will trigger update (not implemented)

brycahta avatar Mar 23 '22 20:03 brycahta

@brycahta sounds like policyDocument should be lateInitiailized?

jaypipes avatar Mar 24 '22 12:03 jaypipes

Issues go stale after 90d of inactivity. Mark the issue as fresh with /remove-lifecycle stale. Stale issues rot after an additional 30d of inactivity and eventually close. If this issue is safe to close now please do so with /close. Provide feedback via https://github.com/aws-controllers-k8s/community. /lifecycle stale

ack-bot avatar Jun 22 '22 17:06 ack-bot

Stale issues rot after 30d of inactivity. Mark the issue as fresh with /remove-lifecycle rotten. Rotten issues close after an additional 30d of inactivity. If this issue is safe to close now please do so with /close. Provide feedback via https://github.com/aws-controllers-k8s/community. /lifecycle rotten

ack-bot avatar Jul 22 '22 17:07 ack-bot

Rotten issues close after 30d of inactivity. Reopen the issue with /reopen. Provide feedback via https://github.com/aws-controllers-k8s/community. /close

ack-bot avatar Aug 21 '22 17:08 ack-bot

@ack-bot: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity. Reopen the issue with /reopen. Provide feedback via https://github.com/aws-controllers-k8s/community. /close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

ack-bot avatar Aug 21 '22 17:08 ack-bot

/reopen

a-hilaly avatar Aug 21 '22 18:08 a-hilaly

@A-Hilaly: Reopened this issue.

In response to this:

/reopen

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

ack-bot avatar Aug 21 '22 18:08 ack-bot

/reopen

image

brycahta avatar Sep 06 '22 18:09 brycahta

Issues go stale after 90d of inactivity. Mark the issue as fresh with /remove-lifecycle stale. Stale issues rot after an additional 30d of inactivity and eventually close. If this issue is safe to close now please do so with /close. Provide feedback via https://github.com/aws-controllers-k8s/community. /lifecycle stale

eks-bot avatar Dec 05 '22 22:12 eks-bot

Stale issues rot after 30d of inactivity. Mark the issue as fresh with /remove-lifecycle rotten. Rotten issues close after an additional 30d of inactivity. If this issue is safe to close now please do so with /close. Provide feedback via https://github.com/aws-controllers-k8s/community. /lifecycle rotten

ack-bot avatar Jan 12 '23 23:01 ack-bot

Rotten issues close after 30d of inactivity. Reopen the issue with /reopen. Provide feedback via https://github.com/aws-controllers-k8s/community. /close

ack-bot avatar Feb 12 '23 04:02 ack-bot

@ack-bot: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity. Reopen the issue with /reopen. Provide feedback via https://github.com/aws-controllers-k8s/community. /close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

ack-prow[bot] avatar Feb 12 '23 04:02 ack-prow[bot]

Issues go stale after 90d of inactivity. Mark the issue as fresh with /remove-lifecycle stale. Stale issues rot after an additional 30d of inactivity and eventually close. If this issue is safe to close now please do so with /close. Provide feedback via https://github.com/aws-controllers-k8s/community. /lifecycle stale

ack-bot avatar May 14 '23 15:05 ack-bot

/lifecycle frozen

RedbackThomson avatar May 16 '23 17:05 RedbackThomson

Do we know what the comment in the unimplemented sdkUpdate(...) fn refers to?

	// TODO(jaypipes): Figure this out...

Is there something tricky here to be figured out? If so, what is the context?

cannonpalms avatar May 03 '24 15:05 cannonpalms