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

Use of new theia (0.3.17) #1162

Merged
merged 1 commit into from
Dec 11, 2018
Merged

Use of new theia (0.3.17) #1162

merged 1 commit into from
Dec 11, 2018

Conversation

benoitf
Copy link
Contributor

@benoitf benoitf commented Dec 11, 2018

Change-Id: I487d014370f7f2f441d7b26fec9b623bb69a04b3
Signed-off-by: Florent Benoit [email protected]

Change-Id: I487d014370f7f2f441d7b26fec9b623bb69a04b3
Signed-off-by: Florent Benoit <[email protected]>
@centos-ci
Copy link
Collaborator

Can one of the admins verify this patch?

2 similar comments
@centos-ci
Copy link
Collaborator

Can one of the admins verify this patch?

@centos-ci
Copy link
Collaborator

Can one of the admins verify this patch?

@benoitf
Copy link
Contributor Author

benoitf commented Dec 11, 2018

cc @l0rd

@ibuziuk ibuziuk self-requested a review December 11, 2018 09:39
@benoitf
Copy link
Contributor Author

benoitf commented Dec 11, 2018

@ibuziuk can you launch the tests ?

@ibuziuk
Copy link
Collaborator

ibuziuk commented Dec 11, 2018

@benoitf yes but those are broken dramatically currently. Trying to grok what is the reason - #1160 (comment)

@ibuziuk
Copy link
Collaborator

ibuziuk commented Dec 11, 2018

[e2e-test-released] [e2e-test-beta]

@ibuziuk
Copy link
Collaborator

ibuziuk commented Dec 11, 2018

[build]

@ibuziuk
Copy link
Collaborator

ibuziuk commented Dec 11, 2018

[test]

@benoitf
Copy link
Contributor Author

benoitf commented Dec 11, 2018

BTW I think the only check for the registry update is that image is able to be built (as it includes some (the only) tests)

@ibuziuk
Copy link
Collaborator

ibuziuk commented Dec 11, 2018

@l0rd has the power of merging without e2e tests been passed ;-)

@l0rd l0rd merged commit 79c482d into openshiftio:master Dec 11, 2018
@l0rd
Copy link
Collaborator

l0rd commented Dec 11, 2018

@benoitf @ibuziuk merged

@centos-ci
Copy link
Collaborator

The E2E smoke test with feature level set to 'released' failed.

Link URL
job log https://ci.centos.org/job/devtools-saas-openshiftio-e2e-smoketest-released/406/console
failure message http://artifacts.ci.centos.org/devtools/e2e/devtools-saas-openshiftio-e2e-smoketest-released/406/failures.txt
artifacts http://artifacts.ci.centos.org/devtools/e2e/devtools-saas-openshiftio-e2e-smoketest-released/406

Please review the job results and re-run the job by commenting [e2e-test-released] if necessary.

@centos-ci
Copy link
Collaborator

The E2E smoke test with feature level set to 'beta' failed.

Link URL
job log https://ci.centos.org/job/devtools-saas-openshiftio-e2e-smoketest-beta/401/console
failure message http://artifacts.ci.centos.org/devtools/e2e/devtools-saas-openshiftio-e2e-smoketest-beta/401/failures.txt
artifacts http://artifacts.ci.centos.org/devtools/e2e/devtools-saas-openshiftio-e2e-smoketest-beta/401

Please review the job results and re-run the job by commenting [e2e-test-beta] if necessary.

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.

4 participants