Bug 1230908

Summary:	pcs cluster cib-push doesn't allow to utilize XPath filter provided by underlying cibadmin
Product:	Red Hat Enterprise Linux 7	Reporter:	Jan Pokorný [poki] <jpokorny>
Component:	pcs	Assignee:	Chris Feist <cfeist>
Status:	CLOSED WONTFIX	QA Contact:	cluster-qe <cluster-qe>
Severity:	unspecified	Docs Contact:
Priority:	low
Version:	7.2	CC:	cfeist, cluster-maint, cluster-qe, tojeline
Target Milestone:	rc
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:	1230368	Environment:
Last Closed:	2016-10-21 13:02:01 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:	1230368
Bug Blocks:	1212909

Description Jan Pokorný [poki] 2015-06-11 18:23:59 UTC

Split from original bug as this is rather orhogonal:

+++ This bug was initially created as a clone of Bug #1230368 +++

Note that this situation (EDIT: see [bug 1230368 comment 0]) was
created just by pcs commands:

# pcs cluster setup --start ...
# pcs -f cib.xml ...; pcs -f cib.xml ...; ...
# pcs cluster cib-push cib.xml --config

so a user's carelessness is not to be blamed here

Note that underlying cibadmin command does support also --xpath option
and it would be helpful:

1/ add support for xpath passthrough (e.g., if scope starts with
   a slash, treat it as an XPath expression)

2/ --config option was made an alias to
>  scope=/cib/config/*[name() != 'nodes']
   so the nodes section is not included (hence overwritten)

Comment 2 Jan Pokorný [poki] 2015-06-24 21:36:09 UTC

This bug shall be resolved so as to make clufter "pcs commands"
output useful when batching the changes is enabled (default).
Hence a blocker for standalone clufter reintroduction: [bug 1212909].

Comment 6 Tomas Jelinek 2016-10-21 13:02:01 UTC

The original issue has been resolved as described in bz1230368.

We cannot really implement what is requested in part 2:
> 2/ --config option was made an alias to
> >  scope=/cib/config/*[name() != 'nodes']
>    so the nodes section is not included (hence overwritten)
If we make it to work this way, it would not be possible to update node attributes and utilization.

The only thing which is to be done is support for xpath in the "pcs cluster cib-push" command. This is rather advanced feature as it requires detailed understanding of CIB structure and pacemaker. Moreover it appears to be no longer demanded.

Closing this as wontfix. We can reopen if there is a demand for this feature in a future.