Browse Source

remove ownership change of ironic-dbsync.log

Remove workaround https://review.openstack.org/#/c/457478/
we don't need it any more in Rocky since the puppet fix was backported
to Newton and the workaround here has a comment that it can be removed
after the Pike release.

This workaround is causing an undercloud upgrade issue where the
existing logs were removed before upgrade.

Change-Id: Idb428e1a7a0cb37d9e7e2cbaa4c9a26efaf2bb34
Closes-bug: #1754658
(cherry picked from commit 410b9ceb02)
changes/39/737639/1
Radoslaw Smigielski 2 years ago
committed by Steve Baker
parent
commit
77955bdc85
1 changed files with 0 additions and 8 deletions
  1. +0
    -8
      instack_undercloud/undercloud.py

+ 0
- 8
instack_undercloud/undercloud.py View File

@@ -2435,14 +2435,6 @@ def install(instack_root, upgrade=False):
_generate_init_data(instack_env)
ovs_interfaces = _get_ovs_interfaces()
if upgrade:
# Even if we backport https://review.openstack.org/#/c/457478/
# into stable branches of puppet-ironic, we still need a way
# to handle existing deployments.
# This task will fix ironic-dbsync.log ownership on existing
# deployments during an upgrade. It can be removed after we
# release Pike.
_run_command(['sudo', '/usr/bin/chown', 'ironic:ironic',
'/var/log/ironic/ironic-dbsync.log'])
_die_tuskar_die()
if CONF.undercloud_update_packages:
_run_yum_clean_all(instack_env)


Loading…
Cancel
Save