Project

General

Profile

Actions
Copy link

action #56876

closed

YAML setting overwrite each other

Added by coolo over 5 years ago. Updated over 5 years ago.

Status:
Resolved
Priority:
Urgent
Assignee:
tinita
Category:
Regressions/Crashes
Target version:
Done
Start date:
2019-09-13
Due date:
% Done:

100%

Estimated time:

Description

proof of concept: https://github.com/os-autoinst/openQA/pull/2329

I deployed this version to fix production, but having a little more compact test case sounds reasonable :)

But note that even this lengthy test case fails randomly - as we do not sort the keys of yaml

Actions
Copy link
 #1

Updated by tinita over 5 years ago

coolo wrote:

But note that even this lengthy test case fails randomly - as we do not sort the keys of yaml

I think we should fix that too. Whenever we iterate over hash keys and it's not super important to be performant and a large hash, then we should sort.

Actions
Copy link
 #2

Updated by tinita over 5 years ago

  • Status changed from New to In Progress
  • Assignee set to tinita
Actions
Copy link
 #3

Updated by tinita over 5 years ago

  • % Done changed from 0 to 100
Actions
Copy link
 #4

Updated by tinita over 5 years ago

PR was merged

Actions
Copy link
 #5

Updated by coolo over 5 years ago

  • Status changed from In Progress to Resolved
  • Target version changed from Current Sprint to Done
Actions
Copy link

Also available in: Atom PDF