)]}'
{"id":"openstack%2Fdevstack~44447","triplet_id":"openstack%2Fdevstack~master~Iad85f573b90c23140012c20c552a17277d9c97a0","project":"openstack/devstack","branch":"master","topic":"setmem","hashtags":[],"change_id":"Iad85f573b90c23140012c20c552a17277d9c97a0","subject":"xenapi: Set VM memory before starting it","status":"MERGED","created":"2013-08-30 12:34:50.000000000","updated":"2013-09-04 18:06:41.000000000","submitted":"2013-09-04 18:06:41.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"56013c816a798b46e202998495715b4aedbebdaf","_number":44447,"virtual_id_number":44447,"owner":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2013-09-04 18:06:41.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},{"value":2,"date":"2013-09-04 14:45:11.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":2,"date":"2013-09-04 11:13:56.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"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":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2013-08-30 13:33:09.000000000","_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},{"value":1,"date":"2013-09-04 14:45:11.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"date":"2013-09-04 11:13:56.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-09-04 11:13:56.000000000","updated_by":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"reviewer":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"state":"REVIEWER"},{"updated":"2013-09-04 14:45:11.000000000","updated_by":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"reviewer":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"state":"REVIEWER"},{"updated":"2013-09-04 18:06:41.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"7df3167d27a59e81447d15e34045a4993ea11d4d","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-08-30 13:00:38.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- gate-tempest-devstack-vm-full http://logs.openstack.org/47/44447/1/check/gate-tempest-devstack-vm-full/f8bbac6 : SUCCESS in 25m 45s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/47/44447/1/check/gate-tempest-devstack-vm-postgres-full/b60ee84 : SUCCESS in 24m 32s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/47/44447/1/check/gate-tempest-devstack-vm-neutron/6cb01c8 : SUCCESS in 13m 53s\n- gate-grenade-devstack-vm http://logs.openstack.org/47/44447/1/check/gate-grenade-devstack-vm/9c84c97 : SUCCESS in 21m 29s\n","accounts_in_message":[],"_revision_number":1},{"id":"75baf264cf7f2f392ce7ee7c23439ca7821ae439","author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"date":"2013-08-30 13:33:09.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"79d0b551abef6cc59db1ab510a90366a6b2b5848","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-08-30 13:59:22.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- gate-tempest-devstack-vm-full http://logs.openstack.org/47/44447/2/check/gate-tempest-devstack-vm-full/0a8287d : SUCCESS in 24m 15s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/47/44447/2/check/gate-tempest-devstack-vm-postgres-full/51112eb : SUCCESS in 26m 10s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/47/44447/2/check/gate-tempest-devstack-vm-neutron/03d3e66 : SUCCESS in 16m 47s\n- gate-grenade-devstack-vm http://logs.openstack.org/47/44447/2/check/gate-grenade-devstack-vm/0ca3f03 : SUCCESS in 21m 17s\n","accounts_in_message":[],"_revision_number":2},{"id":"51154076f684492b86ebd3ac47ccba1163da7d5f","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-09-04 11:13:56.000000000","message":"Patch Set 2: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"f9c736704351dd5f6dba6a999ea8841a5f02303e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-04 14:45:11.000000000","message":"Patch Set 2: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"7b67c8a9c2b90c5f2151273c84d0341b785bb02e","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-09-04 14:45:11.000000000","message":"Patch Set 2: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":2},{"id":"d95807c55dd26f00861286a5898fb34ca06dae3f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-04 18:06:41.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":2},{"id":"87d6b63e5bf768573936d76fd8ea3b6752bb97df","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-04 18:06:41.000000000","message":"Patch Set 2: Verified\n\nBuild succeeded.\n\n- gate-tempest-devstack-vm-full http://logs.openstack.org/47/44447/2/gate/gate-tempest-devstack-vm-full/9bced80 : SUCCESS in 25m 12s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/47/44447/2/gate/gate-tempest-devstack-vm-postgres-full/8faeebd : SUCCESS in 27m 37s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/47/44447/2/gate/gate-tempest-devstack-vm-neutron/d8061bc : SUCCESS in 17m 25s\n- gate-grenade-devstack-vm http://logs.openstack.org/47/44447/2/gate/gate-grenade-devstack-vm/4d209a5 : SUCCESS in 22m 46s\n","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"16ed068db52516238b618408656fa0bc612b9218","revisions":{"4a1c25738d79a03be119b81cd3adee1d2f3ead0a":{"kind":"REWORK","_number":1,"created":"2013-08-30 12:34:50.000000000","uploader":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"ref":"refs/changes/47/44447/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/47/44447/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/47/44447/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/47/44447/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/47/44447/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/47/44447/1"}}},"commit":{"parents":[{"commit":"117af3430f34f375b3343cc062257f87b9e8990c","subject":"Merge \"Add support for heat resource templates\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/117af3430f34f375b3343cc062257f87b9e8990c"}]}],"author":{"name":"Mate Lakat","email":"mate.lakat@citrix.com","date":"2013-08-30 12:28:31.000000000","tz":60},"committer":{"name":"Mate Lakat","email":"mate.lakat@citrix.com","date":"2013-08-30 12:28:31.000000000","tz":60},"subject":"xenapi: Set VM memory before starting it","message":"xenapi: Set VM memory before starting it\n\nIf someone was re-using an existing template, for a memory change, he\nneeded to re-install the vm. This change sets the osdomu mem before\nstarting it, so it doesn\u0027t matter how much memory did the VM had\naccording to the template.\n\nChange-Id: Iad85f573b90c23140012c20c552a17277d9c97a0\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/4a1c25738d79a03be119b81cd3adee1d2f3ead0a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/4a1c25738d79a03be119b81cd3adee1d2f3ead0a"}]},"branch":"refs/heads/master"},"16ed068db52516238b618408656fa0bc612b9218":{"kind":"REWORK","_number":2,"created":"2013-08-30 13:33:09.000000000","uploader":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"ref":"refs/changes/47/44447/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/47/44447/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/47/44447/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/47/44447/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/47/44447/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/47/44447/2"}}},"commit":{"parents":[{"commit":"117af3430f34f375b3343cc062257f87b9e8990c","subject":"Merge \"Add support for heat resource templates\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/117af3430f34f375b3343cc062257f87b9e8990c"}]}],"author":{"name":"Mate Lakat","email":"mate.lakat@citrix.com","date":"2013-08-30 12:28:31.000000000","tz":60},"committer":{"name":"Mate Lakat","email":"mate.lakat@citrix.com","date":"2013-08-30 13:21:10.000000000","tz":60},"subject":"xenapi: Set VM memory before starting it","message":"xenapi: Set VM memory before starting it\n\nIf someone was re-using an existing template, for a memory change, he\nneeded to re-install the vm. This change sets the osdomu mem before\nstarting it, so it doesn\u0027t matter how much memory did the VM had\naccording to the template. It also removes the memory manipulation bits\nfrom install-os-vpx.sh.\n\nChange-Id: Iad85f573b90c23140012c20c552a17277d9c97a0\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/16ed068db52516238b618408656fa0bc612b9218"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/16ed068db52516238b618408656fa0bc612b9218"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}