Skip to content

2023.1: zed merge #1230

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

Merged
merged 20 commits into from
Aug 16, 2024
Merged

2023.1: zed merge #1230

merged 20 commits into from
Aug 16, 2024

Conversation

markgoddard
Copy link
Contributor

  • Stop changing permissions on files on Rocky 9
  • Stop changing permissions on files (Stop changing permissions on files #1119)
  • CI: Allow logging of Rally/Tempest
  • Fix CVE-2024-40767
  • CI: Bump AIO root volume size to 40GB
  • Build nova from our fork
  • Prevent hanging before reboot on systems running molly-guard
  • Add reboot timeout to reboot playbook
  • CIS: Remove always tag from include_role tasks

markgoddard and others added 19 commits July 23, 2024 09:03
A similar change was made for Ubuntu systems in #1119, but it did not
apply to Rocky 9 systems. This changes brings the two into line.

(cherry picked from commit ef96aa2)
These are causing changes to docker overlay filesystems with
possible unintended consequences. It is also really slow to loop
through so many files in ansible.

(cherry picked from commit 0d1dfe2)
By default the 'Run tempest' task has no_log set to avoid revealing
sensitive data. This does not apply in CI, and makes it difficult to
debug failures.

(cherry picked from commit 8384dc4)
CI: Allow logging of Rally/Tempest
Fixes CVE-2024-40767 [1] with updated container images for Nova
services.

[1] https://security.openstack.org/ossa/OSSA-2024-002.html
CI: Bump AIO root volume size to 40GB
This is necessary to address OSSA-2024-002 [1] until patches are merged
upstream.

[1] https://security.openstack.org/ossa/OSSA-2024-002.html
molly-guard can be used to prevent accidental reboots, prompting the
user to input the system's hostname before allowing a reboot. This does
not work well with automation, however.

This change adds the internal reboot executable within molly-guard to
the search path to avoid this issue.
yoga: Fix reboot hang on systems with molly-guard, add timeout variable
yoga: Stop changing permissions on files in CIS
If we have the CIS hardening hook enabled and run a command such as the
following:

  kayobe overcloud host configure -t foo

where 'cis' is not in the specified tags, we see the following error:

  PLAY [Security hardening] *****************************************
  TASK [include_role : ansible-lockdown.rhel9_cis] ******************
  fatal: [controller-01]: FAILED! =>
    msg: |-
      The conditional check 'ansible_facts.os_family == 'RedHat' and
      ansible_facts.distribution_major_version == '9'' failed. The error
      was: error while evaluating conditional (ansible_facts.os_family
      == 'RedHat' and ansible_facts.distribution_major_version == '9'):
      'dict object' has no attribute 'os_family'. 'dict object' has no
      attribute 'os_family'

      The error appears to be in 'etc/kayobe/ansible/cis.yml': line 35,
      column 7, but may be elsewhere in the file depending on the exact
      syntax problem.

      The offending line appears to be:

          - include_role:
            ^ here

This is because the include_role task has the 'always' tag, so runs
despite no facts having been gathered.

The always tag is not required for this task - specifying the 'cis' tag
causes the role to be included. This change fixes the issue by removing
the always tags from these tasks.
CIS: Remove always tag from include_role tasks
@markgoddard markgoddard requested a review from a team as a code owner August 16, 2024 13:58
@markgoddard markgoddard self-assigned this Aug 16, 2024
@markgoddard markgoddard merged commit f30b309 into stackhpc/2023.1 Aug 16, 2024
12 checks passed
@markgoddard markgoddard deleted the 2023.1-zed-merge branch August 16, 2024 16:07
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.

5 participants