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: Add ability to use internal network for Keycloak #507

Conversation

AndrienkoAleksandr
Copy link
Contributor

What does this PR do?

Add ability to use internal network for keycloak.

What issues does this PR fix or reference?

eclipse-che/che#17644

Signed-off-by: Oleksandr Andriienko oandriie@redhat.com

@AndrienkoAleksandr AndrienkoAleksandr force-pushed the addAbilityToUseInternalNetworkForKeycloak branch from 76c5be2 to f80872f Compare October 28, 2020 18:49
@AndrienkoAleksandr
Copy link
Contributor Author

@davidfestal Could you review, please?

Copy link
Contributor

@davidfestal davidfestal left a comment

Choose a reason for hiding this comment

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

I have added 3 comments / questions

Additional remark would be that I don't see any additional test case in order to cover this new option / scenario.

deploy/crds/org_v1_che_cr.yaml Outdated Show resolved Hide resolved
pkg/apis/org/v1/che_types.go Outdated Show resolved Hide resolved
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@AndrienkoAleksandr AndrienkoAleksandr force-pushed the addAbilityToUseInternalNetworkForKeycloak branch from b00e45a to bf520ae Compare November 26, 2020 13:17
@AndrienkoAleksandr
Copy link
Contributor Author

Pr is updated(I applied tests) and re-based.

@openshift-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: AndrienkoAleksandr, tolusha
To complete the pull request process, please assign after the PR has been reviewed.
You can assign the PR to them by writing /assign in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@AndrienkoAleksandr
Copy link
Contributor Author

"Testing stable versions updates (pull_request)" - launched locally - passed.

@AndrienkoAleksandr AndrienkoAleksandr merged commit 73d951b into eclipse-che:master Nov 26, 2020
@AndrienkoAleksandr AndrienkoAleksandr deleted the addAbilityToUseInternalNetworkForKeycloak branch November 26, 2020 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants