-
Notifications
You must be signed in to change notification settings - Fork 266
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
[RFC] Implement module for Slowroll testing #18422
base: master
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||
---|---|---|---|---|---|---|---|---|
@@ -0,0 +1,27 @@ | ||||||||
# SUSE's openQA tests | ||||||||
# | ||||||||
# Copyright SUSE LLC | ||||||||
# SPDX-License-Identifier: FSFAP | ||||||||
|
||||||||
# Summary: Add slowroll repos in the system | ||||||||
# Maintainer: Yiannis Bonatakis <ybonatakis@suse.com> | ||||||||
|
||||||||
use Mojo::Base 'opensusebasetest'; | ||||||||
use testapi; | ||||||||
use serial_terminal 'select_serial_terminal'; | ||||||||
|
||||||||
sub run { | ||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||
select_serial_terminal; | ||||||||
assert_script_run "rm /etc/zypp/repos.d/*"; | ||||||||
assert_script_run "zypper -n --gpg-auto-import-keys ar --refresh http://cdn.opensuse.org/slowroll/repo/oss/ base-oss"; | ||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Now those repos as discussed in a ticket (or was it jitsi) are wrong as they should come from openQA for the to be tested state, not the released repos There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. a side question, why a new test module, when we could use the same or rehuse the same from leap? |
||||||||
assert_script_run "zypper -n --gpg-auto-import-keys ar --refresh http://cdn.opensuse.org/slowroll/repo/non-oss/ base-non-oss"; | ||||||||
assert_script_run "zypper -n --gpg-auto-import-keys ar --refresh -p 80 http://cdn.opensuse.org/update/slowroll/repo/oss/ update"; | ||||||||
|
||||||||
assert_script_run "zypper -n --gpg-auto-import-keys dup"; | ||||||||
} | ||||||||
|
||||||||
sub test_flags { | ||||||||
return {fatal => 1}; | ||||||||
} | ||||||||
|
||||||||
1; |
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.
@foursixnine IMHO this could/should be core-team, ok?
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.
@b10n1k Can you clarify? I don't see a change, or comment here
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.
As the author i used myself. But this can change later when @foursixnine clarify the @okurz 's question
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.
Fine, let's keep as is for now
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.
@okurz indeed! @b10n1k I'm fine with this being en qe-core, I think we can somehow merge the code, but since Yogesh left, we haven't been able to help here again.