action #153805
opencoordination #96596: [qe-core][CI] CI/CD and Coding style improvements
[qe-core] Implement checks so that tests that don't use assert_screen, use use serial console instead
0%
Description
Since we're deprioritizing screen-based testing, something that has bothered me for quite a while is that we often have test modules that are console-only (i.e no ncurses) and could be better handled with a serial console, but they use root console or others. So we'd like to prevent test developers from creating new tests, which such bad practice
In reality what we want to achieve is to Prevent openQA from relying on VNC, as assert_screen needs a tty (either console or x11/wayland)
Particularly relevant for tests with consolebasetest
in their parent hierarchy.
Acceptance criteria:¶
- The RFC request was added to CONTRIBUTING.md with the new rule.
- Pull Request is merged
- A format is considered to allow or skip the rule check purposefully
Updated by szarate about 1 year ago
- Sprint changed from QE-Core: December Sprint 23 (Dec 13 - Jan 10) to QE-Core: February Sprint 24 (Jan 31 - Feb 28)
Updated by szarate about 1 year ago
The security team might need screenshots as part of evidence of what they do, so it's a "in most cases" rather than in all cases.
Possibly there could be a child class of opensusebasetest or flag that could alter the behavior of the checker.
Updated by mgrifalconi 11 months ago
- Sprint changed from QE-Core: February Sprint 24 (Jan 31 - Feb 28) to QE-Core: March Sprint 24 (Mar 06 - Mar 28)
Updated by mgrifalconi about 1 month ago
- Tags changed from platform-team, qe-core-february-sprint to platform-team
- Status changed from In Progress to Feedback
- Assignee deleted (
cvidot) - Target version changed from QE-Core: Ready to future
Nothing happening since few months, updating current status. We will review this later on