Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HACK: try to fetch corrupt user journal files #17837

Closed

Conversation

jelly
Copy link
Member

@jelly jelly commented Oct 24, 2022

@jelly jelly added the no-test For doc/workflow changes, or experiments which don't need a full CI run, label Oct 24, 2022
@jelly jelly changed the title hack: copy journal HACK: try to fetch corrupt user journal files Oct 24, 2022
@jelly jelly temporarily deployed to cockpit-dist October 24, 2022 15:44 Inactive
@jelly jelly force-pushed the tf-debug-systemd-user-journal branch from 1018d3c to 40bed66 Compare October 26, 2022 07:53
@jelly jelly temporarily deployed to cockpit-dist October 26, 2022 07:58 Inactive
@jelly jelly force-pushed the tf-debug-systemd-user-journal branch from 40bed66 to 9dfc61d Compare October 26, 2022 15:53
@jelly jelly temporarily deployed to cockpit-dist October 26, 2022 15:57 Inactive
@jelly jelly force-pushed the tf-debug-systemd-user-journal branch from 9dfc61d to 26d0448 Compare October 27, 2022 07:31
@jelly jelly temporarily deployed to cockpit-dist October 27, 2022 07:38 Inactive
@jelly jelly force-pushed the tf-debug-systemd-user-journal branch from 26d0448 to 75c2ff8 Compare October 27, 2022 10:27
@jelly jelly temporarily deployed to cockpit-dist October 27, 2022 10:35 Inactive
@jelly jelly force-pushed the tf-debug-systemd-user-journal branch from 75c2ff8 to ede2158 Compare October 27, 2022 13:57
@jelly jelly temporarily deployed to cockpit-dist October 27, 2022 14:03 Inactive
@jelly jelly force-pushed the tf-debug-systemd-user-journal branch from ede2158 to 3103232 Compare October 27, 2022 14:55
@jelly jelly temporarily deployed to cockpit-dist October 27, 2022 15:01 Inactive
@jelly jelly force-pushed the tf-debug-systemd-user-journal branch from 3103232 to e88f63f Compare October 28, 2022 13:33
@jelly jelly temporarily deployed to cockpit-dist October 28, 2022 13:39 Inactive
@jelly
Copy link
Member Author

jelly commented Oct 28, 2022

@martinpitt Hey, I'm trying to download_dir /var/log/journal but this doesn't seem to succeed, do you have any idea why it wouldn't work on packit?

Locally I get:

receiving incremental file list
created directory /home/jelle/projects/cockpit/tf-debug-systemd-user-journal/test/TestSystemInfo-testMotd-fedora-36-127.0.0.2-2201-FAIL
journal/
journal/cc8b84b1dda84fd18876e2aff2c3ffd1/
journal/cc8b84b1dda84fd18876e2aff2c3ffd1/system.journal
> warning: Resolving coreutils failed: {"detail":"cannot update repo 'updates': Cannot prepare internal mirrorlist: Curl error (6): Couldn't resolve host name for https://mirrors.fedoraproject
sent 74 bytes  received 16,781,571 bytes  33,563,290.00 bytes/sec
total size is 16,777,216  speedup is 1.00
Failed to generate debug artifact:
Traceback (most recent call last):
  File "/home/jelle/projects/cockpit/tf-debug-systemd-user-journal/test/common/testlib.py", line 1205, in failureHandler
    test_case.copy_journal("FAIL")
  File "/home/jelle/projects/cockpit/tf-debug-systemd-user-journal/test/common/testlib.py", line 1900, in copy_journal
    print("files", os.listdir(directory))
FileNotFoundError: [Errno 2] No such file or directory: 'TestSystemInfo-testMotd-fedora-36-127.0.0.2-2201-FAIL'

@martinpitt
Copy link
Member

@jelly : download() and download_dir() are a bit "strange" historically, they download relative destination paths into test/. So either adjust your listdir call accordingly, or set relative_dir='.` or specify an absolute destination dir.

@martinpitt
Copy link
Member

@jelly btw, there's no difference -- your local output and the one on testing farm fail on exactly the same issue.

@jelly jelly force-pushed the tf-debug-systemd-user-journal branch from e88f63f to 8026a4b Compare October 31, 2022 09:49
@jelly jelly temporarily deployed to cockpit-dist October 31, 2022 09:55 Inactive
@jelly jelly force-pushed the tf-debug-systemd-user-journal branch from 8026a4b to 07fb88f Compare November 1, 2022 10:56
@jelly jelly temporarily deployed to cockpit-dist November 1, 2022 11:03 Inactive
@jelly jelly force-pushed the tf-debug-systemd-user-journal branch from 07fb88f to 94fde72 Compare November 1, 2022 12:14
@jelly jelly temporarily deployed to cockpit-dist November 1, 2022 12:20 Inactive
@jelly jelly force-pushed the tf-debug-systemd-user-journal branch from 94fde72 to 2a82d92 Compare November 1, 2022 14:45
@jelly jelly temporarily deployed to cockpit-dist November 1, 2022 14:50 Inactive
@jelly
Copy link
Member Author

jelly commented Nov 25, 2022

I don't recall seeing this in a while so I'll close this PR.

@jelly jelly closed this Nov 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-test For doc/workflow changes, or experiments which don't need a full CI run,
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants