Skip to content
This repository has been archived by the owner on Nov 5, 2024. It is now read-only.

Add --engine-arn to update-canary #613

Closed
1 of 2 tasks
socketbox opened this issue Oct 4, 2023 · 3 comments
Closed
1 of 2 tasks

Add --engine-arn to update-canary #613

socketbox opened this issue Oct 4, 2023 · 3 comments
Assignees
Labels
feature-request New feature or request service-api This issue pertains to the AWS API synthetics

Comments

@socketbox
Copy link

Describe the feature

I'd like to see update-canary be capable of taking an --engine--arn argument that does what the Java and Go SDK are capable of with setEngineArn and SetEngineArn.

Use Case

When instantiating canaries outside of the AWS Console, either via IaC-related approaches or some other means, it's possible for the canary to become out-of-sync with its attendant Lambda function's latest version.

If there was a different use case that was behind the creation of the setEngineArn/SetEngineArn methods of the SDKs, then I'd think that same use case would apply here as well.

Proposed Solution

As mentioned, I think this could be as simple as adding an --engine-arn option to aws synthetics update-canary.

Other Information

No response

Acknowledgements

  • I may be able to implement this feature request
  • This feature might incur a breaking change

CLI version used

aws-cli/2.11.16 Python/3.11.3 Linux/6.2.0-33-generic

Environment details (OS name and version, etc.)

Linux/6.2.0-33-generic x86-64 Linux Mint 21.2

@socketbox socketbox added feature-request New feature or request needs-triage labels Oct 4, 2023
@tim-finnigan tim-finnigan self-assigned this Oct 5, 2023
@tim-finnigan
Copy link
Contributor

tim-finnigan commented Oct 5, 2023

Hi @socketbox thanks for the feature request. The AWS CLI update-canary command corresponds to the UpdateCanary API. That API is maintained Amazon CloudWatch Synthetics service team, so we would need to forward feature requests like this to that team. They may have a reason for only allowing the setting of EngineArn via CreateCanary but we can try to find out more info. I'm going to transfer this to our cross-SDK repository, as service APIs are used across AWS SDKs and should have consistent behavior. (ref: P101834216)

@tim-finnigan tim-finnigan added service-api This issue pertains to the AWS API synthetics and removed needs-triage labels Oct 5, 2023
@tim-finnigan tim-finnigan transferred this issue from aws/aws-cli Oct 5, 2023
@tim-finnigan
Copy link
Contributor

We heard back from a member of the service team and they have added this feature request to their backlog. We cannot guarantee if or when the service team would implement this feature, but feel free to check back in the future for updates. We recommend reaching out through AWS Support if you have a support plan.

Copy link

This issue is now closed.

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request New feature or request service-api This issue pertains to the AWS API synthetics
Projects
None yet
Development

No branches or pull requests

2 participants