Bug 1270740 – Clone declaration missing from cib2pcscmd result

Bugzilla will be upgraded to version 5.0 on a still to be determined date in the near future. The original upgrade date has been delayed.

Bug 1270740 - Clone declaration missing from cib2pcscmd result

Summary:	Clone declaration missing from cib2pcscmd result

Status:	CLOSED UPSTREAM

Aliases:	None

Product:	Red Hat Enterprise Linux 7
Classification:	Red Hat
Component:	clufter (Show other bugs)
Sub Component:
Version:	7.3
Hardware:	x86_64 Linux

Priority	unspecified Severity high
Target Milestone:	rc
Target Release:	---
Assigned To:	Jan Pokorný
QA Contact:	cluster-qe@redhat.com
Docs Contact:

URL:
Whiteboard:
Keywords:

Depends On:
Blocks:	1269964 1343661
	Show dependency tree / graph

Reported:	2015-10-12 05:41 EDT by Raoul Scarazzini
Modified:	2016-10-11 15:33 EDT (History)
CC List:	2 users (show)

See Also:
Fixed In Version:
Doc Type:	No Doc Update
Doc Text:	2pcscmd conversion now handles clone/master resources; likewise, node properties, utilization (per node or resource) and {op,rsc}_defaults are now covered when converting CIB content into sequence of pcs commands.
Story Points:	---
Clone Of:
Environment:
Last Closed:	2015-12-11 14:58:16 EST
Type:	Bug
Regression:	---
Mount Type:	---
Documentation:	---
CRM:
Verified Versions:
Category:	---
oVirt Team:	---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---

Attachments	(Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

Groups: None (edit)

Description Raoul Scarazzini 2015-10-12 05:41:23 EDT

Description of problem:

It seems that extracting the pcs command list from a given cib does not consider the meta data attributes.

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

clufter-cli-0.50.4-1.el7.noarch

How reproducible:

Always

Steps to Reproduce:
1. Export a cib with resources that have meta data, i.e.:

...
      <clone id="memcached-clone">
        <primitive class="systemd" id="memcached" type="memcached">
          <instance_attributes id="memcached-instance_attributes"/>
          <operations>
            <op id="memcached-start-timeout-60s" interval="0s" name="start" timeout="60s"/>
            <op id="memcached-monitor-interval-60s" interval="60s" name="monitor"/>
          </operations>
        </primitive>
        <meta_attributes id="memcached-clone-meta">
          <nvpair id="memcached-interleave" name="interleave" value="true"/>
        </meta_attributes>
      </clone>
...

2. Extract the pcs command list with clufter:

clufter cib2pcscmd --nocheck -i cib.xml > cib.pcs

Actual results:

pcs -f tmp-cib.xml resource create memcached systemd:memcached op start 'id=memcached-start-timeout-60s' 'interval=0s' 'name=start' 'timeout=60s' monitor 'id=memcached-monitor-interval-60s' 'interval=60s' 'name=
monitor'

Expected results:

pcs -f tmp-cib.xml resource create memcached systemd:memcached --clone interleave=true op start 'id=memcached-start-timeout-60s' 'interval=0s' 'name=start' 'timeout=60s' monitor 'id=memcached-monitor-interval-60s' 'interval=60s' 'name=monitor'

Additional info:

Comment 3 Jan Pokorný 2015-12-11 14:58:16 EST

Thanks for the report.

Should be fixed upstream as of
https://github.com/jnpkrn/clufter/commit/017d101bf2a4ab7d823544344f12e045781800e7

This is planned for new (soonish) release.

Comment 4 Jan Pokorný 2015-12-11 18:41:57 EST

Actually I planted (IMHO) better approach: explicit dedicated command:
https://pagure.io/clufter/e0e8b45dcf2b27bc7bf944e67238aeaaadcd505d

In means the referred case will stay as is modulo following new
final line:

> pcs resource clone memcached 'interleave=true'

Comment 5 Jan Pokorný 2015-12-11 18:45:42 EST

re [comment 4]:
unless '--tmp-cib ""' specified: s/pcs/pcs -f tmp-cib.xml/
(and it will be last but one line)

Comment 6 Jan Pokorný 2016-10-10 09:31:31 EDT

Note this is addressed in RHEL 6.8 through rebase ([bug 1269964])
+ in RHEL 7.3 also through rebase ([bug 1343661]).

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