Skip to content
This repository has been archived by the owner on Apr 4, 2023. It is now read-only.

Rework images for remote plugins #485

Merged
merged 38 commits into from
Oct 25, 2019
Merged

Rework images for remote plugins #485

merged 38 commits into from
Oct 25, 2019

Conversation

evidolob
Copy link
Contributor

What does this PR do?

Rewrites remote plugins images, to avoid extending theia-endpoint-runtime image and using new remote plugin runtime binary injection.

What issues does this PR fix or reference?

eclipse-che/che#14797

Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
Signed-off-by: Yevhen Vydolob <yvydolob@redhat.com>
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
Signed-off-by: Yevhen Vydolob <yvydolob@redhat.com>
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
Signed-off-by: Yevhen Vydolob <yvydolob@redhat.com>
Signed-off-by: Yevhen Vydolob <yvydolob@redhat.com>
Signed-off-by: Yevhen Vydolob <yvydolob@redhat.com>
Signed-off-by: Yevhen Vydolob <yvydolob@redhat.com>
Signed-off-by: Yevhen Vydolob <yvydolob@redhat.com>
@che-bot
Copy link
Contributor

che-bot commented Oct 15, 2019

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has failed:

Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@che-bot
Copy link
Contributor

che-bot commented Oct 18, 2019

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has failed:

Signed-off-by: Mykola Morhun <mmorhun@redhat.com>
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@che-bot
Copy link
Contributor

che-bot commented Oct 22, 2019

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has failed:

@che-bot
Copy link
Contributor

che-bot commented Oct 22, 2019

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has failed:

Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@che-bot
Copy link
Contributor

che-bot commented Oct 24, 2019

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has failed:

Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@che-bot
Copy link
Contributor

che-bot commented Oct 24, 2019

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has failed:

Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@che-bot
Copy link
Contributor

che-bot commented Oct 24, 2019

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has failed:

@che-bot
Copy link
Contributor

che-bot commented Oct 25, 2019

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has failed:

@AndrienkoAleksandr
Copy link
Contributor

@benoitf
Copy link
Contributor

benoitf commented Oct 25, 2019

hi @AndrienkoAleksandr
CMD ${PLUGIN_REMOTE_ENDPOINT_EXECUTABLE} is not added automatically by a broker?

I thought that images were just taking care of running on openshift with an entrypoint and then the command is added by a 3rd party entity ?

@@ -0,0 +1,32 @@
#!/bin/sh
Copy link
Contributor

Choose a reason for hiding this comment

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

maybe entrypoint.sh could be copied from a template with build.include script so we don't need to push it to git 30 times (better to maintain)

The same for the instructions that are part of Dockerfile to make it compliant

@AndrienkoAleksandr
Copy link
Contributor

CMD ${PLUGIN_REMOTE_ENDPOINT_EXECUTABLE} is not added automatically by a broker?

Override Dockerfile#CMD is easy to implement in plugin-broker - one line of the code and fix for few tests.
But we discussed with @amisevsk and decided don't override CMD:
eclipse-che/che-plugin-registry#233 (comment)
eclipse-che/che-plugin-registry#233 (comment)

@benoitf
Copy link
Contributor

benoitf commented Oct 25, 2019

@AndrienkoAleksandr yeah the step 1, it needs to be enhanced later.

Copy link
Contributor

@benoitf benoitf left a comment

Choose a reason for hiding this comment

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

copy/paste and uid fixes could be done at a single location instead of duplicating everything
the same for entrypoint/cmd stuff but it's step 1

@dmytro-ndp
Copy link
Contributor

crw-ci-test

@che-bot
Copy link
Contributor

che-bot commented Oct 25, 2019

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has failed:

Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@AndrienkoAleksandr
Copy link
Contributor

All images manually tested.

@AndrienkoAleksandr AndrienkoAleksandr merged commit d65cb10 into master Oct 25, 2019
@AndrienkoAleksandr AndrienkoAleksandr deleted the reworkImages branch October 25, 2019 11:46
@che-bot
Copy link
Contributor

che-bot commented Oct 25, 2019

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has failed:

vinokurig pushed a commit that referenced this pull request Apr 6, 2021
* Extract and use plugin images digest from env.

Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants