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

Che Operator gets stuck updating endpoint URLs when "legacy" hostname is used #22774

Closed
amisevsk opened this issue Jan 18, 2024 · 1 comment · Fixed by eclipse-che/che-operator#1801
Labels
area/che-operator Issues and PRs related to Eclipse Che Kubernetes Operator kind/bug Outline of a bug - must adhere to the bug report template. severity/P1 Has a major impact to usage or development of the system.

Comments

@amisevsk
Copy link
Contributor

amisevsk commented Jan 18, 2024

Describe the bug

When the Che Operator has to fallback to the legacy format for endpoint hostnames (as added by eclipse-che/che-operator#1702), it can get stuck in a loop updating endpoint hostnames. This occurs because in Golang, map iteration order is random, resulting in a random order number being used for each endpoint (see here.

Che version

7.80@latest

Steps to reproduce

  1. Create a workspace with a long metadata.name, so that the default hostname generated for code-redirect-x endpoints is longer than 63 characters (format is <username>-<workspacename>-<endpointname>)
  2. Ensure workspace is using legacy format for endpoint hostnames (should instead be <workspaceID>-<number>)
  3. Check whether the workspace is able to start.

Expected behavior

We should be using deterministic names for endpoints.

Runtime

Kubernetes (vanilla)

Screenshots

No response

Installation method

other (please specify in additional context)

Environment

other (please specify in additional context)

Eclipse Che Logs

No response

Additional context

Originally reported: #22667
DevWorkspace's workaround for this issue: link

@amisevsk amisevsk added kind/bug Outline of a bug - must adhere to the bug report template. area/che-operator Issues and PRs related to Eclipse Che Kubernetes Operator labels Jan 18, 2024
@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 Jan 18, 2024
@amisevsk
Copy link
Contributor Author

amisevsk commented Jan 19, 2024

This issue was reported as part of #22667 but should have been fixed as of eclipse-che/che-operator#1702 (v7.69) -- cc: @dkwon17 any ideas here? I misunderstood the issue; see updated description.

@amisevsk amisevsk changed the title Che Operator does not ensure ingress hostnames are less than 63 characters Che Operator gets stuck updating endpoint URLs when "legacy" hostname is used Jan 19, 2024
@amisevsk amisevsk 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 Jan 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/che-operator Issues and PRs related to Eclipse Che Kubernetes Operator kind/bug Outline of a bug - must adhere to the bug report template. severity/P1 Has a major impact to usage or development of the system.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants