added relation field

[AlliumApotheosis]

https://hwjiraprd01.corp.emc.com/browse/MON-533

@benbp

[benbp]

👍

[larry-dean]

Is this going to have any values such as 'encloses' and 'enclosedBy'?

[benbp]

Right now we make no restrictions on what you add. It all depends on which logic is using it, so feel free to put a stake in the ground as far as conventions for enclosures.

[benbp]

It's unstructured json data that depends on the implementation, basically.

[anhou]

@VulpesArtificem As I know, @iceiilin is doing same or similar works on this. could you have a sync to avoid duplicated works?

[jfrey]

Is there a spec for this?

[iceiilin]

@jfrey There is a story tracking this https://hwjiraprd01.corp.emc.com/browse/MAG-176, without detailed spec. I have completed it and intended to open a PR after adding PDU obm settings into enclosure node.

For that story, I created a generate-enclosure task to create enclosure info, and it is added to sku discovery workflow.

I can open a PR first about enclosure node creation.

[jfrey]

Ok, we have some overlap here which is a concern. If @VulpesArtificem & @iceiilin can coordinate like @anhou suggested that would be great.

Suggesting we hold PR's from both of you until we are sure we have the right solution in place.

[benbp]

Yeah, as you said, looks like duplication. I suggest we just close this in favor of @iceiilin's PRs, but I would suggest keeping the defaultsTo: [] setting here. I'll add a comment to #14