| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-26 | |||
| 10:37:07 | mamandle | I can push a new patch for review accordingly. | |
| 10:44:53 | openstackgerrit | Alex Szarka proposed openstack/nova master: Raise Exception instead of Exception method call https://review.openstack.org/482200 | |
| 10:47:48 | openstackgerrit | Manas Mandlekar proposed openstack/nova master: Instance remains in migrating state forever https://review.openstack.org/483911 | |
| 10:55:41 | openstackgerrit | Artem Tiumentcev proposed openstack/nova master: Add root and ephemeral disk quota https://review.openstack.org/484390 | |
| 11:07:03 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add sample test for instance audit https://review.openstack.org/480955 | |
| 11:08:02 | Sara_ | Hello all.. I have launched a VM on libvirt compute host..Then I have removed and added the compute host to the cluster..After joining the cluster, the Vm which was launched previously on that moves to error state..with error: No such file or directory: '/var/lib/nova/instances/2f44884d-9877-4da0-abd2-243979b803b7/disk'", "code": 500, "details": " File \"/usr/lib/python2.7/dist-packages/nova/compute/manager.py\", line 375, in d | |
| 11:08:20 | Sara_ | For your info, resume_guest_state_on_host_boot is set to false.Could someone help me out? | |
| 11:16:15 | openstackgerrit | Artem Tiumentcev proposed openstack/nova master: Add root and ephemeral disk quota https://review.openstack.org/484390 | |
| 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/ | |