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

Fix failing Che when used external OIDC provider. Port to 7.23.x #18631

Merged

Conversation

AndrienkoAleksandr
Copy link
Contributor

What does this PR do?

Fix failing Che when used external OIDC provider. Refactor code related to internal network. Add more tests and java docs.

What issues does this PR fix or reference?

#18553

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

PR Checklist

As the author of this Pull Request I made sure that:

Reviewers

Reviewers, please comment how you tested the PR when approving it.

Fix failing Che when used external OIDC provider. Refactor code related to internal network. Add more tests and java docs.

Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@che-bot che-bot added target/branch Indicates that a PR will be merged into a branch other than master. kind/task Internal things, technical debt, and to-do tasks to be performed. status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. labels Dec 16, 2020
@tolusha tolusha mentioned this pull request Dec 16, 2020
56 tasks
@ibuziuk ibuziuk mentioned this pull request Dec 16, 2020
13 tasks
@AndrienkoAleksandr
Copy link
Contributor Author

Retested manually one more time for 7.23.x with Openshift 4.

@AndrienkoAleksandr AndrienkoAleksandr merged commit 2d8d76b into 7.23.x Dec 16, 2020
@AndrienkoAleksandr AndrienkoAleksandr deleted the fixFailingCheWithOIDCProviderPortTo7.23.x branch December 16, 2020 16:41
@che-bot che-bot removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Dec 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/task Internal things, technical debt, and to-do tasks to be performed. target/branch Indicates that a PR will be merged into a branch other than master.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants