Skip to content

Commit b782bcc

Browse files
Merge pull request #1233 from jistr/b/bootstrap-command-ignoring-failures
Don't ignore dnf upgrade failure in edpm_bootstrap_command
2 parents fd51796 + 5c23a0e commit b782bcc

2 files changed

Lines changed: 1 addition & 6 deletions

File tree

docs_user/modules/proc_adopting-compute-services-to-the-data-plane.adoc

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -530,17 +530,14 @@ ifeval::["{build}" != "downstream"]
530530
# https://issues.redhat.com/browse/RHOSZUUL-1517
531531
dnf -y install crypto-policies
532532
update-crypto-policies --set FIPS:NO-ENFORCE-EMS
533-
# FIXME: perform dnf upgrade for other packages in EDPM ansible
534-
# here we only ensuring that decontainerized libvirt can start
535533
./venv/bin/repo-setup current-podified -b antelope -d centos9 --stream
536534
dnf -y upgrade openstack-selinux
537535
rm -f /run/virtlogd.pid
538536
rm -rf repo-setup-main
539537
endif::[]
540538
ifeval::["{build}" == "downstream"]
541539
edpm_bootstrap_command: |
542-
# FIXME: perform dnf upgrade for other packages in EDPM ansible
543-
# here we only ensuring that decontainerized libvirt can start
540+
set -euxo pipefail
544541
dnf -y upgrade openstack-selinux
545542
rm -f /run/virtlogd.pid
546543
endif::[]

tests/roles/dataplane_adoption/defaults/main.yaml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -121,8 +121,6 @@ edpm_bootstrap_command: |
121121
# Exclude ceph-common-18.2.7 as it's pulling newer openssl not compatible
122122
# with rhel 9.2 openssh
123123
dnf config-manager --setopt centos9-storage.exclude="ceph-common-18.2.7" --save
124-
# FIXME: perform dnf upgrade for other packages in EDPM ansible
125-
# here we only ensuring that decontainerized libvirt can start
126124
dnf -y upgrade openstack-selinux
127125
rm -f /run/virtlogd.pid
128126
{%+ endif +%}

0 commit comments

Comments
 (0)