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

test/system: Ensure that $HOME is used as a fallback working directory #1153

Conversation

debarshiray
Copy link
Member

@debarshiray debarshiray commented Nov 9, 2022

This needs a directory that's going to be present on the host operating system across various configurations of all supported distributions, such as the hosts running the CI, but not inside the Toolbx containers.

It looks like /etc/kernel is present on both Debian and Fedora, but absent from the fedora-toolbox images. On a Debian 10 server, it's owned by several packages:

$ dpkg-query --search /etc/kernel
dkms, systemd, grub2-common, initramfs-tools, apt: /etc/kernel

... while on Fedora 36 Workstation:

$ rpm --file --query /etc/kernel
systemd-udev-250.8-1.fc36.x86_64

Currently, there's no way to get assert_line to use the stderr_lines array [1]. This is worked around by assigning stderr_lines to the lines array.

[1] bats-core/bats-assert#42

debarshiray added a commit to debarshiray/toolbox that referenced this pull request Nov 9, 2022
We need a directory that's going to be present on the host operating
system across various configurations of all supported distributions,
such as the hosts running the CI, but not inside the Toolbx containers.

It looks like /etc/kernel is present on both Debian and Fedora, but
absent from the fedora-toolbox images.  On a Debian 10 server, it's
owned by several packages:
  $ dpkg-query --search /etc/kernel
  dkms, systemd, grub2-common, initramfs-tools, apt: /etc/kernel

... while on Fedora 36 Workstation:
  $ rpm --file --query /etc/kernel
  systemd-udev-250.8-1.fc36.x86_64

containers#1153
@debarshiray debarshiray force-pushed the wip/rishi/test-system-run-ensure-home-as-fallback branch from 47ebb7a to 45b409f Compare November 9, 2022 07:37
@softwarefactory-project-zuul
Copy link

Build succeeded.

✔️ system-test-fedora-rawhide SUCCESS in 22m 21s
✔️ system-test-fedora-36 SUCCESS in 10m 07s
✔️ system-test-fedora-35 SUCCESS in 10m 16s

debarshiray added a commit to debarshiray/toolbox that referenced this pull request Nov 10, 2022
This needs a directory that's going to be present on the host operating
system across various configurations of all supported distributions,
such as the hosts running the CI, but not inside the Toolbx containers.

It looks like /etc/kernel is present on both Debian and Fedora, but
absent from the fedora-toolbox images.  On a Debian 10 server, it's
owned by several packages:
  $ dpkg-query --search /etc/kernel
  dkms, systemd, grub2-common, initramfs-tools, apt: /etc/kernel

... while on Fedora 36 Workstation:
  $ rpm --file --query /etc/kernel
  systemd-udev-250.8-1.fc36.x86_64

Currently, there's no way to get assert_line to use the stderr_lines
array [1].  This is worked around by assigning stderr_lines to the lines
array.

[1] bats-core/bats-assert#42

containers#1153
@debarshiray debarshiray force-pushed the wip/rishi/test-system-run-ensure-home-as-fallback branch from 45b409f to cc74311 Compare November 10, 2022 11:26
This needs a directory that's going to be present on the host operating
system across various configurations of all supported distributions,
such as the hosts running the CI, but not inside the Toolbx containers.

It looks like /etc/kernel is present on both Debian and Fedora, but
absent from the fedora-toolbox images.  On a Debian 10 server, it's
owned by several packages:
  $ dpkg-query --search /etc/kernel
  dkms, systemd, grub2-common, initramfs-tools, apt: /etc/kernel

... while on Fedora 36 Workstation:
  $ rpm --file --query /etc/kernel
  systemd-udev-250.8-1.fc36.x86_64

Currently, there's no way to get assert_line to use the stderr_lines
array [1].  This is worked around by assigning stderr_lines to the
'lines' array.

[1] bats-core/bats-assert#42

containers#1153
@debarshiray debarshiray force-pushed the wip/rishi/test-system-run-ensure-home-as-fallback branch from cc74311 to 741603c Compare November 10, 2022 11:28
@softwarefactory-project-zuul
Copy link

Build succeeded.

✔️ system-test-fedora-rawhide SUCCESS in 33m 13s
✔️ system-test-fedora-36 SUCCESS in 10m 24s
✔️ system-test-fedora-35 SUCCESS in 10m 44s

@debarshiray debarshiray merged commit 741603c into containers:main Nov 10, 2022
@debarshiray debarshiray deleted the wip/rishi/test-system-run-ensure-home-as-fallback branch November 10, 2022 14:08
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.

1 participant