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.

Sign up for GitHub

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

Jump to bottom

[che-auth] chectl should not wait for keycloak in nativeUserMode #20009

Closed
Tracked by #19182
sparkoo opened this issue Jun 22, 2021 · 1 comment
Closed
Tracked by #19182

[che-auth] chectl should not wait for keycloak in nativeUserMode #20009

sparkoo opened this issue Jun 22, 2021 · 1 comment
Assignees
@sparkoo
Labels
area/chectl Issues related to chectl, the CLI of Che kind/task Internal things, technical debt, and to-do tasks to be performed. severity/P1 Has a major impact to usage or development of the system.

Comments

@sparkoo
Copy link
Member

sparkoo commented Jun 22, 2021

Is your task related to a problem? Please describe.

When in nativeUserMode enabled, keycloak is not deployed. However, chectl is waiting for it and fails.

apiVersion: org.eclipse.che/v1
kind: CheCluster
metadata:
  name: eclipse-che
spec:
  auth:
    nativeUserMode: true

❗ Note to assignee: In time I'm creating this issue, the che-operator PR to deploy in nativeUserMode is open. If it is still the case, you have to use custom che-operator image.

Describe the solution you'd like

Chectl should not wait for keycloak deployment when nativeUserMode is enabled.

Describe alternatives you've considered

Additional context

[che-auth] - make it possible to deploy Che with new auth #19705
che-operator PR - native user authentication #861 (eclipse-che/che-operator#861)
@sparkoo sparkoo added kind/task Internal things, technical debt, and to-do tasks to be performed. area/chectl Issues related to chectl, the CLI of Che labels Jun 22, 2021
@che-bot che-bot added the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Jun 22, 2021
@sparkoo sparkoo mentioned this issue Jun 22, 2021
Closed
23 tasks
@metlos metlos added severity/P1 Has a major impact to usage or development of the system. and removed status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. labels Jun 22, 2021
@sparkoo sparkoo mentioned this issue Jul 2, 2021
Closed
5 tasks
@sparkoo sparkoo self-assigned this Jul 21, 2021
@sparkoo sparkoo mentioned this issue Jul 21, 2021
Merged
9 tasks
@sparkoo
Copy link
Member Author

sparkoo commented Jul 21, 2021

fixed by che-incubator/chectl#1448

@sparkoo sparkoo closed this as completed Jul 21, 2021
@sparkoo sparkoo mentioned this issue Jul 21, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sparkoo sparkoo

Labels
area/chectl Issues related to chectl, the CLI of Che kind/task Internal things, technical debt, and to-do tasks to be performed. severity/P1 Has a major impact to usage or development of the system.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@metlos @sparkoo @tolusha @che-bot