Skip to content
This repository has been archived by the owner on Dec 13, 2022. It is now read-only.

fix(clapi): fix resource setparam export #6222

Merged
merged 1 commit into from
Jul 6, 2018
Merged

fix(clapi): fix resource setparam export #6222

merged 1 commit into from
Jul 6, 2018

Conversation

cgagnaire
Copy link

@cgagnaire cgagnaire commented Apr 19, 2018

When importing resource macro, CLAPI is expecting macro's id to be provided with setparam action.

Refs: #6221

@cgagnaire
Copy link
Author

This fix respects the way parameter setting has been designed for this type of object.
But in the case of an import in a already populated environment, this won't work as ids will be used by other resource macro definitions.

@chgautier
Copy link
Contributor

Fix OK on branch 6221-setparam-resource-export-fix. The RESOURCECFG set param action uses macro id upon export.

@kduret kduret merged commit 368218a into centreon:2.8.x Jul 6, 2018
kduret pushed a commit that referenced this pull request Jul 6, 2018
kduret pushed a commit that referenced this pull request Jul 6, 2018
loiclau added a commit that referenced this pull request Jul 11, 2018
@cgagnaire cgagnaire deleted the 6221-fix-resource-setparam-export branch July 26, 2018 12:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants