-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Enhance workspace integration test #11333
Conversation
257eef3
to
c70e175
Compare
80169e3
to
03a149f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's not recommended to do this, but rather to record the commitID and send it to the slack before committing the new commit
One problem with this is that the werft job sometimes ignores the cache, and our image names are dependent on the commit id, which can cause a series of strange problems, such as we may inadvertently replace some images that are otherwise good
e.g. some image need download resource from other website, we need prevent overwrite exist image
🤔 But adding a new commit to a new branch, we can't find the original main branch commit id. |
why? e.g. https://werft.gitpod-dev.com/job/gitpod-workspace-run-integration-tests-main.55 if you visit https://werft.gitpod-dev.com/job/gitpod-build-wk-inte-test-20220710000011.0 you got so if you want commit X, use |
/hold |
Signed-off-by: JenTing Hsiao <hsiaoairplane@gmail.com>
Signed-off-by: JenTing Hsiao <hsiaoairplane@gmail.com>
03a149f
to
fcbc4d0
Compare
Ugh, that makes sense. Thanks, @mustard-mh. Removing the commit. |
/unhold |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
Search the job with
with-preview
annotation.Related Issue(s)
Fixes #
How to test
Release Notes
Documentation
None
Werft options: