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

aws iotsitewise describe-portal --portal-id does not include edgeConfig/edgeAccess values within the AWS response. #9182

Closed
1 task
bobneuman opened this issue Jan 2, 2025 · 6 comments
Assignees
Labels
bug This issue is a bug. iotsitewise p2 This is a standard priority issue service-api This issue is due to a problem in a service API, not the SDK implementation.

Comments

@bobneuman
Copy link

Describe the bug

aws iotsitewise describe-portal --portal-id [idnumber] does not display the information related to the edgeConfig enablement that is returned within the response from AWS.
I you run the command with --debug the MainThread - botocore.parsers - DEBUG - Response body: {} includes the "edgeConfig": { "edgeAccess": "ENABLED" }, but the cli output does not.

Regression Issue

  • Select this option if this issue appears to be a regression.

Expected Behavior

I'd expect the cli to display the information in the MainThread - botocore.parsers - DEBUG - Response body: {}

Current Behavior

The cli of aws iotsitewise describe-portal --portal-id _____ fails to include the "edgeConfig": { "edgeAccess": "ENABLED" } or "edgeConfig": null

Reproduction Steps

Create a sitewise edge portal and use the id to query aws iotsitewise describe-portal --portal-id _____ --debug. A review of the MainThread - botocore.parsers - DEBUG - Response body should have "edgeConfig": null. Then using the console navigate to the portal and enable the edge (see. ___ ). Wait a few minutes and rerun the command aws iotsitewise describe-portal --portal-id _____ --debug the debug content will include "edgeConfig": { "edgeAccess": "ENABLED"} , but the cli output will not.

Possible Solution

No response

Additional Information/Context

Update the Create and Update Portal functions of the cli to allow setting the edgeAccess to ENABLED. Cloudtrail captures the following when the setting is altered in the console.

  { .....
    "eventName": "UpdatePortal",
    ....
    "requestParameters": {
        "portalName": "My-Portal",
        "roleArn": "arn:aws:iam::123456789011:role/AWSIoTSiteWiseMonitorServiceRole",
        "clientToken": "aaaaaaaa-bbbb-cccc-ddddddddddd",
        "portalId": "aaaaaaaa-bbbb-cccc-ddddddddddd",
        "portalContactEmail": "[email protected]",
        "edgeConfig": {
            "edgeAccess": "ENABLED"
        },
        "portalDescription": "My-Portal description"
    },

CLI version used

aws-cli/2.22.26 Python/3.12.6

Environment details (OS name and version, etc.)

Linux/6.1.0-27-amd64 exe/x86_64.debian.12

@bobneuman bobneuman added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels Jan 2, 2025
@RyanFitzSimmonsAK RyanFitzSimmonsAK self-assigned this Jan 2, 2025
@RyanFitzSimmonsAK RyanFitzSimmonsAK added investigating This issue is being investigated and/or work is in progress to resolve the issue. p2 This is a standard priority issue iotsitewise service-api This issue is due to a problem in a service API, not the SDK implementation. and removed needs-triage This issue or PR still needs to be triaged. investigating This issue is being investigated and/or work is in progress to resolve the issue. labels Jan 2, 2025
@RyanFitzSimmonsAK
Copy link
Contributor

RyanFitzSimmonsAK commented Jan 6, 2025

Hi @bobneuman, thanks for reaching out. I was able to reproduce this behavior; the unparsed response contains several unmodeled fields, including edgeConfig": { "edgeAccess": "ENABLED" }. Because this operation doesn't document that it returns this value, technically the expected behavior would be for it to be absent. However, given that it's already returning it, the ideal solution would probably be to change the model file so that the parsed response includes it. I've reached out to the service team, and will let you know when I have any updates. In the meantime, you'd need to use the unparsed response if you want to get this value using describe-portal.

Ticket # for internal use : P186030809

@bobneuman
Copy link
Author

RyanFitzSimmonsAK the requirements should align to the published service. The ability to set the edgeConfig": { "edgeAccess": "ENABLED" } for SiteWise via the console has been present for months. The CLI and SDKs should implement those items. Backloged somewhere?

@nateprewitt
Copy link
Member

Hi @bobneuman,

To provide a little more clarity, you can check the official API Reference for AWS IoT Sitewise. They don't currently publish edgeConfig as being part of their public API. This is why all AWS SDKs (including the AWS CLI) currently ignore this value in the response. This may be an oversight from the Sitewise team though.

Once they confirm their intent and officially flag this as part of their public API, it should be available in all of our public tooling. @RyanFitzSimmonsAK will follow up once we get clearer guidance from the service team.

@RyanFitzSimmonsAK
Copy link
Contributor

Hey, thanks for your patience, the service team got back to me. This value is not intended to be part of the response, and won't be added to the public model. Because this behavior is subject to change, customers should not rely on it. Unfortunately, there isn't an intended way to interact with this setting programatically. You'll have to use the console for a supported way to use this feature.

@RyanFitzSimmonsAK RyanFitzSimmonsAK closed this as not planned Won't fix, can't repro, duplicate, stale Jan 10, 2025
Copy link

This issue is now closed. Comments on closed issues are hard for our team to see.
If you need more assistance, please open a new issue that references this one.

@bobneuman
Copy link
Author

@RyanFitzSimmonsAK interesting response from the service team. Although the API docs are not updated with the 'edgeConfig' it does set the value via a direct API call by a lambda. I'll use that until something is published or they deprecate the capability.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug. iotsitewise p2 This is a standard priority issue service-api This issue is due to a problem in a service API, not the SDK implementation.
Projects
None yet
Development

No branches or pull requests

3 participants