Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-01-04
17:03:00 sfinucan mdbooth: I did, but I'd lost the context. Not going to hold it up for my one comment
17:03:22 mdbooth Good, because I don't know the answer :)
17:06:50 mdbooth sfinucan: They mutually conflict (in tests-py3.txt), btw, so at least one of them will have to be remunged whatever.
17:06:59 edleafe cdent: did you mean 'persevering' or 'preseverating'? :)
17:07:18 mdbooth preseverating sounds wonderful
17:07:31 cdent heh, well played
17:07:32 mdbooth and potentially dangerous
17:07:57 cdent how about both?
17:14:54 openstackgerrit Rafael Folco proposed openstack/nova: DELETE all inventory for a resource provider https://review.openstack.org/416669
17:16:54 openstackgerrit Dan Peschman proposed openstack/nova: No API cell up-call to delete consoleauth tokens https://review.openstack.org/415922
17:22:43 mriedem woot https://blueprints.launchpad.net/nova/+spec/consistent-query-parameters-validation is complete
17:22:50 mriedem alex_xu: nice work
17:25:58 openstackgerrit Chris Dent proposed openstack/nova: [placement] Add a bit about extraction plans to placement_dev https://review.openstack.org/416676
17:27:44 openstackgerrit Rafael Folco proposed openstack/nova: DELETE all inventory for a resource provider https://review.openstack.org/416669
17:28:50 cdent mriedem: is this still on your radar? https://review.openstack.org/#/c/414230/ (more logging on rp update failures)
17:29:09 cdent If not, I can probably pick it up
17:29:26 mriedem cdent: i'm not actively on it right now so feel free
17:29:38 openstackgerrit Gábor Antal proposed openstack/nova: Transform instance.volume_attach notification https://review.openstack.org/401992
17:29:42 cdent ✔
17:30:57 openstackgerrit Rafael Folco proposed openstack/nova: DELETE all inventory for a resource provider https://review.openstack.org/416669
17:33:14 openstackgerrit Rafael Folco proposed openstack/nova: DELETE all inventory for a resource provider https://review.openstack.org/416669
17:35:47 openstackgerrit Chris Dent proposed openstack/nova: WIP: Add more debug logging on RP inventory update failures https://review.openstack.org/414230
17:36:15 openstackgerrit Gábor Antal proposed openstack/nova: Transform instance.volume_detach notification https://review.openstack.org/408676
17:37:01 openstackgerrit Gleb Stepanov proposed openstack/python-novaclient: Refactor novaclient code https://review.openstack.org/416682
17:39:17 sfinucan mdbooth: Done (all trivial reapprovals bar the first). You just need to rope someone like japyipes in now :)
17:44:32 openstackgerrit Balazs Gibizer proposed openstack/nova: Transform aggregate.create notification https://review.openstack.org/394511
17:45:13 openstackgerrit Balazs Gibizer proposed openstack/nova: Transform aggregate.delete notification https://review.openstack.org/394512
17:45:40 openstackgerrit Balazs Gibizer proposed openstack/nova: Add aggregate notification related enum values https://review.openstack.org/396209
17:45:53 openstackgerrit Balazs Gibizer proposed openstack/nova: Transform aggregate.add_host notification https://review.openstack.org/396210
17:46:04 openstackgerrit Balazs Gibizer proposed openstack/nova: Transform aggregate.remove_host notification https://review.openstack.org/396211
17:53:05 johnthetubaguy gibi: I wasn't sure about this one, I am curious about your thoughts around that: https://review.openstack.org/#/c/408689/
18:09:46 sean-k-mooney mriedem: QQ when https://review.openstack.org/#/c/411988/ is merged to relese the new version of os-vif will it automatically update the requirements repo or should i submit a patch to the requiremets repo to do that?
18:10:48 openstackgerrit Dan Smith proposed openstack/nova: Multicell support for instance listing https://review.openstack.org/396775
18:10:48 openstackgerrit Dan Smith proposed openstack/nova: Fix non-parameterized service id in hypervisors sample tests https://review.openstack.org/410471
18:10:49 openstackgerrit Dan Smith proposed openstack/nova: Make the SingleCellSimple fixture a little more comprehensive https://review.openstack.org/410472
18:10:49 openstackgerrit Dan Smith proposed openstack/nova: WIP: Make api_samples tests use simple cell environment https://review.openstack.org/410473
18:10:53 openstackgerrit Dan Smith proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379
18:11:40 mriedem sean-k-mooney: you have to submit a patch to raise the minimum required version of os-vif in global-requirements yourself
18:12:02 mriedem the upper-constraints update happens automatically, but if you're going to bump g-r you might as well bump u-c in the same patch - i think you'd have to anyway for the tests to pass
18:13:37 sean-k-mooney mriedem: thanks. im going to follow up with sfinucan again tomorow and see if we can get his outstanding patch merged so that the release can go ahead.
18:14:06 sean-k-mooney ill update the patche to the release repo and to the requirements repo once that is done
18:14:14 sean-k-mooney sorry for the delay
18:14:22 mriedem sean-k-mooney: cool i was going to ask you if his change at the bottom of the series was really required or not
18:15:06 sean-k-mooney we could release without it but he is fixing an issue on an inplace upgrade where the mtu chages.
18:15:23 sean-k-mooney it would be nice to have but not strictly required.
18:15:54 mriedem ok we still have some time
18:17:58 sean-k-mooney i assume the os-vif versions are fix on stable branch correct. e.g. they cannot be raised after the sable/branch is made ?
18:18:49 mriedem sean-k-mooney: correct
18:19:00 mriedem well the minimum can't
18:19:58 sean-k-mooney and uppper contratiants ? i assume they are also fixed or can the be extended if the new version is backwards compatible?
18:21:51 sean-k-mooney we dont need to extend them currently i just wanted to know if we had a reason too could we in the future.
18:23:05 openstackgerrit Sujitha proposed openstack/nova: Document testing process for zero downtime upgrade https://review.openstack.org/395269
18:23:59 mriedem sean-k-mooney: i don't think upper-constraints is auto-updated on stable branches
18:24:15 mriedem we can bump then in the future on stable manually if needed to get like a bug fix or something
18:31:28 mriedem pretty easy hyperv bp patch that needs another +2 here https://review.openstack.org/#/c/248029/
18:35:35 openstackgerrit Christopher Martin proposed openstack/nova: Remove mox from unit/compute/test_compute_api.py (1) https://review.openstack.org/334841
18:35:41 openstackgerrit Sujitha proposed openstack/nova: Adding tags field to InstancePayload https://review.openstack.org/407228
18:38:32 openstackgerrit Chris Dent proposed openstack/nova: [placement] Add an "Adding a Handler" section to placement_dev https://review.openstack.org/416304
18:38:32 openstackgerrit Chris Dent proposed openstack/nova: [placement] placement_dev info for testing and gabbi https://review.openstack.org/412600
18:38:33 openstackgerrit Chris Dent proposed openstack/nova: [placement] placement_dev info for microversion handling https://review.openstack.org/411946
18:38:33 openstackgerrit Chris Dent proposed openstack/nova: [placement] Add a bit about extraction plans to placement_dev https://review.openstack.org/416676
18:45:32 openstackgerrit Julian proposed openstack/nova: Transform keypair.create notification https://review.openstack.org/408689
18:47:03 openstackgerrit Ed Leafe proposed openstack/nova: Do not post allocations that are zero https://review.openstack.org/407180
18:56:35 sean-k-mooney it would be nice to see https://review.openstack.org/#/c/140045/52 merged also. that patch has been around for a while.
18:57:19 mriedem sean-k-mooney: yeah, it's a bit bigger but i'm good with it now
18:57:49 oomichi dansmith: melwitt: could you take a look at https://review.openstack.org/#/c/411816? I am not a specialist at the area and nice to get your reviews I think
18:58:18 sean-k-mooney i actully taught the hyperv os-vif support was already in place in newton but i guess it missed the cut off.
18:58:19 mriedem edleafe: is this what your patch was handling? http://logs.openstack.org/21/386221/3/check/gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial/14df095/logs/screen-n-cpu.txt.gz#_2017-01-04_12_16_04_616
18:58:22 dansmith oomichi: I'm not really either but I'll look
18:58:27 mriedem edleafe: on the ceph jobs we send 0 for disk allocatoins
18:58:30 mriedem which placement hates
18:59:17 mriedem edleafe: actually i guess https://review.openstack.org/#/c/414214/ should handle that for the ceph job
18:59:51 oomichi dansmith: thanks :)
19:01:09 edleafe mriedem: yep
19:01:21 mriedem edleafe: actually no it looks like
19:01:22 mriedem http://logs.openstack.org/14/414214/12/check/gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial/511c30e/logs/screen-n-cpu.txt.gz#_2017-01-03_23_17_44_750
19:01:29 mriedem ^ is the ceph job run on jaypipes' patch
19:01:43 mriedem Sending allocation for instance {'allocations': [{'resource_provider': {'uuid': 'f8ce7899-0b48-4294-8df2-057ac325f4d2'}, 'resources': {'MEMORY_MB': 64, 'VCPU': 1, 'DISK_GB': 0}}]}
19:01:49 edleafe mriedem: ok gimme a few - on a call now
19:02:37 mriedem oh that's allocations,
19:02:40 mriedem jay fixed inventories
19:03:13 jaypipes ya
19:03:20 mriedem ok looking at https://review.openstack.org/#/c/407180/ now
19:03:20 jaypipes edleafe fixed allocs.
19:10:25 mriedem would be cool if we could define ranges on ovo integer fields like we can for IntOpts in oslo.config
19:11:29 openstackgerrit Jay Pipes proposed openstack/nova: Return uuid attribute for aggregates https://review.openstack.org/415031
19:11:56 sean-k-mooney mriedem: if ovo cant do it as an integer type field im sure a bounded integer field could be introduced if needed
19:12:27 sean-k-mooney mriedem: probably not this cycle though
19:14:20 mriedem jaypipes: replied in https://review.openstack.org/#/c/415031/7/nova/tests/functional/api_sample_tests/test_aggregates.py - i think what alex was asking for is redundant
19:14:23 mriedem which is why i removed it
19:16:44 openstackgerrit Rafael Folco proposed openstack/nova: DELETE all inventory for a resource provider https://review.openstack.org/416669
19:28:00 mriedem cdent: edleafe: got some issues in https://review.openstack.org/#/c/407180/ - for whenever
19:30:12 jaypipes mriedem: oh, I figured you removed it because you found a bug in using a regex substitution for uuids.
19:30:34 mriedem nope it was just confusing me and was redundant
19:31:01 mriedem we can't actually compare the response to a known uuid in those tests without hacking the aggregate create db method
19:31:03 mriedem to use a static uuid
19:31:07 mriedem and i didn't want to do that in those tests
19:31:32 mriedem i guess we could for some of the other tests that build on an existing created aggregate where we already know the uuid
19:33:43 jaypipes mriedem: I had copied that subs['uuid'] stuff from test_servers.py :)
19:34:10 mriedem like most tihngs in api samples tests, who knows, things are cludged until they pass

Earlier   Later