Skip to content

Conversation

okurz
Copy link
Member

@okurz okurz commented Jul 14, 2025

No description provided.

@okurz
Copy link
Member Author

okurz commented Jul 15, 2025

I am trying to reproduce the CI failures locally. Right now in my Leap 15.6 based development environment I get different errors from an unexpected generated diff of test commands even with no local changes applied

@okurz
Copy link
Member Author

okurz commented Jul 15, 2025

@andrii-suse when I try to run make test locally I get a lot of errors and diff even if no changes compared to origin/master are applied, e.g.

F-abs/OBS:Server:Unstable test_before_after_diff.sh:bash: abs/OBS:Server:Unstable/print_rsync_iso.sh: No such file or directory
P:abs/OBS:Server:Unstable test_iso_consistent.sh
P:abs/OBS:Server:Unstable test_repo_consistent.sh
F-abs/Test1 test_before_after_diff.sh:bash: abs/Test1/print_rsync_iso.sh: No such file or directory
…
F-ibs/Devel:YaST:Agama:Head/base/ test_before_after_diff.sh:ibs/Devel:YaST:Agama:Head/base//print_openqa.sh: line 56: syntax error near unexpected token `('
F-ibs/Devel:YaST:Agama:Head/base/ test_iso_consistent.sh:ibs/Devel:YaST:Agama:Head/base//print_openqa.sh: line 56: syntax error near unexpected token `('
FAIL ibs/Devel:YaST:Agama:Head/base/: No ISO found in openqa request - is something wrong?
P:ibs/Devel:YaST:Agama:Head/base/ test_repo_consistent.sh
F-ibs/Devel:YaST:Agama:Head/s390x/ test_before_after_diff.sh:ibs/Devel:YaST:Agama:Head/s390x//print_openqa.sh: line 56: syntax error near unexpected token `('
F-ibs/Devel:YaST:Agama:Head/s390x/ test_iso_consistent.sh:ibs/Devel:YaST:Agama:Head/s390x//print_openqa.sh: line 56: syntax error near unexpected token `('
FAIL ibs/Devel:YaST:Agama:Head/s390x/: No ISO found in openqa request - is something wrong?
P:ibs/Devel:YaST:Agama:Head/s390x/ test_repo_consistent.sh
F-ibs/SUSE:ALP:Products:Marble:6.0 test_before_after_diff.sh:ibs/SUSE:ALP:Products:Marble:6.0/print_openqa.sh: line 62: syntax error near unexpected token `('
F-ibs/SUSE:ALP:Products:Marble:6.0 test_iso_consistent.sh:ibs/SUSE:ALP:Products:Marble:6.0/print_openqa.sh: line 62: syntax error near unexpected token `('
FAIL ibs/SUSE:ALP:Products:Marble:6.0: No ISO found in openqa request - is something wrong?
P:ibs/SUSE:ALP:Products:Marble:6.0 test_repo_consistent.sh
F-ibs/SUSE:ALP:Products:Marble:6.0:ToTest test_before_after_diff.sh:FAIL ibs/SUSE:ALP:Products:Marble:6.0:ToTest print_openqa : --- ibs/SUSE:ALP:Products:Marble:6.0:ToTest/print_openqa.before 2025-07-14 19:36:00.760323840 +0200
+++ ibs/SUSE:ALP:Products:Marble:6.0:ToTest/print_openqa.after  2025-07-15 13:54:28.977400609 +0200
@@ -4,9 +4,9 @@
  BUILD=44.3 \
  DISTRI=sle-micro \
  FLAVOR=Default-qcow-Updates \
- ISO=SL-Micro-6.0-aarch64-Build44.3.spdx.json \
  VERSION=6.0 \
  _DEPRIORITIZEBUILD=1
+" ISO=SL-Micro-6.0-aarch64-Build44.3.spdx.json \"

 /usr/bin/openqa-cli api -X post isos?async=1 \
  ARCH=x86_64 \

I reproduced this error on Leap 15.6 using a virtual env with python 3.6 same as 3.11 and also from a clean git checkout. So multiple errors and unexpected diff. Can you provide some pointers?

@andrii-suse
Copy link
Collaborator

That must be a locale problem, check json ci jobs in the gitlab project - they set locale infront

@andrii-suse
Copy link
Collaborator

Ah wait, there are real failures, I will have a look

@andrii-suse
Copy link
Collaborator

Ah wait, there are real failures, I will have a look

Ok, besides locale there must be some leftover folders. I don't know why git doesn't delete them, does it fail on a clean clone with the correct locale?

@okurz okurz force-pushed the fix/deprecated_code branch from 345cddc to eb41857 Compare July 15, 2025 12:54
@okurz
Copy link
Member Author

okurz commented Jul 16, 2025

Ah wait, there are real failures, I will have a look

Ok, besides locale there must be some leftover folders. I don't know why git doesn't delete them, does it fail on a clean clone with the correct locale?

Yes. I used

git clone [email protected]:os-autoinst/openqa-trigger-from-obs otfo2
LC_COLLATE=C make -C otfo2 test

Copy link
Member Author

@okurz okurz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@okurz okurz merged commit 49fa58e into os-autoinst:master Jul 16, 2025
5 checks passed
@okurz okurz deleted the fix/deprecated_code branch July 16, 2025 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants