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

fix: Allow to run smoke test on pull request #262

Merged
merged 14 commits into from
Sep 15, 2023
Merged

Conversation

vitaliy-guliy
Copy link
Contributor

@vitaliy-guliy vitaliy-guliy commented Jul 27, 2023

What does this PR do?

Fixes building of pull-request based che-code docker image

What issues does this PR fix?

eclipse-che/che#22341

How to test this PR?

Just check that smoke test passed for this pull request

Signed-off-by: Vitaliy Gulyy <vgulyy@redhat.com>
@github-actions
Copy link

github-actions bot commented Jul 27, 2023

Click here to review and test in web IDE: Contribute

@github-actions
Copy link

Pull Request images published ✨
image: quay.io/che-incubator-pull-requests/che-code:pr-262-amd64
Dev image: quay.io/che-incubator-pull-requests/che-code-dev:pr-262-dev-amd64

Signed-off-by: Vitaliy Gulyy <vgulyy@redhat.com>
@github-actions
Copy link

Pull Request images published ✨
image: quay.io/che-incubator-pull-requests/che-code:pr-262-amd64
Dev image: quay.io/che-incubator-pull-requests/che-code-dev:pr-262-dev-amd64

Signed-off-by: Vitaliy Gulyy <vgulyy@redhat.com>
@github-actions
Copy link

Pull Request images published ✨
image: quay.io/che-incubator-pull-requests/che-code:pr-262-amd64
Dev image: quay.io/che-incubator-pull-requests/che-code-dev:pr-262-dev-amd64

Signed-off-by: Vitaliy Gulyy <vgulyy@redhat.com>
@github-actions
Copy link

Pull Request images published ✨
image: quay.io/che-incubator-pull-requests/che-code:pr-262-amd64
Dev image: quay.io/che-incubator-pull-requests/che-code-dev:pr-262-dev-amd64

2 similar comments
@github-actions
Copy link

Pull Request images published ✨
image: quay.io/che-incubator-pull-requests/che-code:pr-262-amd64
Dev image: quay.io/che-incubator-pull-requests/che-code-dev:pr-262-dev-amd64

@github-actions
Copy link

Pull Request images published ✨
image: quay.io/che-incubator-pull-requests/che-code:pr-262-amd64
Dev image: quay.io/che-incubator-pull-requests/che-code-dev:pr-262-dev-amd64

Signed-off-by: Vitaliy Gulyy <vgulyy@redhat.com>
Signed-off-by: Vitaliy Gulyy <vgulyy@redhat.com>
Signed-off-by: Vitaliy Gulyy <vgulyy@redhat.com>
Signed-off-by: Vitaliy Gulyy <vgulyy@redhat.com>
Signed-off-by: Vitaliy Gulyy <vgulyy@redhat.com>
Signed-off-by: Vitaliy Gulyy <vgulyy@redhat.com>
Signed-off-by: Vitaliy Gulyy <vgulyy@redhat.com>
Signed-off-by: Vitaliy Gulyy <vgulyy@redhat.com>
@github-actions
Copy link

Pull Request images published ✨
image: quay.io/che-incubator-pull-requests/che-code:pr-262-amd64
Dev image: quay.io/che-incubator-pull-requests/che-code-dev:pr-262-dev-amd64

@vitaliy-guliy vitaliy-guliy changed the title fix: build che-code docker image for smoke test fix: Allow to run smoke test pull request check Sep 15, 2023
@vitaliy-guliy vitaliy-guliy changed the title fix: Allow to run smoke test pull request check fix: Allow to run smoke test on pull request Sep 15, 2023
@vitaliy-guliy vitaliy-guliy marked this pull request as ready for review September 15, 2023 09:01
@vitaliy-guliy
Copy link
Contributor Author

@musienko-maxim

Copy link
Member

@azatsarynnyy azatsarynnyy left a comment

Choose a reason for hiding this comment

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

Thank you @vitaliy-guliy and @musienko-maxim for fixing the smoke tests PR check.
🎉

.github/workflows/smoke-test-pr-check.yaml Outdated Show resolved Hide resolved
Signed-off-by: Vitaliy Gulyy <vgulyy@redhat.com>
@github-actions
Copy link

Pull Request images published ✨
image: quay.io/che-incubator-pull-requests/che-code:pr-262-amd64
Dev image: quay.io/che-incubator-pull-requests/che-code-dev:pr-262-dev-amd64

@vitaliy-guliy vitaliy-guliy merged commit 2601f0e into main Sep 15, 2023
2 checks passed
@vitaliy-guliy vitaliy-guliy deleted the fix-smoke-test branch September 15, 2023 11:18
@devstudio-release
Copy link

Build 3.10 :: code_3.x/862: Console, Changes, Git Data

@devstudio-release
Copy link

Build 3.10 :: code_3.x/863: Console, Changes, Git Data

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

Build 3.10 :: code_3.x/862: SUCCESS

Upstream sync done; /DS_CI/sync-to-downstream_3.x/4572 triggered

@devstudio-release
Copy link

@devstudio-release
Copy link

Build 3.10 :: update-digests_3.x/4254: UNSTABLE

No new images detected: nothing to do!

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

Build 3.10 :: code_3.x/863: SUCCESS

Upstream sync done; /DS_CI/sync-to-downstream_3.x/4573 triggered

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

Build 3.10 :: sync-to-downstream_3.x/4582: SUCCESS

Build container: devspaces-operator-bundle synced; /DS_CI/get-sources-rhpkg-container-build_3.x/4374 triggered; /job/DS_CI/job/dsc_3.x triggered;

@devstudio-release
Copy link

Build 3.10 :: operator-bundle_3.x/2037: SUCCESS

Upstream sync done; /DS_CI/sync-to-downstream_3.x/4582 triggered

@devstudio-release
Copy link

Build 3.10 :: copyIIBsToQuay/1875: Console, Changes, Git Data

@devstudio-release
Copy link

Build 3.10 :: dsc_3.x/1343: Console, Changes, Git Data

@devstudio-release
Copy link

Build 3.10 :: update-digests_3.x/4257: SUCCESS

Detected new images: rebuild operator-bundle
* code; /DS_CI/operator-bundle_3.x/2037 triggered

@github-actions
Copy link

Pull Request images published ✨
image: quay.io/che-incubator-pull-requests/che-code:pr-262-amd64
Dev image: quay.io/che-incubator-pull-requests/che-code-dev:pr-262-dev-amd64

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