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

Surface ALB information into rollout status #1241

Closed
jessesuen opened this issue Jun 2, 2021 · 4 comments
Closed

Surface ALB information into rollout status #1241

jessesuen opened this issue Jun 2, 2021 · 4 comments
Labels
enhancement New feature or request
Milestone

Comments

@jessesuen
Copy link
Member

jessesuen commented Jun 2, 2021

Summary

The recent ALB weight verification feature allowed Argo Rollouts to discover information about the underlying AWS LoadBalancer and TargetGroups, including ARN, IDs, tags, etc... Currently this is only used for verifying the weights of the TargetGroup, but it is not used anywhere else. For debugging and analysis purposes, it would be useful to surface the ALB and target group information in the rollout status. Like so:

status:
  alb:
    loadBalancer:
      name: k8s-devdeplo-testdele-b9061572ad
      arn: arn:aws:elasticloadbalancing:us-west-2:1234567891011:loadbalancer/app/k8s-devdeplo-testdele-abcdef1234/3a47c96b75571d0e
    canaryTargetGroup:
      name: k8s-devdeplo-testdele-ab72c5b132
      arn: arn:aws:elasticloadbalancing:us-west-2:1234567891011:targetgroup/k8s-devdeplo-testdele-ab72c5b132/df2957821c1214fc
    stableTargetGroup:
      name: k8s-devdeplo-testdele-8832d6519a
      arn: arn:aws:elasticloadbalancing:us-west-2:1234567891011:targetgroup/k8s-devdeplo-testdele-8832d6519a/df2957821c1214fc 

Use Cases

When would you use this?

In order to query AWS CloudWatch for metrics about a canary service, we need to know the canary TargetGroup. However, the target group name is not easily known to the user. By filling in the target group information under the rollout status, it would allow users to perform queries about metrics specific to the canary target group by passing the target group name to the cloudwatch query


Message from the maintainers:

Impacted by this bug? Give it a 👍. We prioritize the issues with the most 👍.

@jessesuen jessesuen added the enhancement New feature or request label Jun 2, 2021
@jessesuen jessesuen added this to the v1.1 milestone Jun 2, 2021
@jessesuen jessesuen removed this from the v1.1 milestone Jul 1, 2021
@harikrongali harikrongali added this to the v1.2 milestone Nov 2, 2021
@noam-codefresh
Copy link
Contributor

If I understand correctly, the alb traffic routing can be applied to BlueGreen as well. Am I correct?
If so - do you want the status.alb to include CanaryTargetGroup and StableTargetGroup? Or do you expect it to use ActiveTargetGroup and PreviewTargetGroup?

@noam-codefresh
Copy link
Contributor

I just created a PR - #1625

alexmt pushed a commit that referenced this issue Nov 16, 2021
feat(alb): Surface ALB information into rollout status (#1241) (#1625)
Signed-off-by: Noam Gal <[email protected]>
@alexmt
Copy link
Contributor

alexmt commented Nov 16, 2021

Implemented by #1625 . Thank you @noam-codefresh !

@danil-smirnov
Copy link
Contributor

Hey, it seems this works only if --aws-verify-target-group parameter is passed to the controller. I'd expect this to work always if trafficRouting.alb is used in the rollout

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

5 participants