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

Update procedure for Che v7.41 and earlier #2269

Merged
merged 9 commits into from
Apr 11, 2022

Conversation

l0rd
Copy link
Contributor

@l0rd l0rd commented Mar 31, 2022

What does this pull request change

Replaces #2159

What issues does this pull request fix or reference

eclipse-che/che/issues/20614

Specify the version of the product this pull request applies to

Pull request checklist

The author and the reviewers validate the content of this pull request with the following checklist, in addition to the automated tests.

  • Any procedure:
    • Successfully tested.
  • Any page or link rename:
  • Builds on Eclipse Che hosted by Red Hat.
  • the Validate language on files added or modified step reports no vale warnings.

image

@che-bot che-bot added this to Triage in Reviews Mar 31, 2022
@github-actions
Copy link

Click here to review and test in web IDE: Contribute

Signed-off-by: Mario Loriedo <mario.loriedo@gmail.com>
antora.yml Outdated Show resolved Hide resolved
antora.yml Outdated Show resolved Hide resolved
l0rd and others added 3 commits March 31, 2022 12:45
Co-authored-by: Fabrice Flore-Thébault <ffloreth@redhat.com>
Co-authored-by: Fabrice Flore-Thébault <ffloreth@redhat.com>
Co-authored-by: Fabrice Flore-Thébault <ffloreth@redhat.com>
Reviews automation moved this from Triage to Approved Mar 31, 2022
@themr0c
Copy link
Contributor

themr0c commented Mar 31, 2022

It does replace #2159, correct?

@l0rd
Copy link
Contributor Author

l0rd commented Mar 31, 2022

It does replace #2159, correct?

Correct. Once we merge this one I will close #2159. I am still testing the migration scripts. I will comment once I am done.

Signed-off-by: Mario Loriedo <mario.loriedo@gmail.com>
@l0rd
Copy link
Contributor Author

l0rd commented Mar 31, 2022

I have updated the scripts but for some reason Che is unstable after the migration:

  • The devworkspace controller fails to start with the following message (but that can be fixed removing the devworkspace-webhook-server deployment, mutating, and validating webhooks as described in the doc):
msg":"failed to setup webhooks","error":"conflicting webhook definition found on cluster, webhook mutate.devworkspace-controller.svc clientConfig points at namespace openshift-operators"
  • Posgres pod fails to restart too with the following message:
waiting for server to start....2022-03-31 23:42:43.576 UTC [27] LOG:  redirecting log output to logging collector process
2022-03-31 23:42:43.576 UTC [27] HINT:  Future log output will appear in directory "log".
. stopped waiting
pg_ctl: could not start server
Examine the log output.

That may be related to the fact that I have tested the migration with some running workspaces. I will continue the investigation on Monday.

@themr0c themr0c self-assigned this Apr 1, 2022
@che-bot che-bot moved this from Approved to In progress in Reviews Apr 1, 2022
@themr0c
Copy link
Contributor

themr0c commented Apr 1, 2022

OK. Merge when it's tested successfully.

@themr0c themr0c added this to the 7.44 milestone Apr 1, 2022
@ibuziuk ibuziuk mentioned this pull request Apr 5, 2022
45 tasks
l0rd added 3 commits April 9, 2022 22:50
Signed-off-by: Mario Loriedo <mario.loriedo@gmail.com>
Signed-off-by: Mario Loriedo <mario.loriedo@gmail.com>
Signed-off-by: Mario Loriedo <mario.loriedo@gmail.com>
@themr0c themr0c merged commit 59f7e90 into eclipse-che:master Apr 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Reviews
  
Done
3 participants