Skip to content

Conversation

@richm
Copy link
Contributor

@richm richm commented Apr 8, 2025

There is a new QEMU based test which uses the qemu/kvm capability of
github action runners. This is the basis for new bootc/image mode tests
which we will be rolling out in the near future.

ansible-lint requires that the collection path is set so that the requirements
it installs are installed in the correct place.

There has been some general github action deprecation of python versions and
ubuntu versions that we have had to fix.

Remove CONTRIBUTOR from the list of users who can trigger citest.

For more information, see

@richm richm requested a review from Jakuje as a code owner April 8, 2025 18:13
@richm richm self-assigned this Apr 8, 2025
@richm
Copy link
Contributor Author

richm commented Apr 9, 2025

@Jakuje re: f42 failures - I guess there are some new options or removed options in f42?

@Jakuje
Copy link
Collaborator

Jakuje commented Apr 9, 2025

@Jakuje re: f42 failures - I guess there are some new options or removed options in f42?

Is there way to get some logs from that or output? I do not see in our CI any issues on Fedora 42.

@richm
Copy link
Contributor Author

richm commented Apr 9, 2025

@Jakuje re: f42 failures - I guess there are some new options or removed options in f42?

Is there way to get some logs from that or output? I do not see in our CI any issues on Fedora 42.

https://github.com/linux-system-roles/ssh/actions/runs/14340666754/job/40198837238?pr=177

The failing test is FAIL: tests_all_options.yml

the failure is

  TASK [Reinstall manual pages for openssh-clients on RHEL] **********************
  task path: /home/runner/work/ssh/ssh/tests/tests_all_options.yml:49
  Tuesday 08 April 2025  19:34:24 +0000 (0:00:00.491)       0:00:02.261 ********* 
  fatal: [/home/runner/.cache/linux-system-roles/fedora-42.qcow2]: FAILED! => {
      "changed": true,
      "cmd": [
          "dnf",
          "reinstall",
          "-y",
          "openssh-clients"
      ],
      "delta": "0:00:00.531813",
      "end": "2025-04-08 19:34:25.576412",
      "rc": 1,
      "start": "2025-04-08 19:34:25.044599"
  }
  
  STDERR:
  
  Updating and loading repositories:
  Repositories loaded.
  Failed to resolve the transaction:
  Installed packages for argument 'openssh-clients' are not available in repositories in the same version, available versions: openssh-clients-9.9p1-10.fc42.x86_64, cannot reinstall.
  
  
  MSG:
  
  non-zero return code

@richm richm force-pushed the changes-20250408 branch from 0a128c4 to 6952154 Compare April 9, 2025 15:18
@Jakuje
Copy link
Collaborator

Jakuje commented Apr 9, 2025

Sounds like some repository issue, It should be safe to skip this test on QEMU. This is just to install manual pages in minimized container images. We have some logic to run something only on containers in sshd, but I do not think we have it here.

Copy link
Contributor

@martinpitt martinpitt left a comment

Choose a reason for hiding this comment

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

Green now with the latest update, the previous failure sounds and awful lot like the ansible 2.16 dnf5 incompatibility.

@richm richm force-pushed the changes-20250408 branch 3 times, most recently from 1d332be to e6779df Compare April 11, 2025 00:09
…rsions, ubuntu versions

There is a new QEMU based test which uses the qemu/kvm capability of
github action runners.  This is the basis for new bootc/image mode tests
which we will be rolling out in the near future.

ansible-lint requires that the collection path is set so that the requirements
it installs are installed in the correct place.

There has been some general github action deprecation of python versions and
ubuntu versions that we have had to fix.

Remove `CONTRIBUTOR` from the list of users who can trigger citest.

For more information, see

* linux-system-roles/.github#98
* linux-system-roles/.github#94
* linux-system-roles/.github#93
* linux-system-roles/.github#92
* linux-system-roles/.github#91

Signed-off-by: Rich Megginson <rmeggins@redhat.com>
@richm richm force-pushed the changes-20250408 branch from e6779df to dc60975 Compare April 11, 2025 14:10
@richm richm merged commit 33d77d2 into main Apr 11, 2025
30 checks passed
@richm richm deleted the changes-20250408 branch April 11, 2025 19:13
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.

4 participants