Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

CP-4273: Add Hotfix_apply licensing feature #1060

Merged
merged 1 commit into from Mar 19, 2013

Conversation

Projects
None yet
4 participants
Contributor

mcclurmc commented Mar 5, 2013

This must go into Xapi prior to the corresponding v6d commit.

Signed-off-by: Mike McClurg mike.mcclurg@citrix.com

@johnelse johnelse and 2 others commented on an outdated diff Mar 12, 2013

ocaml/xapi/features.ml
@@ -17,6 +17,8 @@ open D
(* Features and restrictions *)
type feature =
+ | Hotfix_appy
@johnelse

johnelse Mar 12, 2013

Collaborator

Ahem :)

@robhoes

robhoes Mar 12, 2013

Owner

Should that be Hotfix_happy? :)

@mcclurmc

mcclurmc Mar 14, 2013

Contributor

No Rob, hotfixes are /never/ happy :(

Contributor

mcclurmc commented Mar 14, 2013

Besides my boneheaded spelling mistake, we've also decided not to add the RPU feature flag. So I'm going to rebase this branch instead of adding a fixing commit on top.

Mike McClurg CP-4273: Add Hotfix_apply licensing feature
Signed-off-by: Mike McClurg <mike.mcclurg@citrix.com>
ff5a4de
Owner

jonludlam commented Mar 18, 2013

Assigning to @robhoes for review

Owner

robhoes commented Mar 19, 2013

Looks fine now.

@robhoes robhoes added a commit that referenced this pull request Mar 19, 2013

@robhoes robhoes Merge pull request #1060 from mcclurmc/cp4273-hotfix-apply-feature
CP-4273: Add Hotfix_apply licensing feature
60eea31

@robhoes robhoes merged commit 60eea31 into xapi-project:master Mar 19, 2013

1 check passed

default Merged build finished.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment