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
Change Theia GHA tests to use quay.io #1566
Conversation
Signed-off-by: Billie Simmons <49491949+JillieBeanSim@users.noreply.github.com>
Signed-off-by: Billie Simmons <49491949+JillieBeanSim@users.noreply.github.com>
Signed-off-by: Billie Simmons <49491949+JillieBeanSim@users.noreply.github.com>
I have it pulling the image from quay.io with tag 1.18.0. We do have some failing tests that could be due to the update to 1.18.0 @crawr could you help me with trying to fix the tests please |
I pulled the docker image of theia 1.18.0 from quay.io and installed the ZE plugin and it looks like the |
Signed-off-by: Billie Simmons <49491949+JillieBeanSim@users.noreply.github.com>
Signed-off-by: Billie Simmons <49491949+JillieBeanSim@users.noreply.github.com>
Signed-off-by: Billie Simmons <49491949+JillieBeanSim@users.noreply.github.com>
Signed-off-by: Billie Simmons <49491949+JillieBeanSim@users.noreply.github.com>
Signed-off-by: Billie Simmons <49491949+JillieBeanSim@users.noreply.github.com>
Signed-off-by: Billie Simmons <49491949+JillieBeanSim@users.noreply.github.com>
Signed-off-by: Billie Simmons <49491949+JillieBeanSim@users.noreply.github.com>
I was able to fix the permission issues I was seeing by adding This is the command I was using locally to start up the docker image |
@crawr I was curious about the We may want to run more checks to see if anything else is broken in 1.18.0 too. |
The "can't find error" is related to those empty input fields. It's expecting to find the next input prompt after "blank input" but it never comes up. Either we revise the tests to put "fake" in user/pw and values in other fields, or we investigate why it doesn't accept blank input anymore. |
@crawr I am thinking investigate since it will have to be fixed regardless right? I don't think changing the test is good since it is suppose to help us find these breaking changes. |
Signed-off-by: Billie Simmons <49491949+JillieBeanSim@users.noreply.github.com>
@crawr @jellypuno I guess the follow up question is, do we want to go ahead and bring this change in knowing the theia tests are broken due to breaking changes in 1.18.0 or leave as is for the moment until we can investigate. As is, the tests are still broken due to the docker image not being available. We do already have the supported theia version documented so we are good on that. |
My vote: I think we should review this as is, even if the tests are not passing, so we have a docker image going forward for testing. |
Signed-off-by: Billie Simmons <49491949+JillieBeanSim@users.noreply.github.com>
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.
This PR looks good to me! The Theia GHA tests now appear to be pulling the latest image from quay.io. The failing Theia tests may be fixed by @phaumer's work on #1493 (comment), but those changes will not require updates in this PR.
Thanks @JillieBeanSim for this fix!
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! Thanks @JillieBeanSim
Signed-off-by: Billie Simmons 49491949+JillieBeanSim@users.noreply.github.com
Proposed changes
Release Notes
Milestone:
Changelog:
Types of changes
What types of changes does your code introduce to Zowe Explorer?
Put an
x
in the boxes that applyChecklist
Put an
x
in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This checklist will be used as reference for both the contributor and the revieweryarn workspace vscode-extension-for-zowe vscode:prepublish
has been executedReminder
After a PR is merged into the
master
branch, create a PR frommaster
to thenext
branch resolving any conflicts.Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...