Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-07-16
19:04:49 mriedem trump is calling me in to smooth things over
19:08:54 efried mriedem: Does that fancy `oslo.config`:section.opt syntax work from within conf option help strings?
19:09:06 efried mriedem: (and remind me what the exact syntax actually is?)
19:10:41 efried looks like :oslo.config:option:`section.opt`
19:12:20 openstackgerrit Andreas Jaeger proposed openstack/nova master: Move legacy-tempest-dsvm-nova-os-vif in repo https://review.openstack.org/583023
19:14:44 openstackgerrit Andreas Jaeger proposed openstack/nova master: Move legacy-tempest-dsvm-nova-os-vif in repo https://review.openstack.org/583023
19:16:12 openstackgerrit Eric Fried proposed openstack/nova master: Conf option cross-reference in config docs https://review.openstack.org/583025
19:17:32 efried We shall see ^
19:22:19 mriedem efried: maybe? but it wouldn't make any sense in the rendered sample
19:22:37 mriedem nor does any rst fanciness really
19:22:43 efried mriedem: Huh? It would allow you to jump from one opt's doc to another, wouldn't it?
19:22:54 mriedem efried: i'm talking about the literall sample file you can download
19:23:05 mriedem https://docs.openstack.org/nova/latest/configuration/sample-config.html
19:23:23 efried mriedem: Oh. Well, it makes just as much sense as [section]/optname. And it has the advantage of cross-referencing in the HTML doc (if that works).
20:00:14 efried mriedem: The references work fine in the HTML. Are you asserting that http://paste.openstack.org/show/726048/ is unacceptable for the nova.conf.sample?
20:03:38 openstackgerrit Merged openstack/nova master: Add UUID validation for consumer_uuid https://review.openstack.org/580373
20:04:06 mriedem i'm not asserting it's unacceptable
20:04:08 mriedem i'm a big meh
20:09:53 efried mriedem: okay. I'm enough in favor of those links working that I'm willing to go through and convert as many of them as I can find. I'm also looking into oslo-config-generator to see if I can figure out how to make them render as [section]/optname in the sample, so everybody happy.
20:20:33 openstackgerrit Merged openstack/nova master: network: Retrieve tunneled status in '_get_physnet_info' https://review.openstack.org/564445
20:20:48 openstackgerrit Rick Bartra proposed openstack/nova master: Add granularity to os-services API https://review.openstack.org/578553
21:24:05 openstackgerrit Merged openstack/nova master: Update HostState.instances during _consume_selected_host https://review.openstack.org/582976
21:31:37 openstackgerrit Matt Riedemann proposed openstack/nova master: Default AZ for instance if cross_az_attach=False and checking from API https://review.openstack.org/469675
21:38:12 openstackgerrit MultipleCrashes proposed openstack/nova master: Retry decorator fix for instances which go into ERROR state during bulk delete https://review.openstack.org/563418
21:42:35 melwitt sean-k-mooney, efried: this thursday is non-client library release freeze (os-vif, os-traits) so I was planning to propose releases for those anyway if we have stuff to release, fwiw
21:42:45 efried cool
21:44:09 melwitt if one of y'all propose it when it's ready, I will ack it. else I will propose it myself on thurs with whatever has merged by then
22:14:52 sean-k-mooney melwitt: i could pretend to not see that till morning but since im using my personal desktop and laptop for work temporially i have..
22:15:11 sean-k-mooney melwitt: https://review.openstack.org/#/c/559398/4 and the change below it are the two pending changes
22:15:22 melwitt sean-k-mooney: ack, thanks
22:15:36 sean-k-mooney i was going to ask stephenfin to look over them tomorow to make sure the docs are fine.
22:15:48 melwitt sounds like a good plan
22:16:15 sean-k-mooney but ya once he takes a look ill propose the change to the release repo and tag you as a review.
22:16:23 melwitt coolness
22:32:45 openstackgerrit Merged openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240
22:44:01 openstackgerrit Eric Fried proposed openstack/nova master: Conf option cross-reference in config docs https://review.openstack.org/583025
23:50:21 openstackgerrit Yikun Jiang (Kero) proposed openstack/python-novaclient master: Microversion 2.64 - Use new format policy in server group https://review.openstack.org/578261
#openstack-nova - 2018-07-17
00:00:29 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (5) https://review.openstack.org/570842
00:02:19 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (6) https://review.openstack.org/571330
00:07:41 openstackgerrit Yikun Jiang (Kero) proposed openstack/python-novaclient master: Microversion 2.64 - Use new format policy in server group https://review.openstack.org/578261
00:35:33 openstackgerrit Merged openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340
00:46:18 openstackgerrit Merged openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344
00:46:31 openstackgerrit Merged openstack/nova master: Adjust log style and remove ocata support https://review.openstack.org/576462
00:46:47 openstackgerrit Merged openstack/nova master: fix cellv2 delete_host https://review.openstack.org/582171
00:54:37 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (7) https://review.openstack.org/571992
00:55:18 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (8) https://review.openstack.org/571993
02:15:47 openstackgerrit Takashi NATSUME proposed openstack/nova master: Transform volume.usage notification https://review.openstack.org/580345
02:36:32 openstackgerrit Takashi NATSUME proposed openstack/nova master: Transform libvirt.error notification https://review.openstack.org/484851
03:01:50 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add the guideline to write API reference https://review.openstack.org/569058
03:20:56 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Remove "DEPRECATED" tag from Obsolete APIs https://review.openstack.org/583092
04:36:11 vishakha takashin, pl review https://review.openstack.org/#/c/580271/
04:36:52 takashin vishakha: Okay. I will.
04:40:33 openstackgerrit MultipleCrashes proposed openstack/nova master: Retry decorator fix for instances which go into ERROR state during bulk delete https://review.openstack.org/563418
04:40:34 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Remove "DEPRECATED" tag from Obsolete APIs https://review.openstack.org/583092
04:42:31 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Remove "DEPRECATED" tag from Obsolete APIs https://review.openstack.org/583092
05:17:24 mvpnitesh hi all, i've installed Pike release of Openstack, i'm trying to create a instance , i'm getting this error "VirtualInterfaceCreateException: Virtual Interface creation failed" and "BuildAbortException: Build of instance d8a4eb19-533d-4a03-aaa6-6adec81030b1 aborted: Failed to allocate the network(s), not rescheduling."
05:19:35 mvpnitesh Can anyone tel me, what i'm missing
06:23:26 openstackgerrit Merged openstack/nova master: Address nits in server group policy series https://review.openstack.org/581616
06:31:04 openstackgerrit jichenjc proposed openstack/nova master: Enhance doc to guide user to use nova user https://review.openstack.org/583115
07:38:58 openstackgerrit Yikun Jiang (Kero) proposed openstack/python-novaclient master: Microversion 2.64 - Use new format policy in server group https://review.openstack.org/578261
08:42:18 tssurya good morning gibi, whenever you get time could you have a look at https://review.openstack.org/#/c/581243/ ? Its just addressing the nits in the main cell down spec
08:42:42 tssurya thanks in advance :)
08:44:30 gibi tssurya: on a meeting, will check the spec after it
08:44:42 tssurya no problem, thank you
09:22:25 stephenfin gibi: It's trivial, but could you take a look at https://review.openstack.org/#/c/481304/ ?
09:40:14 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge server create for scheduler hint extension https://review.openstack.org/579067
09:41:32 gibi tssurya: https://review.openstack.org/#/c/581243/ looks good to me, but I don't have +2 rights in nova-specs so you have to ping somebody from https://review.openstack.org/#/admin/groups/302,members get it merged
09:41:35 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge server create for multiple-create extension https://review.openstack.org/580017
09:41:59 gibi stephenfin: looking
09:42:07 tssurya gibi: thanks a lot!
09:48:02 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge server create for scheduler hint extension https://review.openstack.org/579067
09:49:23 gibi stephenfin: +2
09:49:41 stephenfin gibi: Ta. I'll drag efried in too when he's about
09:51:31 openstackgerrit Stephen Finucane proposed openstack/nova master: scheduler: Start utilizing RequestSpec.network_metadata https://review.openstack.org/564452
09:51:32 openstackgerrit Stephen Finucane proposed openstack/nova master: Assorted cleanups from numa-aware-vswitches series https://review.openstack.org/582651
10:53:12 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge server create for multiple-create extension https://review.openstack.org/580017
11:03:55 openstackgerrit Merged openstack/os-vif master: doc: Fix arg specs and object types in docs https://review.openstack.org/575151
11:19:47 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge server create for keypair extension https://review.openstack.org/583203
11:26:53 openstackgerrit Balazs Gibizer proposed openstack/nova master: Escalate UUID validation warning to error in test https://review.openstack.org/540386
11:26:54 openstackgerrit Balazs Gibizer proposed openstack/nova master: Fix a newly introduced UUID warning in the unit test https://review.openstack.org/583205
11:27:38 gibi efried, stephenfin: I pushed the fix of the unit test in a separate patch and rebased the UUID warning -> error patch ^^
11:40:56 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Address nit in 79dac41fee178dabb547f4d7bc10609630767131 https://review.openstack.org/583212
12:08:55 sean-k-mooney stephenfin: just thinking about the os-vif lower-constriants job. if i add the docs requirements too it nothing will test it unless i also do a sphinx build. should i add that too?
12:10:35 stephenfin sean-k-mooney: Hmm, I'm not sure about that. Maybe we should ask dhellmann when he's about?
12:12:43 sean-k-mooney i have made the change localy to run the unit tests then the docs.
12:13:07 sean-k-mooney im going to force a unit test to fail to make sure it will fail if either fails
12:15:30 sean-k-mooney yep it does and it bails out without running the docs job which saves time
12:23:58 openstackgerrit sean mooney proposed openstack/os-vif master: convert os-vif docs to follow PTI https://review.openstack.org/559398
12:31:22 openstackgerrit Neha Alhat proposed openstack/python-novaclient master: Remove initialization of logger if logger is None https://review.openstack.org/581218
13:00:49 efried stephenfin, gibi: What am I looking at?
13:01:04 stephenfin efried: https://review.openstack.org/#/c/481304/
13:02:09 gibi efried: and after that, this is also fixed: https://review.openstack.org/#/c/583205 :)
13:02:39 efried stephenfin: +A
13:02:42 efried gibi: looking...
13:02:47 stephenfin efried++
13:04:01 efried gibi: +A on both; if zuul is happy with them, I'm happy :)
13:04:13 gibi efried: I'm happy :) thanks
13:09:31 jaypipes stephenfin: I have absolutely zero clue what is best practice on https://review.openstack.org/#/c/574539/. whatever is your best practice/best knowledge, go with that.
13:11:39 stephenfin jaypipes: Cool. It seems to have been removed from os-vif so we should kill it there too
13:11:57 stephenfin I'll tackle that today so we do so before non-client library freeze

Earlier   Later