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

listener.elbv2 incorrect documentation #2350

Open
michaelvl opened this issue Feb 21, 2025 · 2 comments
Open

listener.elbv2 incorrect documentation #2350

michaelvl opened this issue Feb 21, 2025 · 2 comments
Labels
area/documentation Issues or PRs related to documentation and examples service/elbv2 Indicates issues or PRs that are related to elbv2-controller.

Comments

@michaelvl
Copy link

What is the URL of the document?

https://aws-controllers-k8s.github.io/community/reference/elbv2/v1alpha1/listener/)

Which section(s) is the issue in?

The description of this input:

| | defaultActions.[].redirectConfig.statusCode
Optional | string

What needs fixing?

This input does not seem to be optional, I get the following error when not setting it:

  - message: |
      operation error Elastic Load Balancing v2: CreateListener, 1 validation error(s) found.
      - missing required field, CreateListenerInput.DefaultActions[0].RedirectConfig.StatusCode.
    status: "True"
    type: ACK.Recoverable

Additional context

Setting this input to statusCode: HTTP_301 resolved to problem.

@michaelhtm michaelhtm added area/documentation Issues or PRs related to documentation and examples service/elbv2 Indicates issues or PRs that are related to elbv2-controller. labels Feb 21, 2025
@rushmash91
Copy link
Member

Hi @michaelvl, Thank you for reporting this documentation issue regarding the ELBv2 listener.

To help us properly update the documentation, could you please share your YAML configuration (both the one that produced the error and the working version with statusCode: HTTP_301)?

@michaelvl
Copy link
Author

The following Listener definition applied with line 10 commented out produces the error, removing the # from line 10 causes the resource to sync without errors:

apiVersion: elbv2.services.k8s.aws/v1alpha1
kind: Listener
metadata:
  name: ack-test-redir
spec:
  defaultActions:
  - type: redirect
    redirectConfig:
      port: "443"
      #statusCode: HTTP_301
  loadBalancerRef:
    from:
      name: ack-elbv2-test
  port: 80
  protocol: HTTP

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Issues or PRs related to documentation and examples service/elbv2 Indicates issues or PRs that are related to elbv2-controller.
Projects
None yet
Development

No branches or pull requests

3 participants