[PUP-8597] Modify rich data serialization to use shorter tags instead of __pcore_type__ Created: 2018/03/23 Updated: 2018/09/19 Resolved: 2018/09/17
|Fix Version/s:||PUP 6.0.0|
|Remaining Estimate:||Not Specified|
|Time Spent:||Not Specified|
|Original Estimate:||Not Specified|
|Epic Link:||6.y Rich Data|
|Sprint:||Platform Core KANBAN|
|Release Notes Summary:||The earlier experimental --rich_data format used the tags __pcore_type__ and __pcore_value__, these are now shortened to __ptype and __pvalue respectively. If you are using this experimental feature and have stored serializations you need to change them or write them again with the updated version.|
|QA Risk Assessment:||Needs Assessment|
Serialization of rich data in JSON/YAML is currently using the tags:
Those are longer than they need to be for making them unique/non-clashing with normal attributes.
We should shorten those to:
When writing, but continue to support the longer tags when reading.
|Comment by Rick Sherman [ 2018/03/28 ]|
cc Davin Hanlon
|Comment by Henrik Lindberg [ 2018/03/28 ]|
Rick Sherman That code is not affected, that is the Pcore API - the change is just for the human readable serialization format where the tags before the change were _pcore_type and pcore_value_ - i.e. not the _pcore_type method on an object that returns its type.
You would only have to change if you are reading or writing serializations of puppet objects using the human readable JSON serializer. Do you?
|Comment by Thomas Hallgren [ 2018/05/28 ]|
Merged to master at e3723a0.
|Comment by Josh Cooper [ 2018/07/31 ]|
|Comment by Josh Cooper [ 2018/08/03 ]|
|Comment by Davin Hanlon [ 2018/08/06 ]|
Josh Cooper - from Rick Sherman's earlier comment it looks like it will. However, the YANG modules are not released, so we can make changes before we release them to account for this. Does that seem OK Rick Sherman Thomas Honey?
|Comment by Rick Sherman [ 2018/08/06 ]|
We should be okay. Those modules are still unreleased and we can update them if needed.
|Comment by Josh Cooper [ 2018/08/06 ]|
Thanks all. Is there a specification for the serialization format?
|Comment by Thomas Hallgren [ 2018/08/16 ]|
|Comment by Scott Garman [ 2018/09/17 ]|
Since we had a green run of CI puppet-agent 6.0.0 last night, I'm bulk-changing these issues from Ready for CI -> Resolved in preparation of the release.