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

feat(aws): Add scheme to load balancer details #8797

Merged
merged 2 commits into from
Dec 15, 2020

Conversation

alanmquach
Copy link
Contributor

Just exposing a load balancer's scheme in the details pane:

Screen Shot 2020-12-14 at 4 57 03 PM

(options are internal or internet-facing)

@alanmquach alanmquach added the ready to merge Reviewed and ready for merge label Dec 15, 2020
Copy link
Contributor

@christopherthielen christopherthielen left a comment

Choose a reason for hiding this comment

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

Does titus need this too?

@alanmquach
Copy link
Contributor Author

There is no "titus load balancer" and I was never here. 😄

@mergify mergify bot merged commit 4f13518 into spinnaker:master Dec 15, 2020
christopherthielen added a commit that referenced this pull request Dec 16, 2020
1d4320a refactor(REST): Prefer REST('/foo/bar') over REST().path('foo', 'bar')
97bfbf6 refactor(api-deprecation): API is deprecated, switch to REST()
39b08e7 refactor(api-deprecation): Prefer API.path('foo', 'bar') over API.path('foo').path('bar')
587db3a refactor(api-deprecation): Migrate from API.one/all/withParams/getList() to path/query/get()
807de9a test(mock-http-client): Remove unnecessary API.baseUrl prefix in expectGET/etc calls
924e80b test(mock-http-client): Remove no longer needed references to $httpBackend in unit tests after migration to MockHttpClient
e123818 test(mock-http-client): Manually fix tests which didn't pass after auto-migrating using the eslint rule migrate-to-mock-http-client --fix
ef5d8ea test(mock-http-client): Run eslint rule migrate-to-mock-http-client --fix
4f13518 feat(aws): Add scheme to load balancer details (#8797)
969f4fe Avoid raw "$http" usage (#8790)
40ba12c fix(aws): Misc. fixes and new unit tests (#8742)
mergify bot pushed a commit that referenced this pull request Dec 16, 2020
1d4320a refactor(REST): Prefer REST('/foo/bar') over REST().path('foo', 'bar')
97bfbf6 refactor(api-deprecation): API is deprecated, switch to REST()
39b08e7 refactor(api-deprecation): Prefer API.path('foo', 'bar') over API.path('foo').path('bar')
587db3a refactor(api-deprecation): Migrate from API.one/all/withParams/getList() to path/query/get()
807de9a test(mock-http-client): Remove unnecessary API.baseUrl prefix in expectGET/etc calls
924e80b test(mock-http-client): Remove no longer needed references to $httpBackend in unit tests after migration to MockHttpClient
e123818 test(mock-http-client): Manually fix tests which didn't pass after auto-migrating using the eslint rule migrate-to-mock-http-client --fix
ef5d8ea test(mock-http-client): Run eslint rule migrate-to-mock-http-client --fix
4f13518 feat(aws): Add scheme to load balancer details (#8797)
969f4fe Avoid raw "$http" usage (#8790)
40ba12c fix(aws): Misc. fixes and new unit tests (#8742)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge Reviewed and ready for merge target-release/1.25
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants