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

Updating che on prod to the latest 6.18.1 upstream version #1221

Merged
merged 3 commits into from
Feb 14, 2019

Conversation

ibuziuk
Copy link
Collaborator

@ibuziuk ibuziuk commented Feb 13, 2019

No description provided.

@ibuziuk
Copy link
Collaborator Author

ibuziuk commented Feb 13, 2019

cc @rhopp @ScrewTSW

@ScrewTSW
Copy link

project pull issue on starter-us-east-2a cluster: redhat-developer/rh-che#1248

@ibuziuk
Copy link
Collaborator Author

ibuziuk commented Feb 13, 2019

@ScrewTSW isn't it known openshiftio/openshift.io#4695 ?

@ScrewTSW
Copy link

The regression seems to be gone, the workpsaces are being stopped successfully after the timeout

@ibuziuk
Copy link
Collaborator Author

ibuziuk commented Feb 13, 2019

@ScrewTSW @rhopp it would be great to get info about things that are not covered by e2e / PR check test:

  • idling
  • che 7

@rhopp
Copy link

rhopp commented Feb 13, 2019

@ibuziuk
afaik the e2e tests just check workspace (che 6 based) creation. Nothing more...

@ScrewTSW
Copy link

@ibuziuk
afaik the e2e tests just check workspace (che 6 based) creation. Nothing more...

👍

@ibuziuk
Copy link
Collaborator Author

ibuziuk commented Feb 13, 2019

@rhopp @ScrewTSW that is why I wrote about idling & che 7 that are NOT covered by any of existing tests and expect them to be tested manually before +1 / -1 of the PR

@ScrewTSW
Copy link

@ibuziuk The workspaces are behaving normally on other clusters (excluding starter-us-east-2a where the project import issue is present). The workspace stopping is functional. We agreed with @rhopp to 👍 the PR as the project import issue is a platform issue.

@ibuziuk ibuziuk merged commit fd1102d into master Feb 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants