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

build: Transition from Java EE to Jakarta EE #80

Merged
merged 19 commits into from
Sep 13, 2021

Conversation

skabashnyuk
Copy link
Contributor

@skabashnyuk skabashnyuk commented Aug 10, 2021

What does this PR do?

The transition from Java EE to Jakarta EE
PR can be reviewed commit by commit.
TODO
Everrest CQ - https://dev.eclipse.org/ipzilla/show_bug.cgi?id=23669

Screenshot/screencast of this PR

Знімок екрана 2021-08-10 о 16 28 12

Знімок екрана 2021-08-10 о 16 27 36

What issues does this PR fix or reference?

eclipse-che/che#20247

How to test this PR?

  • build or deploy che-server's image quay.io/skabashn/che-server:jakarta_2021_09_03_14_12_20
  • login as "kubeadmin" on OpenShift
  • Start new workspace.
  • Workspace should start successfully.

PR Checklist

As the author of this Pull Request I made sure that:

Reviewers

Reviewers, please comment how you tested the PR when approving it.

@skabashnyuk skabashnyuk changed the title Transition from Java EE to Jakarta EE build: Transition from Java EE to Jakarta EE Aug 16, 2021
@dmytro-ndp
Copy link
Contributor

[crw-ci-test --rebuild]

1 similar comment
@dmytro-ndp
Copy link
Contributor

[crw-ci-test --rebuild]

@eclipse-che eclipse-che deleted a comment from che-bot Aug 20, 2021
@eclipse-che eclipse-che deleted a comment from che-bot Aug 20, 2021
@eclipse-che eclipse-che deleted a comment from che-bot Aug 20, 2021
@eclipse-che eclipse-che deleted a comment from che-bot Aug 20, 2021
@eclipse-che eclipse-che deleted a comment from che-bot Aug 20, 2021
@eclipse-che eclipse-che deleted a comment from che-bot Aug 20, 2021
@eclipse-che eclipse-che deleted a comment from che-bot Aug 20, 2021
@eclipse-che eclipse-che deleted a comment from che-bot Aug 20, 2021
@eclipse-che eclipse-che deleted a comment from che-bot Aug 20, 2021
@che-bot
Copy link
Contributor

che-bot commented Aug 20, 2021

✅ E2E Happy path tests succeed 🎉

See Details

Test product:

  • Use comment "[crw-ci-test]" to rerun happy path E2E test.
  • Use comment "[crw-ci-test --rebuild]" to re-build the images and rerun happy path E2E test.

Eclipse Che QE channel: https://mattermost.eclipse.org/eclipse/channels/eclipse-che-qe

@skabashnyuk skabashnyuk marked this pull request as ready for review August 23, 2021 07:02
@sparkoo
Copy link
Member

sparkoo commented Aug 23, 2021

I can't see the error in che-server:next

@che-bot
Copy link
Contributor

che-bot commented Aug 25, 2021

✅ E2E Happy path tests succeed 🎉

See Details

Test product:

  • Use comment "[crw-ci-test]" to rerun happy path E2E test.
  • Use comment "[crw-ci-test --rebuild]" to re-build the images and rerun happy path E2E test.

Eclipse Che QE channel: https://mattermost.eclipse.org/eclipse/channels/eclipse-che-qe

Copy link
Contributor

@metlos metlos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree with @sparkoo 's comments and have some more minor things...

@che-bot
Copy link
Contributor

che-bot commented Sep 2, 2021

❌ E2E Happy path tests failed ❗

See Details

Test product:

  • Use comment "[crw-ci-test]" to rerun happy path E2E test.
  • Use comment "[crw-ci-test --rebuild]" to re-build the images and rerun happy path E2E test.

Eclipse Che QE channel: https://mattermost.eclipse.org/eclipse/channels/eclipse-che-qe

@che-bot
Copy link
Contributor

che-bot commented Sep 3, 2021

❌ E2E Happy path tests failed ❗

See Details

Test product:

  • Use comment "[crw-ci-test]" to rerun happy path E2E test.
  • Use comment "[crw-ci-test --rebuild]" to re-build the images and rerun happy path E2E test.

Eclipse Che QE channel: https://mattermost.eclipse.org/eclipse/channels/eclipse-che-qe

@che-bot
Copy link
Contributor

che-bot commented Sep 3, 2021

