Skip to content

Documented service annotation for AWS ELB SSL policy#6261

Merged
tengqm merged 1 commit intokubernetes:release-1.9from
micahhausler:aws-elb-ssl-policy
Nov 22, 2017
Merged

Documented service annotation for AWS ELB SSL policy#6261
tengqm merged 1 commit intokubernetes:release-1.9from
micahhausler:aws-elb-ssl-policy

Conversation

@micahhausler
Copy link
Member

@micahhausler micahhausler commented Nov 9, 2017

This change is dependent on kubernetes/kubernetes#54507

ping @justinsb


This change is Reviewable

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Nov 9, 2017
@k8s-ci-robot k8s-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Nov 9, 2017
@k8sio-netlify-preview-bot
Copy link
Collaborator

k8sio-netlify-preview-bot commented Nov 9, 2017

Deploy preview ready!

Built with commit c59e38a

https://deploy-preview-6261--kubernetes-io-vnext-staging.netlify.com

@chenopis
Copy link
Contributor

chenopis commented Nov 9, 2017

@micahhausler FYI, the Travis CI error has been fixed. Please rebase or update your branch. Thx!

@chenopis chenopis added this to the 1.9 milestone Nov 9, 2017
aws elb describe-load-balancer-policies --query 'PolicyDescriptions[].PolicyName'
```

Any one of those policies can then be specified using the following annotation:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(trying to be picky ...)

using the following annotation:

using the "service.beta.kubernetes.io/aws-load-balancer-ssl-negotiation-policy" annotation, for example:

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No problem! Thanks

@justinsb
Copy link
Member

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Nov 10, 2017
@steveperry-53
Copy link
Contributor

@micahhausler Can you resolve the merge conflict? Thanks.

@micahhausler
Copy link
Member Author

@steveperry-53 This is ready for merge.

@tengqm tengqm merged commit 6154442 into kubernetes:release-1.9 Nov 22, 2017
@micahhausler micahhausler deleted the aws-elb-ssl-policy branch December 15, 2017 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants