[graphics/generator_* ] generated local jobs should categorizing based on graphic chip

Bug #1314610 reported by Yung Shen
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Checkbox Provider - Base
Fix Released
High
Unassigned
Provider for Plainbox - Canonical Certification (Legacy)
Fix Released
High
Unassigned

Bug Description

current graphics/generator_* will produce jobs like below:

    graphics/1_compiz_check_UMA_graphic
    graphics/2_compiz_check_DIscrete_graphic

for the system support nvidia-prime, it requires logout (restart X) to change graphic driver.
so in this order it makes problem, jobs will not run correctly.

we will need a job to ask user to switch driver and also make jobs starts with 1_* runs with the same priority then running 2_* jobs.

Revision history for this message
Daniel Manrique (roadmr) wrote :

Hello Yung,

The switch job was just merged today, so the sequence would be:

- Run all 1_* jobs
- Run the 2_switch job. This job asks the user to switch to the second card. checkbox-gui has to be stopped for this.
- When the user restarts checkbox-gui and selects "continue", it will run all the 2_* jobs.

I expect packages with this change to be available tomorrow. I'll mark this bug as Fix Committed, but if after tomorrow this is not yet working, please set it back to confirmed and tell me what you see.

The plainbox-provider-checkbox and -canonical-certification packages should have bzr2960 for these changes to be there.

Changed in checkbox:
status: New → Fix Committed
affects: checkbox → plainbox-provider-checkbox
Revision history for this message
Daniel Manrique (roadmr) wrote :

This bug affects two providers: -checkbox for the addition of the switcher job, and -canonical-certification for the required whitelist modifications.

Changed in plainbox-provider-canonical-certification:
status: New → Fix Committed
importance: Undecided → High
Changed in plainbox-provider-checkbox:
importance: Undecided → High
Revision history for this message
Po-Hsu Lin (cypressyew) wrote :

Verified with a I+A system today with the latest PPA.
The switch job does exist now , and job sequence works as expected.

For improvements, I think we could make 2_* jobs depend on that switch job.

Revision history for this message
Daniel Manrique (roadmr) wrote :

The dependency would make sense, but we'd have to make a series of changes.

The reason is that dependencies don't support wildcards, and since the names are variable, I'd have to use wildcards.

The dependency for 2_* jobs would need to look like:

depends: 2_switch_card_.*

this is because the 2_switch_card_ job will include the name of the actual card.

But as mentioned, expressions in depends aren't supported; we'd need to use a single complete job name.

We can always change the name to just 2_switch_card which is predictable, but in that case it may be confusing because the job name wouldn't mention the card.

Daniel Manrique (roadmr)
Changed in plainbox-provider-checkbox:
status: Fix Committed → Fix Released
Changed in plainbox-provider-canonical-certification:
status: Fix Committed → Fix Released
milestone: none → 2014w19
Changed in plainbox-provider-checkbox:
milestone: none → 0.5
Changed in plainbox-provider-canonical-certification:
milestone: 2014w19 → 0.1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.