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

🧪devfile migrated to v2.1.0🧪 #148

Closed
wants to merge 5 commits into from
Closed

Conversation

ckavili
Copy link
Contributor

@ckavili ckavili commented Sep 14, 2022

Due to the changes on devspaces and the following error on OpenShift 4.11, devfile is migrated to v2.1.0.

Danger alert:
Failed to create a workspace. Failed to create a new workspace from the devfile, reason: Unable to resolve theia plugins: eclipse/che-machine-exec-plugin/latest is a mandatory plug-in but definition is not found on the plug-in registry. Aborting !

However, I am not sure if this devfile would work on OpenShift 4.10 or before. I don't have an OpenShift 4.10 to try at the moment 🫣🫣 If it works, it is good! But since in tech-exercise documentation, we are directly pointing to main devfile, maybe we need to hold this to merge?

Should we update tech-exercise to point to a specific version? I am not sure if everyone is going to run TL500 on OpenShift 4.11 🤷🏻‍♀️ What do you think @eformat @springdo? 👀

@ckavili ckavili requested review from springdo and eformat September 14, 2022 13:56
@ckavili
Copy link
Contributor Author

ckavili commented Sep 14, 2022

And I guess this should go hand in hand with #147 🙈

@eformat
Copy link
Member

eformat commented Nov 13, 2022

yeah .. also included in #147 which has all the changes for 4.11 ... so could close this since its a subset ?

@ckavili
Copy link
Contributor Author

ckavili commented Nov 21, 2022

covered with #147 - thanks @eformat 🚀 closing this 🙃

@ckavili ckavili closed this Nov 21, 2022
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.

2 participants