START_AFTER_TEST dependency error should use `@$MACHINE` syntax
Reading this message, and considering I was investigating failures to do with replacing test suites with
_staging in the name with parametrized ones, I was mildly confused:
START_AFTER_TEST=RAID0:64bit-staging not found - check for dependency typos and dependency cycles
I feel like this should be
RAID0@64bit-staging using an
@ rather than a
: to separate the name of the test from the machine. This is how it's displayed elsewhere.
- Assignee set to mkittler
- Target version changed from Ready to Current Sprint
@coolo Because you added the relation to the dependency graph: The dependency graph doesn't show the machine within the tool tip which shows
START_AFTER_TEST and the other variables. That is because the graph relies on the job dependencies database table which doesn't contain the original specification anymore. However, it would be possible to show the variables as specified from the job settings table.
- Status changed from New to In Progress
@coolo Consider tooltip of this job within its dependency tree: https://openqa.suse.de/tests/3548056#dependencies
The tooltip shows
STARTS_AFTER_TEST=skip_registration but as visible in the job settings it was actually specified as
START_AFTER_TEST=skip_registration:64bit. So unless we decide to display the variable from the job settings here (instead of making things up from the dependencies table) the issues aren't affecting each other. Not that I thought https://progress.opensuse.org/issues/59082 was only about the test name anyways (and not the dependency specification).