| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-26 | |||
| 11:19:14 | openstackgerrit | Artem Tiumentcev proposed openstack/python-novaclient master: Add 'local_gb' disk quota support https://review.openstack.org/484391 | |
| 11:31:26 | openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance.exists notification https://review.openstack.org/403660 | |
| 11:34:59 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add sample test for instance audit https://review.openstack.org/480955 | |
| 11:39:01 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add sample test for instance audit https://review.openstack.org/480955 | |
| 11:44:10 | kashyap | assert_called_once_with(*args, **kwargs)¶ | |
| 11:44:20 | kashyap | Eurgh, wrong window | |
| 11:44:25 | openstackgerrit | Merged openstack/nova master: Translate the return value of attachment_create and _update https://review.openstack.org/486194 | |
| 11:44:30 | kashyap | Was copy / pasting it in a different buffer | |
| 11:56:30 | openstackgerrit | Alex Szarka proposed openstack/nova master: fix test_rebuild_server_exc instability https://review.openstack.org/487382 | |
| 11:59:12 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Post-migration, set cache value for Cinder volume(s) https://review.openstack.org/485752 | |
| 12:00:41 | openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance-live_migration_abort notification https://review.openstack.org/420453 | |
| 12:08:35 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/477091 | |
| 12:08:47 | openstackgerrit | Radoslav Gerganov proposed openstack/nova master: Add connection pool size to vSphere settings https://review.openstack.org/414085 | |
| 12:39:58 | ioggstream | hi @all | |
| 12:40:24 | mriedem | o/ | |
| 12:40:44 | ioggstream | after a stack update I discovered somebody downsized the flavor disk | |
| 12:40:47 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: query instance after save to avoid partial update https://review.openstack.org/486561 | |
| 12:41:08 | ioggstream | u"Flavor's disk is too small for requested image. Flavor disk is 32212254720 bytes, image is 64424509440 bytes." | |
| 12:41:28 | ioggstream | I fixed the flavor and re-updated the stack, but it didn't fix | |
| 12:42:19 | ioggstream | one host is "Unavailable console type serial." | |
| 12:44:12 | ioggstream | What happens when "Flavor disk is to small?" Nova destroys the disk or just stops? | |
| 12:44:37 | ioggstream | is there a way to access the root volumes and get the data? | |
| 12:46:56 | alex_xu | nova api meeting is 14 mins at #openstack-meeting-4 | |
| 12:47:19 | openstackgerrit | Chris Dent proposed openstack/nova master: style-only: s/context/ctx/ https://review.openstack.org/485791 | |
| 12:47:50 | openstackgerrit | Chris Dent proposed openstack/nova master: use os_traits.MISC_SHARES_VIA_AGGREGATE https://review.openstack.org/485792 | |
| 12:49:46 | cdent | mriedem: those ^ two needed a rebase to be aligned with master (which had introduced more ‘context’. there were previously both +W so could you do the honors? | |
| 12:50:08 | jangutter | Just enough time to watch https://www.youtube.com/watch?v=1g6QJ5TfA7w before the meeting. Voltron! Assemble! | |
| 12:58:07 | cdent | thank mriedem | |
| 12:58:35 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Move the note about '/os-volume_boot' to the correct place https://review.openstack.org/486071 | |
| 13:00:54 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: [WIP] Add support for Windows network commands https://review.openstack.org/487405 | |
| 13:01:17 | alex_xu | nova api meeting is running at #openstack-meeting-4 | |
| 13:04:25 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Microversion 2.53 - services and hypervisors using UUIDs https://review.openstack.org/485435 | |
| 13:05:04 | gibi | mriedem: hi! I added example error messages to the patch https://review.openstack.org/#/c/428199/ ( | |
| 13:05:17 | gibi | mriedem: as we agreed | |
| 13:08:22 | moshele | jaypipes , mriedem: hi can you review https://review.openstack.org/#/c/398265/ ? | |
| 13:18:29 | openstackgerrit | LIU Yulong proposed openstack/nova master: Enable reset keypair while rebuilding instance https://review.openstack.org/379128 | |
| 13:25:10 | mriedem | gmann_: should this be abandoned? https://review.openstack.org/#/c/451331/ | |
| 13:25:42 | mriedem | alex_xu: after the meeting, should this be abandoned? https://review.openstack.org/#/c/458004/ | |
| 13:28:40 | sdague | mriedem: can you set up whatever tests you were running yesterday on this - https://review.openstack.org/#/c/487246/ | |
| 13:28:52 | sdague | that should make things wait for compute service to start correctly | |
| 13:29:15 | mriedem | sdague: multinode neutron job should be sufficient to test that | |
| 13:30:03 | sdague | mriedem: ok, is that just in neutron experimental queue? | |
| 13:30:24 | sdague | or you mean the normal multinode job can see the issue? | |
| 13:31:22 | dansmith | mriedem: are we reverting multicell yet? | |
| 13:31:55 | mriedem | sdague: normal | |
| 13:31:59 | mriedem | dansmith: i haven't heard of anything | |
| 13:32:02 | dansmith | amazing | |
| 13:32:13 | mriedem | gibi: ok looks good, i wanted to show you something else first | |
| 13:32:34 | sdague | mriedem: ok, I'm not sure what behavior you are looking for, but there are result runs here - https://review.openstack.org/#/c/487246 | |
| 13:32:35 | gibi | mriedem: sure | |
| 13:32:56 | mriedem | sdague: yeah i'll look after the api meeting | |
| 13:36:17 | mriedem | gibi: i was reminded of this testtools matcher yesterday https://review.openstack.org/#/c/486850/6/nova/tests/unit/api/openstack/compute/test_versions.py@303 | |
| 13:36:25 | mriedem | matchers.DictMatches(expected)) | |
| 13:36:25 | mriedem | self.assertThat(jsonutils.loads(res.body), | |
| 13:36:30 | mriedem | which is similar to what you're doin | |
| 13:36:37 | mriedem | if it fails, it tells you what the diff is | |
| 13:36:44 | mriedem | gibi: so i wonder if we could re-use that | |
| 13:36:59 | gibi | mriedem: I can check that out | |
| 13:37:11 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Migration from ``ip`` commands to ``pyroute2`` https://review.openstack.org/484386 | |
| 13:38:34 | gibi | mriedem: I will plug that in to the notification sample test to see if it works and then I will check what it prints when something does not match | |
| 13:40:17 | gmann_ | mriedem: hum, i can do abandon. but i wan to give try to merge schema and code split among files. but for next release | |
| 13:40:37 | gmann_ | mriedem: we need to abandon or detach from BP to close BP? | |
| 13:41:45 | mriedem | gmann_: no, i just wanted to clean up the review queue for that bp | |
| 13:42:36 | mriedem | moshele: is that building into an existing bp? | |
| 13:42:40 | mriedem | or something else? | |
| 13:43:25 | moshele | mriedem: I just opened neutron RFE | |
| 13:45:26 | gmann_ | mriedem: ok, i will detach that from it. | |
| 13:46:57 | jaypipes | moshele: done. | |
| 13:52:29 | kashyap | "recheck pep8" is a thing? | |
| 13:52:37 | mriedem | no | |
| 13:53:16 | mriedem | cdent: sdague: wording nit in https://review.openstack.org/#/c/486623/3/releasenotes/notes/deprecate_wsgi_log_format-43a10b7a608ea8f3.yaml | |
| 13:53:47 | cdent | yeah, good point, probably “preferred deployment mode" | |
| 13:53:52 | openstackgerrit | Sean Dague proposed openstack/nova master: deprecate ``wsgi_log_format`` config variable https://review.openstack.org/486623 | |
| 13:54:03 | sdague | mriedem: done | |
| 13:54:13 | kashyap | mriedem: Thanks. Sigh, I made a silly tabs / spaces mistake, and paying the penalty of not running `pep8` locally first on last iteration | |
| 13:55:36 | bauzas | mriedem: could you please point me out the super etherpad for reviews ? | |
| 13:55:45 | bauzas | I mean, the one for the left BPs | |
| 13:56:09 | bauzas | mriedem: yeah I'm lazy and I can find it by myself but hopefully your tab is still open | |
| 13:57:30 | jangutter | https://etherpad.openstack.org/p/nova-pike-feature-freeze-status | |
| 13:58:19 | gmann_ | bauzas: this one - https://etherpad.openstack.org/p/nova-pike-feature-freeze-status | |
| 13:58:30 | bauzas | thanks | |
| 13:59:17 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Post-migration, set cache value for Cinder volume(s) https://review.openstack.org/485752 | |
| 13:59:24 | kashyap | And 'recheck pep8' doesn't fix it, as I botched up indentation. The above should fix it. | |
| 14:02:32 | alex_xu | mriedem: I think I will continue https://review.openstack.org/#/c/458004 after freeze as a bug fix. it just adds a test, it isn't realy part of no more ext BP. | |
| 14:03:04 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge server create schema for availability zone extension https://review.openstack.org/451331 | |
| 14:04:10 | jangutter | sean-k-mooney: you perhaps around to do a (hopefully) final eyeball on https://review.openstack.org/#/c/483459/ ? | |
| 14:05:20 | vdrok | good morning folks | |
| 14:05:30 | gibi | mriedem: it seems replacing JsonEqual with DictMatcher passes the sample tests so this direction looks promising | |
| 14:05:48 | gibi | mriedem: I will rework my improvement patch | |
| 14:06:08 | vdrok | mriedem: seeing http://logs.openstack.org/88/473888/12/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-multinode-ubuntu-xenial/d56cef7/logs/screen-n-cond-cell1.txt.gz#_Jul_26_11_15_52_052437 again in the multinode job :( I thought the option you've added in the last patchset of devstack change was the workaround for the subnode? | |
| 14:10:02 | dansmith | vdrok: jlvillal said you guys had fixed that | |
| 14:10:45 | dansmith | vdrok: this: https://review.openstack.org/#/c/477313/ | |
| 14:10:55 | dansmith | but I guess that was grenade | |
| 14:11:18 | vdrok | dansmith: we did add the nomulticell for grenade. but for the multinode job I thought the fix was https://review.openstack.org/#/c/477556/4..5/lib/nova | |
| 14:11:21 | dansmith | vdrok: this? http://logs.openstack.org/88/473888/12/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-multinode-ubuntu-xenial/d56cef7/logs/screen-n-cond-cell1.txt.gz#_Jul_26_11_15_52_128079 | |
| 14:12:02 | dansmith | vdrok: I don't think so | |
| 14:13:44 | dansmith | vdrok: you won't get any reschedules with the multicell layout, so if you really hit those in regular gate jobs like this then we need to make you non multicell for now I guess | |
| 14:13:47 | dansmith | even in the regular ones | |
| 14:14:43 | jangutter | moshel: regarding my comments on https://review.openstack.org/#/c/398265 it's basically nova sending extra data to os-vif, so that in future os-vif can handle representors uniformly. | |
| 14:14:55 | vdrok | dansmith: hrm, so this means we need to add some stuff to devstack to check that if virt driver = ironic add the nomulticell everywhere? | |