Red Hat Satellite engineering is moving the tracking of its product development work on Satellite to Red Hat Jira (issues.redhat.com). If you're a Red Hat customer, please continue to file support cases via the Red Hat customer portal. If you're not, please head to the "Satellite project" in Red Hat Jira and file new tickets here. Individual Bugzilla bugs will be migrated starting at the end of May. If you cannot log in to RH Jira, please consult article #7032570. That failing, please send an e-mail to the RH Jira admins at rh-issues@redhat.com to troubleshoot your issue as a user management inquiry. The email creates a ServiceNow ticket with Red Hat. Individual Bugzilla bugs that are migrated will be moved to status "CLOSED", resolution "MIGRATED", and set with "MigratedToJIRA" in "Keywords". The link to the successor Jira issue will be found under "Links", have a little "two-footprint" icon next to it, and direct you to the "Satellite project" in Red Hat Jira (issue links are of type "https://issues.redhat.com/browse/SAT-XXXX", where "X" is a digit). This same link will be available in a blue banner at the top of the page informing you that that bug has been migrated.
Bug 1450948 - Required updates to installer / upgrade wrt. qpid-cpp upgrade to 0.34
Summary: Required updates to installer / upgrade wrt. qpid-cpp upgrade to 0.34
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Installation
Version: 6.3.0
Hardware: x86_64
OS: Linux
urgent
high
Target Milestone: Unspecified
Assignee: Chris Roberts
QA Contact: Jitendra Yejare
URL: http://projects.theforeman.org/issues...
Whiteboard:
Depends On:
Blocks: 1367735 1463800 1468450
TreeView+ depends on / blocked
 
Reported: 2017-05-15 12:49 UTC by Pavel Moravec
Modified: 2019-04-01 20:27 UTC (History)
11 users (show)

Fixed In Version: tfm-rubygem-katello-3.4.4,katello-installer-base-3.4.5.18-1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1468450 (view as bug list)
Environment:
Last Closed: 2018-02-21 16:54:37 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Foreman Issue Tracker 19929 0 None None None 2017-06-05 17:03:51 UTC

Description Pavel Moravec 2017-05-15 12:49:15 UTC
Description of problem:
Due to bz1367735, qpid-cpp-* packages will be updated from 0.30 to 0.34 in Satellite 6.3. That brings several recommended or mandatory updates to installer / upgrade procedure.

While the original BZ is filed against qpid component, let track here the installer bits. If you see this as a duplicate, close this BZ as dup of 1367735.


Version-Release number of selected component (if applicable):
Satellite 6.3


How reproducible:
n.a.


Steps to Reproduce / What is required:
1) add to /etc/qpid/qpidd.conf:

session-max-unacked=10
wcache-page-size=4

to the installer by default

2) the upgrade needs an automated script (working both on RHEL6 and RHEL7 with _different_ path to linear store) for the linearstore directory structure update per [1]/[2]

[1] https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_MRG/3/html-single/Messaging_Installation_and_Configuration_Guide/index.html#Upgrade_a_MRG_Messaging_3.1_Server_to_MRG_Messaging_3.2
[2] https://bugzilla.redhat.com/show_bug.cgi?id=1263141

Comment 2 Pavel Moravec 2017-06-06 18:32:46 UTC
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}

Comment 3 Satellite Program 2017-06-21 20:18:14 UTC
Moving this bug to POST for triage into Satellite 6 since the upstream issue http://projects.theforeman.org/issues/19929 has been resolved.

Comment 5 Satellite Program 2017-08-03 22:20:52 UTC
Moving this bug to POST for triage into Satellite 6 since the upstream issue http://projects.theforeman.org/issues/19929 has been resolved.

Comment 6 Mike Cressman 2017-08-24 13:26:26 UTC
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.

Comment 7 Jitendra Yejare 2017-12-11 11:37:19 UTC
Thanks for the fix,

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

Comment 8 Pavel Moravec 2017-12-12 07:11:22 UTC
(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)

Comment 9 Jitendra Yejare 2017-12-12 09:29:06 UTC
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 ?

Comment 10 Pavel Moravec 2017-12-12 10:43:22 UTC
(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 )

Comment 11 Jitendra Yejare 2017-12-12 11:16:54 UTC
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.

Comment 13 Jitendra Yejare 2018-01-15 10:38:58 UTC
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.

Comment 14 Satellite Program 2018-02-21 16:54:37 UTC
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


Note You need to log in before you can comment on or make changes to this bug.