Skip to content

Conversation

robhoes
Copy link
Member

@robhoes robhoes commented Mar 5, 2018

Signed-off-by: Rob Hoes rob.hoes@citrix.com

Signed-off-by: Rob Hoes <rob.hoes@citrix.com>
@coveralls
Copy link

coveralls commented Mar 5, 2018

Coverage Status

Coverage increased (+0.002%) to 18.641% when pulling 84dc8fb on robhoes:datamodel into 78a918d on xapi-project:master.

Copy link
Contributor

@edwintorok edwintorok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this affect only the generated docs?

field ~qualifier:StaticRO ~ty:domain_type ~lifecycle:[Published, rel_jura, ""] ~default_value:(Some (VEnum "unspecified")) "domain_type" "The type of domain that will be created when the VM is started";
field ~qualifier:StaticRO ~ty:domain_type
~lifecycle:[
Prototyped, rel_jura, "Internal-only field; not yet in the public API";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

in datamodel.ml this field is Published, this one is Prototyped. Is that intended?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did it this way, because the VM.domain_type field was part of the datamodel in jura, but it was marked as internal_only, and therefore not visible in the external API. Therefore, "published" in kolkata. The VM_metrics.current_domain_type was also includes in jura, but not internal_only, and I had therefore marked it as "published" in jura already (I wanted to also make this field internal, but somehow the datamodel didn't allow this).

However, even VM_metrics.current_domain_type wasn't actually useful in jura, because we never set it to anything other than unspecified. Perhaps it is better to call it "prototyped" as well for this reason.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Submitted a fixup for this.

@robhoes
Copy link
Member Author

robhoes commented Mar 6, 2018

Yes, this is just for the docs.

Copy link
Contributor

@edwintorok edwintorok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, LGTM

@robhoes robhoes merged commit 8bdaedd into xapi-project:master Mar 6, 2018
@robhoes robhoes deleted the datamodel branch March 6, 2018 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants