Bug 1450948

The script from [2] bug had some issues, here is improved version optimized to Satellite (that does not use qpid transactions so tpl can be ignored):


#!/usr/bin/env bash

# Qpid dir
QD=/var/lib/qpidd

# Qpid data dir
#QDD=${QD} ## for rhel6
QDD=${QD}/.qpidd ## for rhel7

# Qpid linearstore dir
QLSD=${QDD}/qls

# migrating dat dir with bdb
mv ${QLSD}/dat ${QLSD}/dat2

# preparing new structure
mkdir -p ${QLSD}/p001/efp/2048k/in_use
mkdir -p ${QLSD}/p001/efp/2048k/returned
#mkdir -p ${QLSD}/tpl2    ## not present in Satellite6
mkdir -p ${QLSD}/jrnl2

# movig queue's journal
for qd in ${QLSD}/jrnl/*; do
  bnqd=$(basename $qd)
  echo "migrating journal files of queue $bnqd"
  mkdir -p "${QLSD}/jrnl2/${bnqd}"
  # for each jrnl file in the queue's dir, move the raw file to "in_use" dir and create symlink from the queue's dir to the file
  for jrnlfile in $(ls $qd); do
    mv ${qd}/${jrnlfile} "${QLSD}/p001/efp/2048k/in_use/"
    ln -s "${QLSD}/p001/efp/2048k/in_use/${jrnlfile}" "${QLSD}/jrnl2/${bnqd}/${jrnlfile}"
  done
done

# moving transactions  ## not present in Satellite6
#if [[ -d ${QLSD}/tpl ]]; then
#  for tpl in ${QLSD}/tpl/*; do
#    echo $tpl
#    mv ${tpl} ${QLSD}/p001/efp/2084k/in_use/
#    ln -s "${QLSD}/p001/efp/in_use/$(basename ${tpl})" "${QLSD}/tpl2/$(basename ${tpl})"
#  done
#fi

echo "restoring file permissions/ownership/SELinux.."
# restore access permissions
chown -R qpidd:qpidd ${QD}
# restore SELinux context by current policy
restorecon -FR ${QD}

Moving this bug to POST for triage into Satellite 6 since the upstream issue http://projects.theforeman.org/issues/19929 has been resolved.

Moving this bug to POST for triage into Satellite 6 since the upstream issue http://projects.theforeman.org/issues/19929 has been resolved.

If we are using the latest qpid-dispatch in Sat 6.3, (0.8.0-10 or later), we will need to remove the session-max-unacked config option from the qpidd.conf
file.  As of this version, this setting is no longer required, and may in fact lead to problems.  Contact me if there are questions about this.

Mike C.

Thanks for the fix,

But How can QE verify this bug? Steps and expected behavior would be helpful.

(In reply to Jitendra Yejare from comment #7)
> Thanks for the fix,
> 
> But How can QE verify this bug? Steps and expected behavior would be helpful.

1) /etc/qpid/qpidd.conf has:

wcache-page-size=4

2) the config file does _not_ have session-max-unacked parameter (originally requested, but not further required - rather the parameter would cause qpidd wont run)

3) upgrade from Sat 6.1 (or some older version of 6.2 with qpid-cpp 0.30-9) to 6.3 does upgrade /var/lib/qpidd properly (the script has been already verified elsewhere, I expect)

Pavel,


(In reply to Pavel Moravec from comment #8)

> 1) /etc/qpid/qpidd.conf has:
> 
> wcache-page-size=4
I dont see this yet in qpidd.conf on an upgraded 6.3 satellite from 6.2
> 2) the config file does _not_ have session-max-unacked parameter (originally
> requested, but not further required - rather the parameter would cause qpidd
> wont run)
Yeah So I don't see this parameter as expected
> 3) upgrade from Sat 6.1 (or some older version of 6.2 with qpid-cpp 0.30-9)
> to 6.3 does upgrade /var/lib/qpidd properly (the script has been already
> verified elsewhere, I expect)
The upgrade from 6.2 to 6.3 snap 28 is successful.


So that said the first expected behavior is still not working. Shall I fail QA this bug ?

(In reply to Jitendra Yejare from comment #9)
> Pavel,
> 
> 
> (In reply to Pavel Moravec from comment #8)
> 
> > 1) /etc/qpid/qpidd.conf has:
> > 
> > wcache-page-size=4
> I dont see this yet in qpidd.conf on an upgraded 6.3 satellite from 6.2
> > 2) the config file does _not_ have session-max-unacked parameter (originally
> > requested, but not further required - rather the parameter would cause qpidd
> > wont run)
> Yeah So I don't see this parameter as expected
> > 3) upgrade from Sat 6.1 (or some older version of 6.2 with qpid-cpp 0.30-9)
> > to 6.3 does upgrade /var/lib/qpidd properly (the script has been already
> > verified elsewhere, I expect)
> The upgrade from 6.2 to 6.3 snap 28 is successful.
> 
> 
> So that said the first expected behavior is still not working. Shall I fail
> QA this bug ?

I think so, that parameter should be there (and it is really missing there - despite it has been pushed to upstream in https://github.com/theforeman/puppet-qpid/commit/5e3bb24cb1da38040548c4b59e9af12f69355c40 )

Failed QA !

@Satellite 6.3 snap 28

Thanks, Pavel,

As per comment 9 and comment 10, wcache-page-size=4 parameter is not added to /etc/qpid/qpidd.conf file.

Verified!

@ Satellite 6.3 snap 32

As per comment 8, observations are:

1. wcache-page-size is present qpidd.conf
# cat /etc/qpid/qpidd.conf | grep wcache
wcache-page-size=4

2. session-max-unacked parameter no more in qpidd conf:
# cat /etc/qpid/qpidd.conf | grep session-max-unacked
#

3. Upgrade from satellite 6.2 to 6.3 is successful.


Hence changing the state to verified.

Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA.
> > 
> > For information on the advisory, and where to find the updated files, follow the link below.
> > 
> > If the solution does not work for you, open a new bug report.
> > 
> > https://access.redhat.com/errata/RHSA-2018:0336