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

Cherry-pick #16289 to 7.9.x release branch #16290

Merged
merged 1 commit into from
Mar 10, 2020

Conversation

amisevsk
Copy link
Contributor

@amisevsk amisevsk commented Mar 9, 2020

Cherry pick changes from #16289 into release.

@che-bot che-bot added target/branch Indicates that a PR will be merged into a branch other than master. status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. labels Mar 9, 2020
@che-bot
Copy link
Contributor

che-bot commented Mar 9, 2020

❌ E2E Happy path tests failed ❗

See Details

Tested with Eclipse Che Multiuser User on K8S (minikube v1.1.1)

⚠️ https://github.com/orgs/eclipse/teams/eclipse-che-qa please check this report.

ℹ️ Use comment "crw-ci-test" to rerun happy path E2E test.

@nickboldt nickboldt mentioned this pull request Mar 9, 2020
27 tasks
This commit is a cherry-pick of 78ee3bf from master

Fixes an issue where container names for brokers could be longer than 63
characters:

- If a broker image is referenced by tag, trim the digest to 10 chars and
use that instead of the full thing
- If the generated name is longer than 63 chars for whatever reason,
trim it to allow the workspace to start.

Signed-off-by: Angel Misevski <amisevsk@redhat.com>
@amisevsk amisevsk force-pushed the handle-broker-digests-7.9.x branch from ef7d88b to 6bef13f Compare March 10, 2020 19:28
@amisevsk amisevsk merged commit 94ec670 into eclipse-che:7.9.x Mar 10, 2020
@amisevsk amisevsk deleted the handle-broker-digests-7.9.x branch March 10, 2020 19:29
@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 Mar 10, 2020
@che-bot
Copy link
Contributor

che-bot commented Mar 10, 2020

❌ E2E Happy path tests failed ❗

See Details

Tested with Eclipse Che Multiuser User on K8S (minikube v1.1.1)

⚠️ https://github.com/orgs/eclipse/teams/eclipse-che-qa please check this report.

ℹ️ Use comment "crw-ci-test" to rerun happy path E2E test.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

None yet

4 participants