Project

General

Profile

action #80682

coordination #80142: [saga][epic] Scale out: Redundant/load-balancing deployments of openQA, easy containers, containers on kubernetes

coordination #43706: [epic] Generate "download&use" docker image of openQA for SUSE QA

Automatic tests for our openQA containers - worker only

Added by okurz over 2 years ago. Updated about 2 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Feature requests
Target version:
Start date:
2020-11-27
Due date:
% Done:

0%

Estimated time:
Difficulty:

Description

Motivation

Without automatic tests we would not call any contributions properly long-term supportable. These tests could be very simple, e.g. something like podman run --rm -it .... openqa-worker --help or something.

Acceptance criteria

  • AC2: The worker service(s) is/are started completely without errors from container within CI tests, i.e. stops asking for webUI server to connect to

Suggestions

  • Find out if OBS can execute tests for container images
  • Consider kiwi vs Dockerfile
  • Setup tests as part of the openQA CI (e.g. CircleCI or GitHub Actions)

Related issues

Copied from openQA Project - action #80520: Automatic tests for our openQA containers - webUI onlyResolved2020-11-27

Copied to openQA Project - action #80684: Automatic tests for our openQA containers - worker+webui connectionWorkable2020-11-27

History

#1 Updated by okurz over 2 years ago

  • Copied from action #80520: Automatic tests for our openQA containers - webUI only added

#2 Updated by okurz over 2 years ago

  • Copied to action #80684: Automatic tests for our openQA containers - worker+webui connection added

#3 Updated by ilausuch over 2 years ago

  • Status changed from Workable to In Progress
  • Assignee set to ilausuch

#4 Updated by openqa_review over 2 years ago

  • Due date set to 2020-12-19

Setting due date based on mean cycle time of SUSE QE Tools

#6 Updated by cdywan over 2 years ago

  • Status changed from In Progress to Feedback

This has been discussed a bit to see what the best way forward is. #81118 is about testing for os-autoinst and currently proposes a PR using GitHub Actions.

It was also suggested to extend our openqa in openqa tests, where already test the scenarios of installing openQA from git and from packages respectively

#7 Updated by cdywan over 2 years ago

  • Due date deleted (2020-12-19)

ilausuch Please mention the result of trying the openQA test approach herelater. For now I'm resetting the due date due to holidays.

#8 Updated by ilausuch over 2 years ago

  • Status changed from Feedback to In Progress

#9 Updated by openqa_review over 2 years ago

  • Due date set to 2021-02-09

Setting due date based on mean cycle time of SUSE QE Tools

#10 Updated by ilausuch over 2 years ago

The current test (https://openqa.opensuse.org/tests/1607448) works:

  • Cannot finally connect to the webUI because doesn't have the credentials. Obtaining a key/secret is not trivial, so this test cannot continue until we have this feature.

#11 Updated by cdywan over 2 years ago

ilausuch wrote:

The current test (https://openqa.opensuse.org/tests/1607448) works:

  • Cannot finally connect to the webUI because doesn't have the credentials. Obtaining a key/secret is not trivial, so this test cannot continue until we have this feature.

Would you mind filing a new ticket for the missing feature to create users with credentials? I'd consider it out of scope for this ticket.

#12 Updated by ilausuch over 2 years ago

#13 Updated by okurz over 2 years ago

Our ACs as defined in this ticket already correctly describe what is needed and that does not include the "proper connection", just trying to start and failing as expected.

#14 Updated by cdywan over 2 years ago

Ack. ilausuch were you going to update the ticket status, too? Since you said the test is working above.

#16 Updated by ilausuch over 2 years ago

  • Status changed from In Progress to Feedback

The final PR for worker container testing is this https://github.com/os-autoinst/os-autoinst-distri-openQA/pull/61

#17 Updated by ilausuch over 2 years ago

  • Status changed from Feedback to Resolved

#18 Updated by ilausuch over 2 years ago

  • Status changed from Resolved to In Progress

The PR is merged

#19 Updated by cdywan over 2 years ago

  • Status changed from In Progress to Resolved

I suspect this was meant to be Resolved.

#20 Updated by okurz about 2 years ago

  • Due date deleted (2021-02-09)

Also available in: Atom PDF