Project

General

Profile

Actions

action #9962

closed

migration_offline_sle12: https://openqa.suse.de//tests/168857 failed because of "missing INSTLANG" in the backend

Added by dmaiocchi almost 9 years ago. Updated almost 9 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Bugs in existing tests
Target version:
-
Start date:
2015-12-18
Due date:
% Done:

100%

Estimated time:
Difficulty:


Checklist

  • SLE
  • TW
  • Leap
Actions #1

Updated by okurz almost 9 years ago

seems INSTLANG is necessary after all, see https://openqa.suse.de/tests/168857/modules/welcome/steps/3 but not needed for https://openqa.suse.de/tests/168858/modules/welcome/steps/3 can anyone explain that to me?

INSTLANG seems to not be set anywhere in os-autoinst-distri-opensuse (neither "master" nor "sle12") and also not in https://openqa.suse.de/admin/test_suites and does not appear in https://openqa.suse.de/tests/168858 or https://openqa.suse.de/tests/168857 . where should it come from?

Actions #2

Updated by okurz almost 9 years ago

  • Category set to Bugs in existing tests
  • Status changed from New to Feedback
  • Assignee deleted (okurz)
Actions #3

Updated by okurz almost 9 years ago

  • Target version set to 156

A later iteration of the test ran at https://openqa.suse.de/tests/177141/modules/welcome/steps/3/src
But during the time between the failing and the working again test I do not find any changes in neither os-autoinst nor os-autoinst-distri-opensuse (neither master nor sle branch) mentioning INSTLANG. So maybe the database was changed somehow? In openqa.suse.de/admin/ I don't see the variable mentioned where I didn't expect to see it before. Checking https://openqa.suse.de/admin/auditlog reveals

{ needle=>"/var/lib/openqa/share/tests/sle/needles/scc-addon-license-ha-20160105.png", tags =>["ENV-DISTRI-sle", "ENV-INSTLANG-en_US", "ENV-VERSION-12-SP2-HA", "scc-addon-license-ha",], update=>"no",}

but I don't assume this would make it fix for the mentioned test. So how was it fixed? I would like to understand this for the future.

Actions #4

Updated by RBrownSUSE almost 9 years ago

  • Assignee set to RBrownSUSE
  • Target version changed from 156 to 154
  • % Done changed from 0 to 80
Actions #5

Updated by okurz almost 9 years ago

  • Subject changed from migration_offline_sle12: https://openqa.suse.de//tests/168857, failed to migration_offline_sle12: https://openqa.suse.de//tests/168857 failed because of "missing INSTLANG" in the backend
Actions #6

Updated by RBrownSUSE almost 9 years ago

  • Status changed from Feedback to Resolved
  • % Done changed from 80 to 100

PR Accepted

Actions #7

Updated by okurz almost 9 years ago

  • Status changed from Resolved to Feedback

@RBrownSUSE can you explain me regarding to https://progress.opensuse.org/issues/9962#note-3 why it was already partially fixed in before?

Actions #8

Updated by RBrownSUSE almost 9 years ago

  • Checklist item changed from to [ ] SLE, [ ] TW, [ ] Leap
  • Target version deleted (154)
Actions #9

Updated by okurz almost 9 years ago

  • Checklist item changed from to [x] SLE
Actions #10

Updated by okurz almost 9 years ago

  • Checklist item changed from to [ ] SLE
Actions #11

Updated by okurz almost 9 years ago

Potentially related https://openqa.suse.de/tests/181150/modules/partitioning_finish/steps/1
It seems to me the needles can't match because INSTLANG definition is also missing here?

Actions #12

Updated by RBrownSUSE almost 9 years ago

okurz wrote:

Potentially related https://openqa.suse.de/tests/181150/modules/partitioning_finish/steps/1
It seems to me the needles can't match because INSTLANG definition is also missing here?

That's not related at all - https://openqa.suse.de/tests/181150/modules/partitioning_finish/steps/1 failed because it was expecting the Timezone screen and still had the Partitioning screen 30 seconds after clicking Next (it updated a few seconds later, and shown by the follow-up needle)

Please try your best to avoid conflating totally unrelated issues. This issue is resolved and was only changed to feedback because you wanted further explanation

Actions #13

Updated by RBrownSUSE almost 9 years ago

  • Priority changed from Urgent to Normal
Actions #14

Updated by okurz almost 9 years ago

RBrownSUSE wrote:

Please try your best to avoid conflating totally unrelated issues. This issue is resolved and was only changed to feedback because you wanted further explanation

Sorry to disappoint you but I am already giving my best :-/ Seemed to me like it was missing the INSTLANG tag here

Actions #15

Updated by okurz almost 9 years ago

  • Checklist item changed from [ ] SLE, [ ] TW, [ ] Leap to [x] SLE, [x] TW, [x] Leap
  • Status changed from Feedback to Resolved

fixed, see #9962#note-4

Actions

Also available in: Atom PDF