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

botocore/botocore/data/ssm-incidents/2018-05-10 /waiters-2.json = Formatted Differently than other waiters-2.json Files #3157

Closed
colinbjohnson opened this issue Apr 17, 2024 · 5 comments
Assignees
Labels
bug This issue is a confirmed bug. p2 This is a standard priority issue waiter

Comments

@colinbjohnson
Copy link

Describe the bug

The problem is that the botocore/botocore/data/ssm-incidents/2018-05-10 /waiters-2.json file is formatted differently than all other waiters-2.json json files.

Expected Behavior

I expect all .json files in a given repository to be formatted similarly.

Current Behavior

The particular json file is not formatted in the same manner as other .json files.

Reproduction Steps

Possible Solution

I will provide a PR.

Additional Information/Context

I don't know if there is a linting tool for json files but it would be nice if all of these files conformed to a similar standard.

SDK version used

1.34.85

Environment details (OS name and version, etc.)

macOS, 14.2.1 (23C71)

@colinbjohnson colinbjohnson added bug This issue is a confirmed bug. needs-triage This issue or PR still needs to be triaged. labels Apr 17, 2024
@tim-finnigan tim-finnigan self-assigned this Apr 19, 2024
@tim-finnigan tim-finnigan added investigating This issue is being investigated and/or work is in progress to resolve the issue. and removed needs-triage This issue or PR still needs to be triaged. labels Apr 19, 2024
@tim-finnigan
Copy link
Contributor

Hi @colinbjohnson thanks for reaching out. The latest EC2 waiter models are in this file. Can you specify the differences you're seeing? Looking at these two files, they seem to follow the same basic structure (apart from the order of keys, and the ssm-incidents file containing a description):

image

Have you observed any inconsistencies in functionality between the waiters, or can you provide more context on your use case?

Waiters are also now maintained by service teams, so if any bugs or feature requests involving waiters would likely need to get forwarded to the corresponding service team. But minor formatting changes may not get prioritized unless there is some use case in which you are blocked.

@tim-finnigan tim-finnigan added response-requested Waiting on additional info and feedback. p2 This is a standard priority issue waiter and removed investigating This issue is being investigated and/or work is in progress to resolve the issue. labels Apr 19, 2024
@colinbjohnson
Copy link
Author

Two things to look at (which a linter might catch <- I just don't know if there are linters available that can catch these things):

  1. The acceptor configuration in most waiters files utilize list items that end on one line and start on the next line:

Typical acceptor block:

},
{

as opposed to a single-line format used in botocore/botocore/data/ssm-incidents/2018-05-10 /waiters-2.json:

}, {
  1. The number of spaces after a "key" in most waiters files is typically 0 spaces whereas in botocore/botocore/data/ssm-incidents/2018-05-10 /waiters-2.json it is 1 space.

Typical waiters file:

"operation": "GetReplicaSet"

Versus botocore/botocore/data/ssm-incidents/2018-05-10 /waiters-2.json:

"operation" : "GetReplicaSet"

@colinbjohnson
Copy link
Author

I completely understand how this wouldn't be a priority but I'd also be open to submitting a (separate) PR that includes JSON linting. My own hope would be that something (admittedly small) like JSON linting would help establish the importance of consistency across a codebase and across AWS teams.

@github-actions github-actions bot removed the response-requested Waiting on additional info and feedback. label Apr 20, 2024
@tim-finnigan
Copy link
Contributor

Thanks for following up and expanding on the issue. I brought this up with the team for discussion and the takeaway was that this would need to be addressed by the service team. The reason is because waiter models are shared across SDKs, and any changes we would make to the models would be overridden by changes made by the service teams.

That being said, I think we should strive for consistency among these models. If you have a support plan then I recommend reaching out through AWS Support requesting that the SSM team update the format of their waiter model. Alternatively, you can create an issue in our cross-SDK repository and someone can reach out on your behalf.

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a confirmed bug. p2 This is a standard priority issue waiter
Projects
None yet
Development

No branches or pull requests

2 participants