)]}'
{"id":"openstack%2Fnova-specs~622893","triplet_id":"openstack%2Fnova-specs~master~Ie57414b232126d29f484e104931cacdb97a0d7a5","project":"openstack/nova-specs","branch":"master","topic":"bp/virtual-persistent-memory","hashtags":[],"change_id":"Ie57414b232126d29f484e104931cacdb97a0d7a5","subject":"Virtual persistent memory libvirt driver implementation","status":"ABANDONED","created":"2018-12-05 08:38:53.000000000","updated":"2019-05-27 05:56:21.000000000","total_comment_count":31,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"af29346e919cc742852824f0ce9ff9038c728503","_number":622893,"virtual_id_number":622893,"owner":{"_account_id":11347,"name":"Rui Zang","email":"rui.zang@yandex.com","username":"rzang"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":23360,"name":"Eric Lei","email":"leiyashuai@inspur.com","username":"leiyashuai"},{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},{"date":"2019-05-23 08:45:22.000000000","_account_id":11347,"name":"Rui Zang","email":"rui.zang@yandex.com","username":"rzang"},{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},{"_account_id":22378,"name":"Kaiyuan Qi","email":"qiky@inspur.com","username":"qiky"},{"value":1,"date":"2019-05-20 09:07:26.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":17261,"name":"guochunting","email":"guochunting@inspur.com","username":"friendgct"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"disliked":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":23360,"name":"Eric Lei","email":"leiyashuai@inspur.com","username":"leiyashuai"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":11347,"name":"Rui Zang","email":"rui.zang@yandex.com","username":"rzang"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":-1,"date":"2019-05-20 18:37:44.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":-1,"date":"2019-05-21 06:54:39.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22378,"name":"Kaiyuan Qi","email":"qiky@inspur.com","username":"qiky"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":17261,"name":"guochunting","email":"guochunting@inspur.com","username":"friendgct"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":23360,"name":"Eric Lei","email":"leiyashuai@inspur.com","username":"leiyashuai"},{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":11347,"name":"Rui Zang","email":"rui.zang@yandex.com","username":"rzang"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},{"_account_id":22378,"name":"Kaiyuan Qi","email":"qiky@inspur.com","username":"qiky"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":17261,"name":"guochunting","email":"guochunting@inspur.com","username":"friendgct"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":23360,"name":"Eric Lei","email":"leiyashuai@inspur.com","username":"leiyashuai"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":11347,"name":"Rui Zang","email":"rui.zang@yandex.com","username":"rzang"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":22378,"name":"Kaiyuan Qi","email":"qiky@inspur.com","username":"qiky"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":17261,"name":"guochunting","email":"guochunting@inspur.com","username":"friendgct"}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},{"_account_id":11347,"name":"Rui Zang","email":"rui.zang@yandex.com","username":"rzang"},{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"_account_id":17261,"name":"guochunting","email":"guochunting@inspur.com","username":"friendgct"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":22378,"name":"Kaiyuan Qi","email":"qiky@inspur.com","username":"qiky"},{"_account_id":23360,"name":"Eric Lei","email":"leiyashuai@inspur.com","username":"leiyashuai"},{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2018-12-21 15:29:12.000000000","updated_by":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"reviewer":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"state":"REVIEWER"},{"updated":"2019-01-07 06:38:01.000000000","updated_by":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"reviewer":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"state":"REVIEWER"},{"updated":"2019-02-22 00:36:07.000000000","updated_by":{"_account_id":17261,"name":"guochunting","email":"guochunting@inspur.com","username":"friendgct"},"reviewer":{"_account_id":17261,"name":"guochunting","email":"guochunting@inspur.com","username":"friendgct"},"state":"REVIEWER"},{"updated":"2019-02-24 03:54:42.000000000","updated_by":{"_account_id":22378,"name":"Kaiyuan Qi","email":"qiky@inspur.com","username":"qiky"},"reviewer":{"_account_id":22378,"name":"Kaiyuan Qi","email":"qiky@inspur.com","username":"qiky"},"state":"REVIEWER"},{"updated":"2019-02-26 08:31:33.000000000","updated_by":{"_account_id":23360,"name":"Eric Lei","email":"leiyashuai@inspur.com","username":"leiyashuai"},"reviewer":{"_account_id":23360,"name":"Eric Lei","email":"leiyashuai@inspur.com","username":"leiyashuai"},"state":"REVIEWER"},{"updated":"2019-04-09 16:30:30.000000000","updated_by":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"reviewer":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"state":"REVIEWER"},{"updated":"2019-04-18 05:38:22.000000000","updated_by":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"reviewer":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"state":"REVIEWER"},{"updated":"2019-05-20 09:07:26.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2019-05-20 18:37:44.000000000","updated_by":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"reviewer":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"state":"REVIEWER"},{"updated":"2019-05-21 06:54:39.000000000","updated_by":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"reviewer":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"state":"REVIEWER"}],"messages":[{"id":"8c8819a490c2be5143236a366aefad2e69271f78","author":{"_account_id":11347,"name":"Rui Zang","email":"rui.zang@yandex.com","username":"rzang"},"date":"2018-12-05 08:38:53.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"355b0de3d2978e6c586f72387540e6e9d882ac10","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-12-05 08:54:53.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs http://logs.openstack.org/93/622893/1/check/openstack-tox-docs/2884c45/html/ : SUCCESS in 6m 26s\n- openstack-tox-pep8 http://logs.openstack.org/93/622893/1/check/openstack-tox-pep8/07ce0b6/ : SUCCESS in 5m 04s","accounts_in_message":[],"_revision_number":1},{"id":"5c239ed264dde9abca5d87a5f38c865864fcb496","author":{"_account_id":11347,"name":"Rui Zang","email":"rui.zang@yandex.com","username":"rzang"},"date":"2018-12-10 07:25:54.000000000","message":"Uploaded patch set 2: Commit message was updated.","accounts_in_message":[],"_revision_number":2},{"id":"0967a705dc43b30abd42095418091b4c9146e2c8","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-12-10 07:37:05.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs http://logs.openstack.org/93/622893/2/check/openstack-tox-docs/657cd08/html/ : SUCCESS in 5m 48s\n- openstack-tox-pep8 http://logs.openstack.org/93/622893/2/check/openstack-tox-pep8/573661b/ : SUCCESS in 4m 29s","accounts_in_message":[],"_revision_number":2},{"id":"76940814d3fec9ae6ea52539169db51d5c7dd22f","author":{"_account_id":11347,"name":"Rui Zang","email":"rui.zang@yandex.com","username":"rzang"},"date":"2019-01-03 08:31:57.000000000","message":"Topic changed from topic/pmem-specs to bp/virtual-persistent-memory","accounts_in_message":[],"_revision_number":2},{"id":"310c7b7a36710e6aae381a5cad76eb64ed4816b3","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2019-01-07 06:38:01.000000000","message":"Patch Set 2: Code-Review-1\n\n(6 comments)\n\nthis is a fairly soft -1 the main concern being the\ncredentials used by the daxio utility and how we specify\nthe ssh key in particular.","accounts_in_message":[],"_revision_number":2},{"id":"44ebb747c6e82b63ee4652f9c44f5f7796f30133","author":{"_account_id":11347,"name":"Rui Zang","email":"rui.zang@yandex.com","username":"rzang"},"date":"2019-01-07 16:43:12.000000000","message":"Patch Set 2:\n\n(6 comments)\n\nHey, Sean, thank you very much for the valuable comments.","accounts_in_message":[],"_revision_number":2},{"id":"4e50b97adb673a565d7ab0c4045e449d4024e117","author":{"_account_id":11347,"name":"Rui Zang","email":"rui.zang@yandex.com","username":"rzang"},"date":"2019-01-08 15:08:20.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"847979e0445ad23db35e0b0b8b7d81fa005e8364","author":{"_account_id":11347,"name":"Rui Zang","email":"rui.zang@yandex.com","username":"rzang"},"date":"2019-01-08 15:08:42.000000000","message":"Patch Set 2:\n\n(3 comments)","accounts_in_message":[],"_revision_number":2},{"id":"c297adc4bbb189854cc19fab9dcb0c95a6e2b471","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-01-08 15:22:37.000000000","message":"Patch Set 3: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- openstack-tox-docs http://logs.openstack.org/93/622893/3/check/openstack-tox-docs/5582be2/html/ : SUCCESS in 6m 11s\n- openstack-tox-pep8 http://logs.openstack.org/93/622893/3/check/openstack-tox-pep8/2d51ded/ : FAILURE in 4m 33s","accounts_in_message":[],"_revision_number":3},{"id":"ad34fde89e2a44370f88b771c325382d6a4c1ae1","author":{"_account_id":11347,"name":"Rui Zang","email":"rui.zang@yandex.com","username":"rzang"},"date":"2019-01-08 15:33:17.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"23aa88016f0f4a1e2a36da39bb6f0446ee29a304","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-01-08 15:46:33.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs http://logs.openstack.org/93/622893/4/check/openstack-tox-docs/ba2fa93/html/ : SUCCESS in 6m 28s\n- openstack-tox-pep8 http://logs.openstack.org/93/622893/4/check/openstack-tox-pep8/9eb5ab3/ : SUCCESS in 4m 52s","accounts_in_message":[],"_revision_number":4},{"id":"dd26749a767553b696322602976ac10a9ab515c3","author":{"_account_id":17261,"name":"guochunting","email":"guochunting@inspur.com","username":"friendgct"},"date":"2019-02-22 00:36:07.000000000","message":"Patch Set 4: Code-Review+1","accounts_in_message":[],"_revision_number":4},{"id":"273aeef1fd9445d10c9d38d31f825ada8db5f2d0","author":{"_account_id":11347,"name":"Rui Zang","email":"rui.zang@yandex.com","username":"rzang"},"date":"2019-03-19 05:11:20.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"0d075f403732b55cfdba750335fd5dbf54ca6a16","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-03-19 05:31:51.000000000","message":"Patch Set 5: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs http://logs.openstack.org/93/622893/5/check/openstack-tox-docs/86de005/html/ : SUCCESS in 18m 54s\n- openstack-tox-pep8 http://logs.openstack.org/93/622893/5/check/openstack-tox-pep8/bf45576/ : SUCCESS in 18m 10s","accounts_in_message":[],"_revision_number":5},{"id":"a8ef0e1a38d5326600a1e4c57dab3e5cee8774e8","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-03-19 17:18:36.000000000","message":"Patch Set 5: Code-Review-1\n\nSimple add-on to the main doc.\n\nBut...\n\nI\u0027m wondering at the dividing line between the generic and libvirt-specific parts of this blueprint. For instance, either there\u0027s nothing libvirt-specific about the pmem concept in general - including the way we\u0027re going to model it in placement - in which case the conf options should be defined in a non-libvirt-specific group so they can be reused by other drivers; or it\u0027s entirely libvirt-specific and we should just consolidate the two documents.\n\nSo which is it?","accounts_in_message":[],"_revision_number":5},{"id":"75e176c56fd25d25f2973c4902f4a9e2efb223fb","author":{"_account_id":11347,"name":"Rui Zang","email":"rui.zang@yandex.com","username":"rzang"},"date":"2019-03-20 07:00:26.000000000","message":"Patch Set 5:\n\n\u003e Simple add-on to the main doc.\n \u003e \n \u003e But...\n \u003e \n \u003e I\u0027m wondering at the dividing line between the generic and\n \u003e libvirt-specific parts of this blueprint. For instance, either\n \u003e there\u0027s nothing libvirt-specific about the pmem concept in general\n \u003e - including the way we\u0027re going to model it in placement - in which\n \u003e case the conf options should be defined in a non-libvirt-specific\n \u003e group so they can be reused by other drivers; or it\u0027s entirely\n \u003e libvirt-specific and we should just consolidate the two documents.\n \u003e \n \u003e So which is it?\n\nIt is more like the first. PMEM concept is in general, so the main spec. However, the PMEM concept can be implement by multiple drivers. We just enable the libvirt driver. Thus this spec.\nThe main spec regulates the drivers should report the PMEM namespaces to placement, as to how, it is left to each independent driver implementation. Each driver may take its most convenient way to specify the configs and create namespaces accordingly.\nTo be specific, the PMEM namespace configuration is put into the libvirt section of the config file. Yes, the config format itself can be promoted to the main spec, however, I do not see much difference as leaving it to the drivers.\n\nThanks a lot for the review!","accounts_in_message":[],"_revision_number":5},{"id":"95bc8b36839bd6ff17f6981b3728f7921e89749e","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-04-09 16:30:30.000000000","message":"Patch Set 5:\n\nDo we really need a spec for the low level implementation? Why can\u0027t sufficient detail be in https://review.openstack.org/#/c/601596/? Or at the very least this should be rebased on top of https://review.openstack.org/#/c/601596/ since there is a logical dependency.","accounts_in_message":[],"_revision_number":5},{"id":"888d86fb949695f24731817182481381b339271b","author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"date":"2019-04-18 05:38:22.000000000","message":"Patch Set 5: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":5},{"id":"8c1388d34f790bdb525e2bfae7a548d33a86a5af","author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"date":"2019-04-24 04:11:44.000000000","message":"Patch Set 5: Code-Review-1\n\n(1 comment)\n\nYea, I also feel this is useless now. In the previous PTG, we want to separate the virt driver implementation detail out of the main spec. But yes, in the end, the virt driver implementation is pretty light, maybe just about few invoke of ndctl cmd. @Jay, are you ok merge this back to the main spec?","accounts_in_message":[],"_revision_number":5},{"id":"6d4ff508346cac3aec77e2b44b32f40b20fc3e38","author":{"_account_id":11347,"name":"Rui Zang","email":"rui.zang@yandex.com","username":"rzang"},"date":"2019-05-20 08:46:38.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"24401d7ee00e87636aa585ce294fa483f4dafa4c","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-05-20 09:07:26.000000000","message":"Patch Set 6: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs http://logs.openstack.org/93/622893/6/check/openstack-tox-docs/b8f50fe/html/ : SUCCESS in 7m 23s\n- openstack-tox-pep8 http://logs.openstack.org/93/622893/6/check/openstack-tox-pep8/be7a685/ : SUCCESS in 6m 20s","accounts_in_message":[],"_revision_number":6},{"id":"a6d601ccf3c27a743211e470011f2e7178d1752b","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-05-20 18:37:44.000000000","message":"Patch Set 6: Code-Review-1\n\n(8 comments)\n\nI\u0027m still -1 on keeping these specs separate. The discussion here of how to name and map namespaces is inextricable from the discussion of same in the other spec.\n\nUnless some other core feels quite strongly about this, I would like to see the specs combined.\n\nMore -1 for lack of clarity on the mechanism of mapping the resource class names to the namespaces. TL;DR:\n\n- If we can rely on ndctl -n being available, the conf option should simply be a whitelist of those labels.\n- If not, the conf option should be a list of $rc_name:$byte_capacity tuples so that the $byte_capacity can be used to match which $rc_name corresponds to which namespace.\n\nThe former would be much nicer IMO; otherwise you have to differentiate namespaces of the same size somehow.","accounts_in_message":[],"_revision_number":6},{"id":"c081c5cce74cc3a135b7421e0f216996770b54b0","author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"date":"2019-05-21 06:54:39.000000000","message":"Patch Set 6: Code-Review-1\n\n(2 comments)","accounts_in_message":[],"_revision_number":6},{"id":"83ef8238831a5c99077be25b0140d8f48b37027a","author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"date":"2019-05-21 07:39:48.000000000","message":"Patch Set 6:\n\n(1 comment)","accounts_in_message":[],"_revision_number":6},{"id":"8fe9d9bfd94b57a28a4622fa9dabf0cf923c9ea2","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-05-21 19:50:19.000000000","message":"Patch Set 6:\n\n(2 comments)","accounts_in_message":[],"_revision_number":6},{"id":"297cdb79d8b87d90a1df3d8f6c90fad62b759bec","author":{"_account_id":11347,"name":"Rui Zang","email":"rui.zang@yandex.com","username":"rzang"},"date":"2019-05-23 08:45:22.000000000","message":"Patch Set 6:\n\n(1 comment)","accounts_in_message":[],"_revision_number":6},{"id":"af29346e919cc742852824f0ce9ff9038c728503","author":{"_account_id":11347,"name":"Rui Zang","email":"rui.zang@yandex.com","username":"rzang"},"date":"2019-05-27 05:56:21.000000000","message":"Abandoned\n\nMerged this spec to the main spec.","accounts_in_message":[],"_revision_number":6}],"current_revision_number":6,"current_revision":"29f3bc9e18301c967af90c7a02fb03c2ad528c8b","revisions":{"45e3335fd3434d798468108d06189051b6bc6770":{"kind":"REWORK","_number":1,"created":"2018-12-05 08:38:53.000000000","uploader":{"_account_id":11347,"name":"Rui Zang","email":"rui.zang@yandex.com","username":"rzang"},"ref":"refs/changes/93/622893/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova-specs","ref":"refs/changes/93/622893/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/93/622893/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/93/622893/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/93/622893/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova-specs refs/changes/93/622893/1"}}},"commit":{"parents":[{"commit":"4558953da275f3de15eee101473cca9e18fc1662","subject":"Merge \"Re-propose numa-aware-live-migration spec\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/4558953da275f3de15eee101473cca9e18fc1662"}]}],"author":{"name":"Rui Zang","email":"rui.zang@intel.com","date":"2018-12-05 08:34:50.000000000","tz":-480},"committer":{"name":"Rui Zang","email":"rui.zang@intel.com","date":"2018-12-05 08:34:50.000000000","tz":-480},"subject":"Virtual persistent memory libvirt driver implementation","message":"Virtual persistent memory libvirt driver implementation\n\nThis spec documents the libvirt driver\u0027s implementation of\nvirtual persistent memory.\n\nChange-Id: Ie57414b232126d29f484e104931cacdb97a0d7a5\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/45e3335fd3434d798468108d06189051b6bc6770"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/45e3335fd3434d798468108d06189051b6bc6770"}]},"branch":"refs/heads/master"},"4fafdd86d9f8146e389ad07352c9883aecc88003":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2018-12-10 07:25:54.000000000","uploader":{"_account_id":11347,"name":"Rui Zang","email":"rui.zang@yandex.com","username":"rzang"},"ref":"refs/changes/93/622893/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova-specs","ref":"refs/changes/93/622893/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/93/622893/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/93/622893/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/93/622893/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova-specs refs/changes/93/622893/2"}}},"commit":{"parents":[{"commit":"4558953da275f3de15eee101473cca9e18fc1662","subject":"Merge \"Re-propose numa-aware-live-migration spec\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/4558953da275f3de15eee101473cca9e18fc1662"}]}],"author":{"name":"Rui Zang","email":"rui.zang@intel.com","date":"2018-12-05 08:34:50.000000000","tz":-480},"committer":{"name":"Rui Zang","email":"rui.zang@intel.com","date":"2018-12-10 07:24:26.000000000","tz":-480},"subject":"Virtual persistent memory libvirt driver implementation","message":"Virtual persistent memory libvirt driver implementation\n\nThis spec documents the libvirt driver\u0027s implementation of\nvirtual persistent memory.\n\nChange-Id: Ie57414b232126d29f484e104931cacdb97a0d7a5\nImplements: blueprint virtual-persistent-memory\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/4fafdd86d9f8146e389ad07352c9883aecc88003"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/4fafdd86d9f8146e389ad07352c9883aecc88003"}]},"branch":"refs/heads/master"},"96633f55b627b54915523972e1d8ca25956fc342":{"kind":"REWORK","_number":3,"created":"2019-01-08 15:08:20.000000000","uploader":{"_account_id":11347,"name":"Rui Zang","email":"rui.zang@yandex.com","username":"rzang"},"ref":"refs/changes/93/622893/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova-specs","ref":"refs/changes/93/622893/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/93/622893/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/93/622893/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/93/622893/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova-specs refs/changes/93/622893/3"}}},"commit":{"parents":[{"commit":"4558953da275f3de15eee101473cca9e18fc1662","subject":"Merge \"Re-propose numa-aware-live-migration spec\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/4558953da275f3de15eee101473cca9e18fc1662"}]}],"author":{"name":"Rui Zang","email":"rui.zang@intel.com","date":"2018-12-05 08:34:50.000000000","tz":-480},"committer":{"name":"Rui Zang","email":"rui.zang@intel.com","date":"2019-01-08 15:08:12.000000000","tz":-480},"subject":"Virtual persistent memory libvirt driver implementation","message":"Virtual persistent memory libvirt driver implementation\n\nThis spec documents the libvirt driver\u0027s implementation of\nvirtual persistent memory.\n\nChange-Id: Ie57414b232126d29f484e104931cacdb97a0d7a5\nImplements: blueprint virtual-persistent-memory\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/96633f55b627b54915523972e1d8ca25956fc342"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/96633f55b627b54915523972e1d8ca25956fc342"}]},"branch":"refs/heads/master"},"bf68e821cefc429ac55f19c210719b1674e33631":{"kind":"REWORK","_number":4,"created":"2019-01-08 15:33:17.000000000","uploader":{"_account_id":11347,"name":"Rui Zang","email":"rui.zang@yandex.com","username":"rzang"},"ref":"refs/changes/93/622893/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova-specs","ref":"refs/changes/93/622893/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/93/622893/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/93/622893/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/93/622893/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova-specs refs/changes/93/622893/4"}}},"commit":{"parents":[{"commit":"4558953da275f3de15eee101473cca9e18fc1662","subject":"Merge \"Re-propose numa-aware-live-migration spec\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/4558953da275f3de15eee101473cca9e18fc1662"}]}],"author":{"name":"Rui Zang","email":"rui.zang@intel.com","date":"2018-12-05 08:34:50.000000000","tz":-480},"committer":{"name":"Rui Zang","email":"rui.zang@intel.com","date":"2019-01-08 15:33:09.000000000","tz":-480},"subject":"Virtual persistent memory libvirt driver implementation","message":"Virtual persistent memory libvirt driver implementation\n\nThis spec documents the libvirt driver\u0027s implementation of\nvirtual persistent memory.\n\nChange-Id: Ie57414b232126d29f484e104931cacdb97a0d7a5\nImplements: blueprint virtual-persistent-memory\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/bf68e821cefc429ac55f19c210719b1674e33631"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/bf68e821cefc429ac55f19c210719b1674e33631"}]},"branch":"refs/heads/master"},"c5b77b4a3825100d4ff1b62b31161fc7d0893a42":{"kind":"REWORK","_number":5,"created":"2019-03-19 05:11:20.000000000","uploader":{"_account_id":11347,"name":"Rui Zang","email":"rui.zang@yandex.com","username":"rzang"},"ref":"refs/changes/93/622893/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova-specs","ref":"refs/changes/93/622893/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/93/622893/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/93/622893/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/93/622893/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova-specs refs/changes/93/622893/5"}}},"commit":{"parents":[{"commit":"d796b33774cc4e21c663885a57141e93052e5546","subject":"Merge \"Amend the detach-boot-volume design\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/d796b33774cc4e21c663885a57141e93052e5546"}]}],"author":{"name":"Rui Zang","email":"rui.zang@intel.com","date":"2018-12-05 08:34:50.000000000","tz":-480},"committer":{"name":"Rui Zang","email":"rui.zang@intel.com","date":"2019-03-19 05:11:05.000000000","tz":-420},"subject":"Virtual persistent memory libvirt driver implementation","message":"Virtual persistent memory libvirt driver implementation\n\nThis spec documents the libvirt driver\u0027s implementation of\nvirtual persistent memory.\n\nChange-Id: Ie57414b232126d29f484e104931cacdb97a0d7a5\nImplements: blueprint virtual-persistent-memory\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/c5b77b4a3825100d4ff1b62b31161fc7d0893a42"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/c5b77b4a3825100d4ff1b62b31161fc7d0893a42"}]},"branch":"refs/heads/master"},"29f3bc9e18301c967af90c7a02fb03c2ad528c8b":{"kind":"REWORK","_number":6,"created":"2019-05-20 08:46:38.000000000","uploader":{"_account_id":11347,"name":"Rui Zang","email":"rui.zang@yandex.com","username":"rzang"},"ref":"refs/changes/93/622893/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova-specs","ref":"refs/changes/93/622893/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/93/622893/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/93/622893/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/93/622893/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova-specs refs/changes/93/622893/6"}}},"commit":{"parents":[{"commit":"fe9ee4a816d90d960d1b9d930650a0ee97c378d9","subject":"support virtual persistent memory","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/fe9ee4a816d90d960d1b9d930650a0ee97c378d9"}]}],"author":{"name":"Rui Zang","email":"rui.zang@intel.com","date":"2018-12-05 08:34:50.000000000","tz":-480},"committer":{"name":"Rui Zang","email":"rui.zang@intel.com","date":"2019-05-20 08:45:55.000000000","tz":-420},"subject":"Virtual persistent memory libvirt driver implementation","message":"Virtual persistent memory libvirt driver implementation\n\nThis spec documents the libvirt driver\u0027s implementation of\nvirtual persistent memory.\n\nChange-Id: Ie57414b232126d29f484e104931cacdb97a0d7a5\nImplements: blueprint virtual-persistent-memory\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/29f3bc9e18301c967af90c7a02fb03c2ad528c8b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/29f3bc9e18301c967af90c7a02fb03c2ad528c8b"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}