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

Apply workaround for yast content not being loaded for MU sp4 #15343

Merged
merged 1 commit into from
Aug 26, 2022

Conversation

skliu
Copy link
Contributor

@skliu skliu commented Aug 10, 2022

Apply workaround for yast content not being loaded for MU sp4

@skliu skliu force-pushed the yast_resize branch 3 times, most recently from b163aa6 to 27461bd Compare August 10, 2022 08:23
@jknphy jknphy added the qe-yast label Aug 10, 2022
tests/yast2_gui/yast2_bootloader.pm Outdated Show resolved Hide resolved
tests/yast2_gui/yast2_bootloader.pm Show resolved Hide resolved
tests/yast2_gui/yast2_bootloader.pm Outdated Show resolved Hide resolved
tests/yast2_gui/yast2_bootloader.pm Show resolved Hide resolved
@jknphy
Copy link
Contributor

jknphy commented Aug 10, 2022

@skliu any clue about what is this failure about at the end of the VR?

@skliu
Copy link
Contributor Author

skliu commented Aug 15, 2022

@skliu any clue about what is this failure about at the end of the VR?

I think it's due to the time out.

@jknphy
Copy link
Contributor

jknphy commented Aug 16, 2022

I see too much troubles with this workaround, now that we have libyui, we could switch to
yast2_gui/bootloader/bootcode_options
which covers only the first tab, but we can plan for covering a bit more.
For the instserver have you investigated the error? did you file a bug for it?

@jknphy
Copy link
Contributor

jknphy commented Aug 19, 2022

@skliu @JRivrain @ge0r Do we have a solid solution already for bootloader?, if not, please @skliu apply yast2_gui/bootloader/bootcode_options instead of needle-based module. And then we can merge this PR.
If we don't have a clear idea for the second issue, file a ticket with current findings, looks less important that bootloader module, we can live without it for a while.

@JRivrain
Copy link
Contributor

JRivrain commented Aug 19, 2022

@skliu any clue about what is this failure about at the end of the VR?

I think it's due to the time out.

I had mentioned it in https://progress.opensuse.org/issues/112589: there are more issues with this module, just applying the workaround is not enough to fix it I'm afraid.

@jknphy
Copy link
Contributor

jknphy commented Aug 22, 2022

@JRivrain we just need to fix the first error, correct, but we need a ticket with clear info to tackle the second error as part of the outcome of this task, that ticket that I rejected didn't have that info (including links to our observations could also help for next time)

@skliu are we missing something before merging? I remembered we talked offline about creating new module to be schedule before this to do zypper ref and autoimport the keys. Are you planning to add that?

@skliu
Copy link
Contributor Author

skliu commented Aug 22, 2022

@JRivrain we just need to fix the first error, correct, but we need a ticket with clear info to tackle the second error as part of the outcome of this task, that ticket that I rejected didn't have that info (including links to our observations could also help for next time)

@skliu are we missing something before merging? I remembered we talked offline about creating new module to be schedule before this to do zypper ref and autoimport the keys. Are you planning to add that?

only the coping CD problem. http://d218.qam.suse.de/tests/131#step/yast2_instserver/43

@skliu skliu force-pushed the yast_resize branch 7 times, most recently from 99b5188 to f6a205a Compare August 25, 2022 06:55
@skliu
Copy link
Contributor Author

skliu commented Aug 25, 2022

For the instserver have you investigated the error? did you file a bug for it?

Yes, I did. the first error is due to the time out of coping CD contents, and the rest errors are all due to the failure of sending "alt-f", so "yast2 instserver" window is not disappeared, all the errors happens randomly. BTW: where can I file a bug? on https://bugzilla.suse.com/ ?

@skliu
Copy link
Contributor Author

skliu commented Aug 25, 2022

@JRivrain we just need to fix the first error, correct, but we need a ticket with clear info to tackle the second error as part of the outcome of this task, that ticket that I rejected didn't have that info (including links to our observations could also help for next time)

@skliu are we missing something before merging? I remembered we talked offline about creating new module to be schedule before this to do zypper ref and autoimport the keys. Are you planning to add that?

the module is created.

@skliu
Copy link
Contributor Author

skliu commented Aug 25, 2022

I see too much troubles with this workaround, now that we have libyui, we could switch to yast2_gui/bootloader/bootcode_options which covers only the first tab, but we can plan for covering a bit more. For the instserver have you investigated the error? did you file a bug for it?

I created a progress ticket for it. https://progress.opensuse.org/issues/115748

@jknphy
Copy link
Contributor

jknphy commented Aug 25, 2022

LGTM

@jknphy jknphy merged commit a88bf04 into os-autoinst:master Aug 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants