{"comment":{"url":"https://api.github.com/repos/joyent/rfd/comments/27974040","html_url":"https://github.com/joyent/rfd/commit/105881d7d608baa558b3ea005070c51abc3245e9#commitcomment-27974040","id":27974040,"user":{"login":"mgerdts","id":8992389,"avatar_url":"https://avatars3.githubusercontent.com/u/8992389?v=4","gravatar_id":"","url":"https://api.github.com/users/mgerdts","html_url":"https://github.com/mgerdts","followers_url":"https://api.github.com/users/mgerdts/followers","following_url":"https://api.github.com/users/mgerdts/following{/other_user}","gists_url":"https://api.github.com/users/mgerdts/gists{/gist_id}","starred_url":"https://api.github.com/users/mgerdts/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mgerdts/subscriptions","organizations_url":"https://api.github.com/users/mgerdts/orgs","repos_url":"https://api.github.com/users/mgerdts/repos","events_url":"https://api.github.com/users/mgerdts/events{/privacy}","received_events_url":"https://api.github.com/users/mgerdts/received_events","type":"User","site_admin":false},"position":391,"line":291,"path":"rfd/0121/README.md","commit_id":"105881d7d608baa558b3ea005070c51abc3245e9","created_at":"2018-03-07T14:27:31Z","updated_at":"2018-03-07T14:27:31Z","author_association":"CONTRIBUTOR","body":"OK, thanks.   I need to spend some time with the `vmadm(1m)` man page and related code to figure out the mapping between the various properties that can be set (in json) to be sure that they fall into one of two categories:\r\n\r\n1. Standard properties that are explicitly listed above aside from `property` custom properties.  The standard properties should all be handled by the zones framework.\r\n2. The custom properties, specified by `property` properties.  These are for use by people or automation that is not part of the zones framework. For brands that are part of illumos, these properties should be completely ignored by the brand hooks (i.e. `/usr/lib/brand/<brand>/*`).  In the case of SmartOS, custom properties should be used only by things like `vmadm`, `vmadmd`, and the metadata server.\r\n\r\nThat will ensure that we didn't miss any needed standard properties and help point to how `proptable.js` needs to change.  I'll probably remove the table above in favor of one table that contains the complete mapping."}}
{"actor":{"display_login":"mgerdts"}}