| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-16 | |||
| 00:33:55 | dansmith | having spent hours on the patch the way it is, I don't think it can get much gnarly-er | |
| 00:34:19 | dansmith | I didn't really start grok'ing most of it until I was ass-deep in trying to split it up | |
| 00:34:22 | melwitt | mriedem: if we rollback on a destroy fail, then things should be working right I thought | |
| 00:34:39 | melwitt | dansmith: hah, that's fair. yeah, go ahead and upload | |
| 00:34:54 | mriedem | melwitt: only if we hit InstanceNotFound | |
| 00:34:59 | mriedem | melwitt: they didn't say what error they hit though | |
| 00:39:25 | melwitt | hm, well in _delete_while_booting, it quota.commits immediately, before trying to delete, so that seems wrong. the rollback for InstanceNotFound also wouldn't do anything if commit already happened. but I didn't think we touched _delete_while_booting | |
| 00:40:09 | melwitt | yeah, these are all committing immediately, which I think would make rollback do nothing | |
| 00:40:24 | openstackgerrit | Dan Smith proposed openstack/nova master: Count resources to check quota for cells https://review.openstack.org/416521 | |
| 00:40:25 | openstackgerrit | Dan Smith proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945 | |
| 00:40:25 | openstackgerrit | Dan Smith proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946 | |
| 00:40:26 | openstackgerrit | Dan Smith proposed openstack/nova master: Add Quotas.check_deltas() and set the stage for magic to happen https://review.openstack.org/446239 | |
| 00:40:26 | openstackgerrit | Dan Smith proposed openstack/nova master: Count server group quotas https://review.openstack.org/446240 | |
| 00:40:27 | openstackgerrit | Dan Smith proposed openstack/nova master: Count tenant_networks quotas https://review.openstack.org/446241 | |
| 00:40:27 | openstackgerrit | Dan Smith proposed openstack/nova master: Count used_limits quotas https://review.openstack.org/446242 | |
| 00:40:28 | openstackgerrit | Dan Smith proposed openstack/nova master: Remove useless quota_usage_refresh from nova-manage https://review.openstack.org/446243 | |
| 00:40:28 | openstackgerrit | Dan Smith proposed openstack/nova master: Add get_count_by_vm_state() to Instance object https://review.openstack.org/446244 | |
| 00:40:29 | openstackgerrit | Dan Smith proposed openstack/nova master: Add SecurityGroup.get_counts() https://review.openstack.org/446245 | |
| 00:40:29 | openstackgerrit | Dan Smith proposed openstack/nova master: Add FixedIP.get_count_by_project() https://review.openstack.org/446246 | |
| 00:40:30 | openstackgerrit | Dan Smith proposed openstack/nova master: Add FloatingIP.get_count_by_project() https://review.openstack.org/446247 | |
| 00:40:48 | melwitt | thanks dansmith | |
| 00:40:54 | dansmith | the final patch is like half the size | |
| 00:41:29 | melwitt | woohoo | |
| 00:41:42 | dansmith | fwiw, this makes it so much more obvious why this is good: | |
| 00:41:43 | dansmith | https://review.openstack.org/#/c/446239/1 | |
| 00:41:52 | dansmith | in just quota stuff, +500,-1000 | |
| 00:42:20 | dansmith | anyway, I've made my mess, time for dinner | |
| 00:42:22 | melwitt | yeah, that's a win for sure | |
| 00:42:38 | melwitt | haha, o/ | |
| 00:44:00 | melwitt | mriedem: quotas.commit and quotas.rollback are mutually exclusive and I should have noticed that on the review. rollback won't work if commit already happened | |
| 00:49:25 | mriedem | melwitt: then wouldn't it also be wrong in _delete_while_booting? https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1807 | |
| 00:49:28 | mriedem | that's what i copied this from | |
| 00:50:06 | melwitt | yes, I think it's wrong there too | |
| 00:51:18 | melwitt | I'm looking to see what rollback does if commit already occurred | |
| 00:51:55 | melwitt | I think it's just a no-op because it would find no reservation record | |
| 00:52:37 | melwitt | commit would have deleted the reservation record. then rollback comes along and reads the reservation, finds nothing, and then does nothing | |
| 00:55:17 | mriedem | well hells bells | |
| 00:55:30 | mriedem | i'm pretty sure _delete_while_booting copied that behavior from _delete | |
| 00:56:02 | mriedem | so normal delete we get the reservation here https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1943 | |
| 00:56:46 | mriedem | so it probably copied from https://github.com/openstack/nova/blob/master/nova/compute/api.py#L2028 | |
| 00:57:15 | mriedem | looks like in that flow though, we don't commit unless we delete https://github.com/openstack/nova/blob/master/nova/compute/api.py#L2023 | |
| 00:57:49 | mriedem | so (1) create reservation, (2) delete instance = works, then commit; else rollback | |
| 00:57:52 | mriedem | i think that's how things are working in the old delete flow | |
| 00:57:57 | mriedem | before the cellsv2ification | |
| 01:01:03 | mriedem | also, i think i'm *this* close to getting this functional test to pass | |
| 01:01:10 | mriedem | fixtures up the wazoo | |
| 01:06:08 | mriedem | melwitt: i also went with your idea to stub out Claim.__init__, that was a good idea | |
| 01:08:56 | mriedem | oh f yes it works now | |
| 01:20:03 | mriedem | here it comes | |
| 01:20:19 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add populate_retry to schedule_and_build_instances https://review.openstack.org/444106 | |
| 01:20:20 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add a functional regression/recreate test for bug 1671648 https://review.openstack.org/446209 | |
| 01:20:20 | openstack | bug 1671648 in OpenStack Compute (nova) "Instances are not rescheduled after deploy fails" [High,In progress] https://launchpad.net/bugs/1671648 - Assigned to Shunli Zhou (shunliz) | |
| 01:41:26 | openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova master: Clarify os-start API description https://review.openstack.org/446264 | |
| 03:25:26 | openstackgerrit | Wang Shilong proposed openstack/nova master: Lustre support https://review.openstack.org/446288 | |
| 04:34:18 | openstackgerrit | Maho Koshiya proposed openstack/nova master: Add confirm resized server functional negative tests https://review.openstack.org/421074 | |
| 05:29:40 | jith | Hi all, I have configured openstack kilo setup in debian jessie with one controller node and two compute nodes. I have used glusterfs for shared storage. I have mounted the shared storage in /var/lib/nova/instances on both nodes. I can do migration of vm’s between nodes.. But live-migration throws following error.. Pls do guide me.. http://pastebin.com/DHPWEkaZ | |
| 05:56:24 | openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Permit Live Migration of Rescued Instances https://review.openstack.org/308198 | |
| 06:49:02 | jith | Hi all, I have configured openstack kilo setup in debian jessie with one controller node and two compute nodes. I have used glusterfs for shared storage. I have mounted the shared storage in /var/lib/nova/instances on both nodes. I can do migration of vm’s between nodes.. But live-migration throws following error.. Pls do guide me.. http://pastebin.com/DHPWEkaZ | |
| 06:55:55 | openstackgerrit | lcsong proposed openstack/nova master: Modify some grammatical mistakes. https://review.openstack.org/446321 | |
| 07:05:17 | openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: API changes for live migration of rescued instance https://review.openstack.org/328280 | |
| 07:13:36 | alexey_weyl | Hi, I have a small question about the availability zone. | |
| 07:14:53 | openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Port binding based on events during live migration https://review.openstack.org/434870 | |
| 07:14:59 | alexey_weyl | When I change the name or the availability zone itself in the host aggregates I don't receive an update event on the oslo bus. Is nove supposed to send such an event? | |
| 07:15:09 | alexey_weyl | *nove=nova | |
| 07:36:23 | openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova-specs master: Live Migration of Rescued Instances https://review.openstack.org/347161 | |
| 07:41:47 | openstackgerrit | Mikhail Feoktistov proposed openstack/nova master: Add is_vz_container function https://review.openstack.org/445947 | |
| 07:55:40 | openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova-specs master: Live Migration of Rescued Instances https://review.openstack.org/347161 | |
| 07:58:37 | nirendra | Nova evacuate on Ocata is failing. Is there any known bug corresponding to this or a workaround | |
| 08:10:11 | gmann | alex_xu: hi | |
| 08:10:53 | alex_xu | gmann: hi | |
| 08:11:20 | gmann | alex_xu: show_extensions seems to enforce all extension discovery policy - https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/extension_info.py#L285 | |
| 08:11:40 | gmann | alex_xu: instead of asked extension discovery policy only | |
| 08:12:14 | gmann | self._get_extensions(context) is being called in show and it checks all extension discovery policy | |
| 08:13:00 | alexey_weyl | Hi, I wanted to ask please why there is no id for the availability-zone? | |
| 08:13:54 | alex_xu | gmann: emm...looks like yes | |
| 08:14:30 | alex_xu | gmann: but it works correctly? | |
| 08:15:55 | gmann | alex_xu: because of fatal=False on L226 right? | |
| 08:16:42 | gmann | if anyone disabled other extension on policy discovery then it would not affect of getting asked extension right? | |
| 08:17:16 | alex_xu | gmann: yes | |
| 08:18:20 | gmann | alex_xu: yea, actually it came up on https://review.openstack.org/#/c/431740/4 i am not sure why they cannot use show_extension for all, let me check with author | |
| 08:20:07 | Kevin_Zheng | ping gibi | |
| 08:20:44 | alexey_weyl | can anyone please help me | |
| 08:20:50 | alexey_weyl | I wanted to ask please why there is no id for the availability-zone? | |
| 08:21:43 | alex_xu | gmann: emm....strange | |
| 08:22:48 | gmann | alex_xu: yea from user point of view show_extension will be same, its only internal implementation which does extra checks on policies | |
| 08:24:06 | gmann | alexey_weyl: for AZ we have only name | |
| 08:24:12 | alex_xu | gmann: but the discovery policy is something we deprecated? | |
| 08:24:21 | alex_xu | why people still depend on it | |
| 08:24:46 | gmann | alexey_weyl: i mean its always different name so id not needed | |
| 08:25:32 | gmann | alex_xu: you mean extensions right or discovery policy also we deprecated ? | |
| 08:27:48 | bauzas | good morning Nova | |
| 08:28:03 | alex_xu | gmann: i think the discovery policy | |
| 08:28:33 | alex_xu | gmann: https://review.openstack.org/#/c/427872/ johnthetubaguy propose to remove in it in that spec | |
| 08:28:53 | alex_xu | gmann: not suer whether we have note to say it already deprecated | |
| 08:29:30 | gmann | alex_xu: yea, i was thinking the same. i think we do not have as there was no good way to deprecate policy... | |
| 08:29:45 | openstackgerrit | jichenjc proposed openstack/nova master: Prevent delete cell0 in nova-manage command https://review.openstack.org/433476 | |
| 08:29:56 | alexey_weyl | gmann: that is correct, but if i change the name of the availability zone then vitrage don't know of which zone it was originally | |
| 08:30:46 | alexey_weyl | gmann: meanning, Vitrage don't know what zone was changed, because the previous name doesn't appear. | |
| 08:30:48 | alex_xu | gmann: not like config opt | |
| 08:34:11 | gmann | alex_xu: yea | |