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

ECS Client has missing SettingName for the dualStackIPv6 option #800

Open
3 tasks done
jmenga-aset opened this issue Mar 13, 2024 · 4 comments
Open
3 tasks done

ECS Client has missing SettingName for the dualStackIPv6 option #800

jmenga-aset opened this issue Mar 13, 2024 · 4 comments
Assignees
Labels
bug Something isn't working p2 service-api This issue pertains to the AWS API

Comments

@jmenga-aset
Copy link

jmenga-aset commented Mar 13, 2024

Checkboxes for prior research

Describe the bug

The ECS Client SettingName enum is missing an option for the dualStackIPv6 option (as indicated here)

SDK version number

@aws-sdk/[email protected]

Which JavaScript Runtime is this issue in?

Node.js

Details of the browser/Node.js/ReactNative version

v20.10.0

Reproduction Steps

import { SettingName } from '@aws-sdk/client-ecs'

console.log(SettingName)

Observed Behavior

The following output is generated:

{
  AWSVPC_TRUNKING: 'awsvpcTrunking',
  CONTAINER_INSIGHTS: 'containerInsights',
  CONTAINER_INSTANCE_LONG_ARN_FORMAT: 'containerInstanceLongArnFormat',
  FARGATE_FIPS_MODE: 'fargateFIPSMode',
  FARGATE_TASK_RETIREMENT_WAIT_PERIOD: 'fargateTaskRetirementWaitPeriod',
  GUARD_DUTY_ACTIVATE: 'guardDutyActivate',
  SERVICE_LONG_ARN_FORMAT: 'serviceLongArnFormat',
  TAG_RESOURCE_AUTHORIZATION: 'tagResourceAuthorization',
  TASK_LONG_ARN_FORMAT: 'taskLongArnFormat'
}

Expected Behavior

There should be a key/value pair for the value dualStackIPv6

e.g.

{
  DUAL_STACK_IPV6: 'dualStackIPv6'
}

Possible Solution

Update the SettingName object to the following:

export const SettingName = {
  AWSVPC_TRUNKING: "awsvpcTrunking",
  CONTAINER_INSIGHTS: "containerInsights",
  CONTAINER_INSTANCE_LONG_ARN_FORMAT: "containerInstanceLongArnFormat",
  DUAL_STACK_IPV6: 'dualStackIPv6'
  FARGATE_FIPS_MODE: "fargateFIPSMode",
  FARGATE_TASK_RETIREMENT_WAIT_PERIOD: "fargateTaskRetirementWaitPeriod",
  GUARD_DUTY_ACTIVATE: "guardDutyActivate",
  SERVICE_LONG_ARN_FORMAT: "serviceLongArnFormat",
  TAG_RESOURCE_AUTHORIZATION: "tagResourceAuthorization",
  TASK_LONG_ARN_FORMAT: "taskLongArnFormat",
} as const;

Additional Information/Context

No response

@jmenga-aset jmenga-aset added bug Something isn't working needs-triage labels Mar 13, 2024
@RanVaknin
Copy link

Hi @jmenga-aset ,

Thanks for reaching out. Its not clear if this was never modeled, or was removed. I'm curious, if you had to supply the string value directly instead of using the enum, does dualStackIPv6 actually work?

The reason why I'm asking is sometimes service teams remove enum values (in favor of introducing new parameters or deprecating old ones) and forget to remove the corresponding documentation.

Thanks,
Ran~

@RanVaknin RanVaknin self-assigned this Mar 15, 2024
@RanVaknin RanVaknin added response-requested This issue requires a response to continue p2 service-api This issue pertains to the AWS API and removed needs-triage labels Mar 15, 2024
Copy link

This issue has not received a response in 1 week. If you still think there is a problem, please leave a comment to avoid the issue from automatically closing.

@github-actions github-actions bot added the closing-soon This issue will be closed soon label Mar 26, 2024
@mixja
Copy link

mixja commented Mar 26, 2024

Yes dualStackIPv6 is a documented setting and works - see docs

@github-actions github-actions bot removed closing-soon This issue will be closed soon response-requested This issue requires a response to continue labels Mar 27, 2024
@RanVaknin
Copy link

Hi there,

I have created an internal ticket (P146481495) with the EC2 service team for them to update their model with the missing enum.
Since the AWS SDKs are code-generated from those API models, the SDK team cannot add those directly. Instead, the service team itself would be the one needing to make the change on the backend.

Moving to cross SDK repo for tracking.

Thank,
Ran~

@RanVaknin RanVaknin transferred this issue from aws/aws-sdk-js-v3 Aug 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working p2 service-api This issue pertains to the AWS API
Projects
None yet
Development

No branches or pull requests

3 participants