❌ E2E Happy path tests failed ❗

See Details

Test product:

  • Use comment "[crw-ci-test]" to rerun happy path E2E test.
  • Use comment "[crw-ci-test --rebuild]" to re-build the images and rerun happy path E2E test.

Eclipse Che QE channel: https://mattermost.eclipse.org/eclipse/channels/eclipse-che-qe

@skabashnyuk
Copy link
Contributor Author

[crw-ci-test --rebuild]

@che-bot
Copy link
Contributor

che-bot commented Sep 3, 2021

❌ E2E Happy path tests failed ❗

See Details

Test product:

  • Use comment "[crw-ci-test]" to rerun happy path E2E test.
  • Use comment "[crw-ci-test --rebuild]" to re-build the images and rerun happy path E2E test.

Eclipse Che QE channel: https://mattermost.eclipse.org/eclipse/channels/eclipse-che-qe

@skabashnyuk
Copy link
Contributor Author

[crw-ci-test --rebuild]

@skabashnyuk
Copy link
Contributor Author

@sparkoo @metlos I've addressed your comments. Code tested with che7 and devworkspaces.

@che-bot
Copy link
Contributor

che-bot commented Sep 3, 2021

❌ E2E Happy path tests failed ❗

See Details

Test product:

  • Use comment "[crw-ci-test]" to rerun happy path E2E test.
  • Use comment "[crw-ci-test --rebuild]" to re-build the images and rerun happy path E2E test.

Eclipse Che QE channel: https://mattermost.eclipse.org/eclipse/channels/eclipse-che-qe

@skabashnyuk
Copy link
Contributor Author

[crw-ci-test --rebuild]

@che-bot
Copy link
Contributor

che-bot commented Sep 3, 2021

✅ E2E Happy path tests succeed 🎉

See Details

Test product:

  • Use comment "[crw-ci-test]" to rerun happy path E2E test.
  • Use comment "[crw-ci-test --rebuild]" to re-build the images and rerun happy path E2E test.

Eclipse Che QE channel: https://mattermost.eclipse.org/eclipse/channels/eclipse-che-qe

Copy link
Member

@sparkoo sparkoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tested with devworkspaces

@che-bot
Copy link
Contributor

che-bot commented Sep 9, 2021

❌ E2E Happy path tests failed ❗

See Details

Test product:

  • Use comment "[crw-ci-test]" to rerun happy path E2E test.
  • Use comment "[crw-ci-test --rebuild]" to re-build the images and rerun happy path E2E test.

Eclipse Che QE channel: https://mattermost.eclipse.org/eclipse/channels/eclipse-che-qe

@skabashnyuk
Copy link
Contributor Author

[crw-ci-test --rebuild]

@che-bot
Copy link
Contributor

che-bot commented Sep 9, 2021

❌ E2E Happy path tests failed ❗

See Details

Test product:

  • Use comment "[crw-ci-test]" to rerun happy path E2E test.
  • Use comment "[crw-ci-test --rebuild]" to re-build the images and rerun happy path E2E test.

Eclipse Che QE channel: https://mattermost.eclipse.org/eclipse/channels/eclipse-che-qe

@che-bot
Copy link
Contributor

che-bot commented Sep 9, 2021

❌ E2E Happy path tests failed ❗

See Details

Test product:

  • Use comment "[crw-ci-test]" to rerun happy path E2E test.
  • Use comment "[crw-ci-test --rebuild]" to re-build the images and rerun happy path E2E test.

Eclipse Che QE channel: https://mattermost.eclipse.org/eclipse/channels/eclipse-che-qe

@che-bot
Copy link
Contributor

che-bot commented Sep 9, 2021

✅ E2E Happy path tests succeed 🎉

See Details

Test product:

  • Use comment "[crw-ci-test]" to rerun happy path E2E test.
  • Use comment "[crw-ci-test --rebuild]" to re-build the images and rerun happy path E2E test.

Eclipse Che QE channel: https://mattermost.eclipse.org/eclipse/channels/eclipse-che-qe

@skabashnyuk skabashnyuk merged commit 359ebb3 into eclipse-che:main Sep 13, 2021
@skabashnyuk skabashnyuk deleted the jakarta branch September 13, 2021 05:20
@skabashnyuk skabashnyuk added this to the 7.37 milestone Sep 13, 2021
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.

5 participants