Skip to content
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

rum - AppMonitor returns name instead of id #2187

Closed
piwysocki opened this issue Oct 21, 2022 · 2 comments
Closed

rum - AppMonitor returns name instead of id #2187

piwysocki opened this issue Oct 21, 2022 · 2 comments
Assignees
Labels
kind/bug Some behavior is incorrect or out of spec resolution/fixed This issue was fixed

Comments

@piwysocki
Copy link

What happened?

I'm creating new rum AppMonitor and trying to obtain an id:

const appMonitor = new aws.rum.AppMonitor("name_of_app_monitor", { domain: "*.example.com" });
console.log(appMonitor.id); //this logs name instead of an id

We have to put AppMonitor ID into RUM code snippet:

https://github.com/aws-observability/aws-rum-web/blob/main/docs/cdn_installation.md#arguments

id

Steps to reproduce

  1. Create AppMonitor using Pulumi AWS Classic package
    const appMonitor = new aws.rum.AppMonitor("name_of_app_monitor", { domain: "*.example.com" });
  2. Log appMonitor.id
    console.log(appMonitor.id); //this logs name instead of an id

Expected Behavior

Expected: appMonitor.id should be

The provider-assigned unique ID for this managed resource.

Actual Behavior

appMonitor.id is name

Output of pulumi about

No response

Additional context

screenshot from AWS

Documentation

Contributing

Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

@piwysocki piwysocki added kind/bug Some behavior is incorrect or out of spec needs-triage Needs attention from the triage team labels Oct 21, 2022
@danielrbradley
Copy link
Member

Hi @piwysocki, this appears to be exactly the same core issue as the one you filed in pulumi/pulumi-aws-native#333.

This provider is generated based on the terraform AWS definitions and would therefore need to be fixed there first. It appears that this is the relevant upstream issue: hashicorp/terraform-provider-aws#26430

@danielrbradley danielrbradley added awaiting-upstream The issue cannot be resolved without action in another repository (may be owned by Pulumi). and removed needs-triage Needs attention from the triage team labels Oct 24, 2022
@mikhailshilkov mikhailshilkov added resolution/fixed This issue was fixed and removed awaiting-upstream The issue cannot be resolved without action in another repository (may be owned by Pulumi). labels Sep 19, 2024
@mikhailshilkov mikhailshilkov self-assigned this Sep 19, 2024
@mikhailshilkov
Copy link
Member

appMonitorId has been exported upstream in hashicorp/terraform-provider-aws#26994

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Some behavior is incorrect or out of spec resolution/fixed This issue was fixed
Projects
None yet
Development

No branches or pull requests

3 participants