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

Add salt and rabbitmq test suites #1549

Merged
merged 2 commits into from
Jul 21, 2016
Merged

Conversation

okurz
Copy link
Member

@okurz okurz commented Jul 21, 2016

  • Add basic salt test suite
  • Add rabbitmq test suite

screenshot of local verification on Tumbleweed:
openqa_rabbitmq_salt

@sysrich
Copy link
Member

sysrich commented Jul 21, 2016

Adding new tests and fixing loading consoles within snapshots? This seems like two very different topics, shouldn't that be two different pull requests?

SCNR ;)

select_console 'user-console';
assert_script_sudo("zypper -n in salt-master salt-minion");
assert_script_sudo("systemctl start salt-master");
assert_script_run("systemctl status salt-server");
Copy link
Member

Choose a reason for hiding this comment

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

I think you need to do s/salt-server/salt-master

@okurz okurz changed the title Add salt and rabbitmq test suites; Fix consoletests continuation from snapshots [WIP] Add salt and rabbitmq test suites Jul 21, 2016
@okurz
Copy link
Member Author

okurz commented Jul 21, 2016

split out the "fix consoletests part". Thanks for the suggestion. I was just trying to tackle these issues :-)

Based on https://www.rabbitmq.com/tutorials/tutorial-one-python.html

Solely added because someone added "rabbitmq" to the Leap42.2 test plan :-)

Local verification done.
Local verification done.
@okurz
Copy link
Member Author

okurz commented Jul 21, 2016

rabbitmq fails on Tumbleweed because "ssl" is missing for python2 but python-pika is only available for python2. I would say this is a missing package problem in Tumbleweed. How should we proceed? It works on Leap42.2, though.

@okurz okurz changed the title [WIP] Add salt and rabbitmq test suites Add salt and rabbitmq test suites Jul 21, 2016
@sysrich
Copy link
Member

sysrich commented Jul 21, 2016

File bugs, merge the test but run it as an extratest as not to block TW
releases?

On 21 July 2016 at 17:28, Oliver Kurz notifications@github.com wrote:

rabbitmq fails on Tumbleweed because "ssl" is missing for python2 but
python-pika is only available for python2. I would say this is a missing
package problem in Tumbleweed. How should we proceed? It works on Leap42.2,
though.


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#1549 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABJGoLTKh_8RioYIaHO_7Yivf3HXYMX3ks5qX5AkgaJpZM4JR17L
.

@okurz okurz merged commit 30faf49 into os-autoinst:master Jul 21, 2016
@okurz
Copy link
Member Author

okurz commented Jul 21, 2016

just merged as discussed with sysrich. Will handle the missing package on TW based on the fail. Also, for Leap42.2 we are missing extratests.

@DimStar77 Tumbleweed seems to miss python2-pyOpenSSL

@lnussel Leap42.2 are missing all the "extra tests": Compare to https://openqa.opensuse.org/tests/overview?distri=opensuse&groupid=1&version=Tumbleweed and look for "extra_tests_on_…". Can we add them?

@okurz okurz deleted the feature/rabbitmq branch July 21, 2016 15:32
@DimStar77
Copy link
Contributor

On Thu, 2016-07-21 at 08:32 -0700, Oliver Kurz wrote:

just merged as discussed with sysrich. Will handle the missing
package on TW based on the fail. Also, for Leap42.2 we are missing
extratests.

@DimStar77 Tumbleweed seems to miss python2-pyOpenSSL

python-pyOpenSSL has been part of the distribution since 2011,
currently in version 16.0.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants