Posted	Nick	Remark
#openstack-nova - 2018-01-06
02:04:01	openstackgerrit	melanie witt proposed openstack/nova master: Add ConsoleAuthToken object https://review.openstack.org/320063
02:04:01	openstackgerrit	melanie witt proposed openstack/nova master: Optionalize instance_uuid in console_auth_token_get_valid() https://review.openstack.org/481700
02:04:02	openstackgerrit	melanie witt proposed openstack/nova master: Use ConsoleAuthToken object to generate authorizations https://review.openstack.org/325414
02:04:02	openstackgerrit	melanie witt proposed openstack/nova master: Add periodic task to clean expired console tokens https://review.openstack.org/325381
02:04:03	openstackgerrit	melanie witt proposed openstack/nova master: Convert websocketproxy to use db for token validation https://review.openstack.org/333990
12:51:48	openstackgerrit	Merged openstack/nova master: manager: more detailed info of unsupported compute driver https://review.openstack.org/530701
14:44:46	openstackgerrit	Matt Riedemann proposed openstack/python-novaclient master: Microversion 2.58 - Instance actions list pagination https://review.openstack.org/528601
15:06:07	openstackgerrit	Hongbin Lu proposed openstack/nova master: Update FAQs about listing hosts in cellv2 https://review.openstack.org/531559
15:08:29	openstackgerrit	Hongbin Lu proposed openstack/nova master: Update FAQs about listing hosts in cellv2 https://review.openstack.org/531559
15:40:48	openstackgerrit	Merged openstack/nova master: Add access_url_base to console_auth_tokens table https://review.openstack.org/334614
19:49:53	openstackgerrit	Jackie Truong proposed openstack/nova master: Add trusted_certs object https://review.openstack.org/489408
19:56:46	openstackgerrit	Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
#openstack-nova - 2018-01-07
04:22:32	ninja1WHXFQ	▄▄▄▄▄▄▄▄▄▄▄▄▄▄ LRH IS LIVE NOW!! TODAYS EDITION SLIMER GETS FUCKED IN VEGAS!! https://www.youtube.com/user/l0de/live CALL 315-505-4666 tignww: purplerbot alex_xu_ sapcc-bot ▄▄▄▄▄▄▄▄▄▄
04:22:32	ninja1WHXFQ	▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄ LRH IS LIVE NOW!! TODAYS EDITION SLIMER GETS FUCKED IN VEGAS!! https://www.youtube.com/user/l0de/live CALL 315-505-4666 ruyjey: frickler rm_work purplerbot ▄▄▄▄▄▄▄▄▄▄▄▄▄▄
10:59:46	openstackgerrit	Jie Li proposed openstack/nova master: Support volume-backed server rescue https://review.openstack.org/531524
11:39:50	openstackgerrit	Andreas Jaeger proposed openstack/nova master: Fix OpenStack capitalization https://review.openstack.org/531593
13:32:10	openstackgerrit	Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.openstack.org/526900
13:43:08	Rambo	Good mornig,everyone, Can you help me to review it ?The link is https://review.openstack.org/#/c/531524/
13:43:35	Rambo	Another is :https://review.openstack.org/#/c/528740/
13:43:50	Rambo	Thank you very much
15:08:44	mriedem	ildikov: http://logs.openstack.org/86/531386/5/check/tempest-full/9531761/job-output.txt.gz#_2018-01-06_19_38_04_914985
15:15:25	ildikov	mriedem: you mean no more errors anywhere? :)
15:16:21	ildikov	mriedem: and thank you of course!
15:18:01	mriedem	didn't check the n-cpu logs but the test passed
15:26:07	ildikov	mriedem: this looks a little scary: Do NOT duplicate iSCSI InitiatorNames
15:45:12	ildikov	I also saw this with the second detach: Couldn't find iSCSI nodes because iscsiadm err: iscsiadm: No records found
15:46:15	ildikov	so we might still have issues with lvm detach or I just freaked out by an irrelevant line
21:31:42	mriedem	ildikov: would have to ask hemna probably, i think os-brick might just log some stuff that is normal if a thing it's trying to remove is already gone
21:31:49	mriedem	i don't know about the duplicate initiator names
22:32:09	ildikov	mriedem: we had a similar log message before with the second detach, but here it looks like a simple debug log
22:32:34	ildikov	mriedem: I just instinctively freaked out about it...
22:32:46	ildikov	Will ask around tomorrow
#openstack-nova - 2018-01-08
01:54:04	Rambo	hello,everyone
01:54:29	Rambo	Can you help me to review it ?The link is https://review.openstack.org/#/c/531524/
01:54:50	Rambo	Another is :https://review.openstack.org/#/c/528740/
01:54:58	Rambo	Thank you very much
01:55:35	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: [DNM] add uuid in server migration https://review.openstack.org/531633
02:17:42	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Add index(updated_at) on migrations table. https://review.openstack.org/531132
02:21:47	alex_xu_	yikun: sorry for reply late https://review.openstack.org/#/c/330406/78/api-ref/source/parameters.yaml@4148
02:54:04	yikun	alex_xu_, ok, thanks, saw it, I will update and add the uuid on server migrations in this patch.
03:00:40	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Add index(instance_uuid, updated_at) on instance_actions table https://review.openstack.org/530429
03:00:40	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Fix 500 in test_resize_server_negative_invalid_state https://review.openstack.org/531117
03:11:05	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: [DNM] add uuid in server migration https://review.openstack.org/531633
03:13:17	yikun	^ I submit the separate one, just for convenient reviewing, and if all is ok, I will merge it into parent patch.
03:54:42	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Add migration db and object pagination support. https://review.openstack.org/514904
03:54:43	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406
03:54:43	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Add cross cell sort support for get_migrations https://review.openstack.org/517273
03:54:44	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: [DNM] add uuid in server migration https://review.openstack.org/531633
04:43:00	gcb	alex_xu_, could you please help review https://review.openstack.org/528620 a simple adoption of oslo function
04:43:39	alex_xu_	gcb: will check it after lunch
04:45:03	gcb	alex_xu_: thanks
04:45:10	alex_xu_	gcb: np
05:44:16	openstackgerrit	Rajesh Tailor proposed openstack/nova master: Allow force-delete even if task_state is not None https://review.openstack.org/530879
05:56:46	openstackgerrit	Kevin Zhao proposed openstack/nova master: Modify the test case of get_disk_mapping_rescue_with_config https://review.openstack.org/494156
06:10:19	openstackgerrit	Yikun Jiang (Kero) proposed openstack/python-novaclient master: Microversion 2.58 - Instance actions list pagination https://review.openstack.org/528601
06:35:23	openstackgerrit	Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.openstack.org/526900
06:37:18	openstackgerrit	Kevin Zhao proposed openstack/nova master: libvirt: use 'host-passthrough' as default on AArch64 https://review.openstack.org/530965
06:52:26	bhagyashris	ianw: Hi, are you around ?
07:24:36	openstackgerrit	Merged openstack/nova master: Use method validate_integer from oslo.utils https://review.openstack.org/528620
07:51:58	openstackgerrit	Sriharsha Basavapatna proposed openstack/os-vif master: Fix VF-rep lookup routine to use parent PF number https://review.openstack.org/531674
08:03:55	hrw	morning
08:35:04	openstackgerrit	Mr Rambo proposed openstack/nova master: Support volume-backed server rebuild https://review.openstack.org/528740
08:44:23	bauzas	good morning Nova
08:44:37	gibi_away	good morning bauzas and Nova
08:48:30	hrw	speadup zuul, speed up...
09:08:47	bauzas	just in case people wanna give me a Christmas present, here is a series loving reviews https://review.openstack.org/#/c/514763/
09:08:49	bauzas	:)
09:19:55	openstackgerrit	Kevin Zhao proposed openstack/nova master: libvirt: use 'host-passthrough' as default on AArch64 https://review.openstack.org/530965
09:24:47	openstackgerrit	Mr Rambo proposed openstack/nova master: Support volume-backed server rebuild https://review.openstack.org/528740
09:50:38	ameeda	hello
09:51:10	ameeda	can you please review my code here https://review.openstack.org/#/c/528069/ ? and let me know if I have to consider force_delete and soft_delete
10:16:51	openstackgerrit	Lajos Katona proposed openstack/nova master: WIP: Add nested resources to server moving tests https://review.openstack.org/527728
10:17:36	openstackgerrit	Jie Li proposed openstack/nova master: Support volume-backed server rescue https://review.openstack.org/531524
10:22:27	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/os-vif master: Migrate from 'ip' commands to 'pyroute2' https://review.openstack.org/484386
10:48:49	openstackgerrit	chenpengzi proposed openstack/nova master: For a pre-generated example of the latest policy.yaml https://review.openstack.org/531713
11:10:27	openstackgerrit	chenpengzi proposed openstack/nova master: For a pre-generated example of the latest policy.yaml https://review.openstack.org/531713
11:41:28	openstackgerrit	chenpengzi proposed openstack/nova master: Add reference to policy sample https://review.openstack.org/531713
11:45:56	openstackgerrit	Jianle He proposed openstack/nova-specs master: Back up volume-backed-instance https://review.openstack.org/530214
12:09:26	gibi	bauzas: left some questions / nits in https://review.openstack.org/#/c/514763/ Merry Christmas! ;)
12:14:09	openstackgerrit	Merged openstack/nova master: Remove 'nova-manage host' and 'nova-manage agent' https://review.openstack.org/521837
12:20:12	ameeda	bauzas: can you please review my code here https://review.openstack.org/#/c/526900/
12:45:37	hrw	stephenfin: https://review.openstack.org/#/c/530965/ got +1 from zuul, has test coverage. can you +2 again?
12:47:01	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/531737
12:47:52	ameeda	Hello
12:48:10	ameeda	FutureWarning: fake is an invalid UUID. Using UUIDFields with invalid UUIDs is no longer supported, and will be removed in a future release. Please update your code to input valid UUIDs or accept ValueErrors for invalid UUIDs. See https://docs.openstack.org/oslo.versionedobjects/latest/reference/fields.html#oslo_versionedobjects.fields.UUIDField for further details
12:48:10	ameeda	I got this error when I run tox -e py27
12:48:23	ameeda	any idea ?
12:49:27	sean-k-mooney	ameeda: when using oslo versioned objects the validator for uuid fields check that the value is really a uuid. fake is not a uuid hence the error
12:49:44	sean-k-mooney	ameeda: is that from https://review.openstack.org/#/c/526900
12:51:51	ameeda	sean-k-mooney: thanks for reply, no its from this https://review.openstack.org/#/c/528385/ when I move my code to _cleanup_volumes and modify the unit test for that
12:53:40	ameeda	sean-k-mooney: here is my edit http://paste.openstack.org/show/640858/
12:53:41	sean-k-mooney	oh ok you can use uuidgen on the commandline to generate a uuid for the tests for example e21d0851-3c56-4b9d-a133-a2c1ba59778a whcich you can use instead of fake to resolve the warning
12:54:49	ameeda	sean-k-mooney: so I should use this uuid inside my code ? since the error appears in another file
12:56:03	sean-k-mooney	only if its cause by your code. if the warning is present without your patch you dont need to fix it.
12:56:09	ktibi	Hi, do you think it's good if nova doesn't check if neutron have free IP before spawn instance ?
12:57:02	ktibi	because I have a strange case where user spawn 200 vms with 50 free IP, and all my compute become disabled with "consecutive_build_service_disable_threshold"
12:57:46	sean-k-mooney	ameeda: oslo versioned objects uuid filed not emit this warning if invalid uuids are used but at present its only a warning to allow projects to fix there tests. in the future it will be changed to an error but only once most project have updated there tests
12:58:55	ameeda	sean-k-mooney: when I remove my code, the test success completely, and the error appears at another file where I add my code
12:59:53	sean-k-mooney	ktibi: strictly speaking nova has no requirement that a vm will have an ip. it just requires that vms have mac adressses.
13:00:11	ameeda	sean-k-mooney: if I do this http://paste.openstack.org/show/640870/ it will success ?

Earlier   Later