*** jmlowe has quit IRC | 00:03 | |
*** tbachman has quit IRC | 00:09 | |
*** Sukhdev has joined #openstack-nova | 00:15 | |
*** john5223_ has joined #openstack-nova | 00:16 | |
*** hongbin has quit IRC | 00:23 | |
*** gbarros has joined #openstack-nova | 00:27 | |
*** edmondsw has quit IRC | 00:27 | |
*** edmondsw has joined #openstack-nova | 00:28 | |
*** edmondsw has quit IRC | 00:32 | |
*** gbarros has quit IRC | 00:32 | |
*** salv-orlando has joined #openstack-nova | 00:38 | |
*** tbachman has joined #openstack-nova | 00:40 | |
*** vladikr has quit IRC | 00:41 | |
*** andreas_s has joined #openstack-nova | 00:42 | |
*** qsyqian has joined #openstack-nova | 00:43 | |
*** qsyqian has quit IRC | 00:43 | |
*** salv-orlando has quit IRC | 00:43 | |
*** andreas_s has quit IRC | 00:47 | |
*** edmondsw has joined #openstack-nova | 00:51 | |
*** sdague has quit IRC | 00:55 | |
*** sree has joined #openstack-nova | 00:55 | |
*** edmondsw has quit IRC | 00:55 | |
*** Nil_ has quit IRC | 00:56 | |
*** cheneydc has joined #openstack-nova | 00:57 | |
*** TuanLA has joined #openstack-nova | 00:59 | |
*** sree has quit IRC | 00:59 | |
*** phuongnh has joined #openstack-nova | 01:06 | |
*** ankit has joined #openstack-nova | 01:06 | |
*** moshele has joined #openstack-nova | 01:13 | |
*** yassine has joined #openstack-nova | 01:21 | |
*** yassine is now known as Guest95810 | 01:22 | |
*** gongysh has joined #openstack-nova | 01:35 | |
*** tetsuro has joined #openstack-nova | 01:38 | |
*** salv-orlando has joined #openstack-nova | 01:39 | |
openstackgerrit | Yikun Jiang proposed openstack/nova master: Implement query param schema for migration index https://review.openstack.org/518644 | 01:40 |
---|---|---|
*** sree has joined #openstack-nova | 01:42 | |
*** gcb has joined #openstack-nova | 01:42 | |
*** eharney has quit IRC | 01:42 | |
*** salv-orlando has quit IRC | 01:44 | |
*** Apoorva_ has joined #openstack-nova | 01:44 | |
*** sree has quit IRC | 01:46 | |
*** Apoorva has quit IRC | 01:48 | |
*** Apoorva_ has quit IRC | 01:49 | |
*** moshele has quit IRC | 01:50 | |
*** mriedem has quit IRC | 01:54 | |
*** moshele has joined #openstack-nova | 01:56 | |
*** artom_ has joined #openstack-nova | 02:01 | |
*** sbezverk has quit IRC | 02:03 | |
*** moshele has quit IRC | 02:07 | |
*** READ10 has joined #openstack-nova | 02:08 | |
*** sbezverk has joined #openstack-nova | 02:09 | |
*** Tom has joined #openstack-nova | 02:12 | |
*** Tom is now known as Guest95794 | 02:12 | |
*** Guest95794 is now known as Tommy_ | 02:12 | |
*** Tommy_ is now known as Guest74410 | 02:13 | |
*** namnh has joined #openstack-nova | 02:13 | |
*** takashin has joined #openstack-nova | 02:13 | |
*** Guest74410 has quit IRC | 02:13 | |
*** Tom-Tom has joined #openstack-nova | 02:13 | |
openstackgerrit | licanwei proposed openstack/nova master: cli 'nova-manage db sync' can't upgrade cell1 https://review.openstack.org/519275 | 02:15 |
*** annp has joined #openstack-nova | 02:16 | |
*** links has joined #openstack-nova | 02:17 | |
*** vladikr has joined #openstack-nova | 02:21 | |
*** sapd__ has joined #openstack-nova | 02:24 | |
*** sapd_ has quit IRC | 02:24 | |
*** bkopilov has quit IRC | 02:25 | |
*** tetsuro has quit IRC | 02:29 | |
*** vladikr has quit IRC | 02:30 | |
*** yamahata has quit IRC | 02:30 | |
*** Shunli has joined #openstack-nova | 02:31 | |
*** sapcc-bot has quit IRC | 02:35 | |
*** READ10 has quit IRC | 02:35 | |
*** sapcc-bot has joined #openstack-nova | 02:35 | |
*** salv-orlando has joined #openstack-nova | 02:40 | |
openstackgerrit | Eric Fried proposed openstack/nova master: finish refactor AllocCandidates._get_by_filters() https://review.openstack.org/516782 | 02:43 |
openstackgerrit | Eric Fried proposed openstack/nova master: required traits for no sharing providers https://review.openstack.org/517027 | 02:43 |
openstackgerrit | Eric Fried proposed openstack/nova master: Add ProviderSummary.resource_class_names @property https://review.openstack.org/517646 | 02:43 |
openstackgerrit | Eric Fried proposed openstack/nova master: handle traits with sharing providers https://review.openstack.org/517119 | 02:43 |
openstackgerrit | Eric Fried proposed openstack/nova master: Test alloc_cands with non overlapping sharing RPs https://review.openstack.org/519380 | 02:43 |
openstackgerrit | Yikun Jiang proposed openstack/nova master: Implement query param schema for migration index https://review.openstack.org/518644 | 02:43 |
*** salv-orlando has quit IRC | 02:44 | |
*** gyee_ has quit IRC | 02:47 | |
*** hongbin has joined #openstack-nova | 02:53 | |
*** ankit has quit IRC | 02:55 | |
*** READ10 has joined #openstack-nova | 02:55 | |
*** sree has joined #openstack-nova | 03:02 | |
*** fragatina has quit IRC | 03:03 | |
*** fragatina has joined #openstack-nova | 03:03 | |
*** sree has quit IRC | 03:06 | |
*** fragatin_ has joined #openstack-nova | 03:07 | |
*** fragatina has quit IRC | 03:07 | |
*** fragatin_ has quit IRC | 03:11 | |
openstackgerrit | Yikun Jiang proposed openstack/nova master: Extract SortContext/SortWrapper of instance_list into utils https://review.openstack.org/518871 | 03:17 |
*** sapd_ has joined #openstack-nova | 03:17 | |
*** sapd__ has quit IRC | 03:18 | |
*** gbarros has joined #openstack-nova | 03:18 | |
openstackgerrit | Yikun Jiang proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406 | 03:18 |
openstackgerrit | Yikun Jiang proposed openstack/nova master: Add cross cell sort support for get_migrations https://review.openstack.org/517273 | 03:18 |
*** itlinux has joined #openstack-nova | 03:23 | |
openstackgerrit | Yikun Jiang proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406 | 03:24 |
*** fragatina has joined #openstack-nova | 03:25 | |
*** Shunli has quit IRC | 03:27 | |
*** takashin has quit IRC | 03:27 | |
*** takashin has joined #openstack-nova | 03:28 | |
*** hemna_ has quit IRC | 03:29 | |
*** READ10 has quit IRC | 03:29 | |
*** fragatina has quit IRC | 03:29 | |
*** trungnv has quit IRC | 03:31 | |
*** msno has joined #openstack-nova | 03:35 | |
*** salv-orlando has joined #openstack-nova | 03:41 | |
*** akekane has joined #openstack-nova | 03:41 | |
*** akekane is now known as abhishekk | 03:41 | |
*** trungnv has joined #openstack-nova | 03:43 | |
*** salv-orlando has quit IRC | 03:45 | |
*** gongysh has quit IRC | 03:46 | |
*** bkopilov has joined #openstack-nova | 03:48 | |
*** yamamoto has joined #openstack-nova | 03:52 | |
*** lei-zh has joined #openstack-nova | 03:52 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | 03:56 |
*** Apoorva has joined #openstack-nova | 03:56 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608 | 03:57 |
*** vivsoni__ has quit IRC | 03:59 | |
*** vivsoni has joined #openstack-nova | 03:59 | |
openstackgerrit | Yikun Jiang proposed openstack/nova master: Add pagination and changes-since for instance-actions https://review.openstack.org/326326 | 04:02 |
*** itlinux has quit IRC | 04:03 | |
*** msno has quit IRC | 04:03 | |
*** udesale has joined #openstack-nova | 04:04 | |
*** itlinux has joined #openstack-nova | 04:04 | |
*** itlinux has quit IRC | 04:09 | |
*** mdnadeem has joined #openstack-nova | 04:12 | |
*** lei-zh has quit IRC | 04:17 | |
*** psachin has joined #openstack-nova | 04:18 | |
openstackgerrit | Yikun Jiang proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406 | 04:18 |
*** dave-mccowan has quit IRC | 04:19 | |
*** itlinux has joined #openstack-nova | 04:20 | |
*** andreas_s has joined #openstack-nova | 04:25 | |
*** andreas_s has quit IRC | 04:29 | |
*** psachin has quit IRC | 04:38 | |
*** salv-orlando has joined #openstack-nova | 04:41 | |
*** hieulq has quit IRC | 04:43 | |
*** TuanLA has quit IRC | 04:43 | |
*** TuanLA has joined #openstack-nova | 04:44 | |
*** hieulq has joined #openstack-nova | 04:44 | |
*** salv-orlando has quit IRC | 04:46 | |
*** psachin has joined #openstack-nova | 04:46 | |
*** TuanLA has quit IRC | 04:47 | |
*** itlinux has quit IRC | 04:52 | |
*** gbarros has quit IRC | 04:55 | |
*** hongbin has quit IRC | 05:01 | |
*** msno has joined #openstack-nova | 05:04 | |
*** tetsuro has joined #openstack-nova | 05:05 | |
*** sapd__ has joined #openstack-nova | 05:09 | |
*** sapd_ has quit IRC | 05:10 | |
*** sree has joined #openstack-nova | 05:13 | |
*** sree_ has joined #openstack-nova | 05:14 | |
*** sree_ is now known as Guest23423 | 05:14 | |
*** tbachman_ has joined #openstack-nova | 05:16 | |
*** sree has quit IRC | 05:17 | |
*** tbachman has quit IRC | 05:18 | |
*** tbachman_ is now known as tbachman | 05:18 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 05:19 |
*** sbezverk has quit IRC | 05:20 | |
*** Apoorva has quit IRC | 05:27 | |
*** yamahata has joined #openstack-nova | 05:31 | |
*** janki has joined #openstack-nova | 05:34 | |
*** diga has joined #openstack-nova | 05:40 | |
*** yangyapeng has joined #openstack-nova | 05:41 | |
*** salv-orlando has joined #openstack-nova | 05:42 | |
*** erlon has quit IRC | 05:45 | |
*** salv-orlando has quit IRC | 05:47 | |
*** moshele has joined #openstack-nova | 05:51 | |
*** msno has quit IRC | 05:54 | |
*** hshiina has joined #openstack-nova | 05:55 | |
*** ratailor has joined #openstack-nova | 06:00 | |
*** dikonoor has joined #openstack-nova | 06:01 | |
*** lpetrut_ has joined #openstack-nova | 06:01 | |
*** dikonoo has quit IRC | 06:02 | |
*** sridharg has joined #openstack-nova | 06:02 | |
*** janki has quit IRC | 06:05 | |
*** tetsuro has quit IRC | 06:05 | |
*** janki has joined #openstack-nova | 06:20 | |
*** trinaths has joined #openstack-nova | 06:25 | |
*** Guest23423 has quit IRC | 06:28 | |
*** salv-orlando has joined #openstack-nova | 06:30 | |
openstackgerrit | Merged openstack/nova master: Correct log message when removing a security group https://review.openstack.org/519313 | 06:44 |
*** yangyapeng has quit IRC | 06:46 | |
*** yangyapeng has joined #openstack-nova | 06:47 | |
*** armax has quit IRC | 06:50 | |
*** armax has joined #openstack-nova | 06:51 | |
*** sahid has joined #openstack-nova | 06:51 | |
*** armax has quit IRC | 06:51 | |
*** armax has joined #openstack-nova | 06:52 | |
*** armax has quit IRC | 06:52 | |
*** mtreinish has quit IRC | 06:55 | |
*** mtreinish has joined #openstack-nova | 06:55 | |
*** lpetrut_ has quit IRC | 07:00 | |
*** rcernin has quit IRC | 07:02 | |
*** belmoreira has joined #openstack-nova | 07:05 | |
*** edand has joined #openstack-nova | 07:08 | |
*** andreas_s has joined #openstack-nova | 07:11 | |
*** Oku_OS-away is now known as Oku_OS | 07:12 | |
*** hoonetorg has quit IRC | 07:15 | |
*** fragatina has joined #openstack-nova | 07:15 | |
*** hieulq has quit IRC | 07:16 | |
*** namnh has quit IRC | 07:16 | |
*** phuongnh has quit IRC | 07:16 | |
*** annp has quit IRC | 07:16 | |
*** annp has joined #openstack-nova | 07:16 | |
*** phuongnh has joined #openstack-nova | 07:16 | |
*** namnh has joined #openstack-nova | 07:16 | |
*** hieulq has joined #openstack-nova | 07:16 | |
openstackgerrit | Yikun Jiang proposed openstack/nova master: Add migration_get_by_uuid in db api. https://review.openstack.org/511421 | 07:18 |
openstackgerrit | Yikun Jiang proposed openstack/nova master: Add migration db and object pagination support. https://review.openstack.org/514904 | 07:18 |
openstackgerrit | Yikun Jiang proposed openstack/nova master: Implement query param schema for migration index https://review.openstack.org/518644 | 07:18 |
openstackgerrit | Yikun Jiang proposed openstack/nova master: Extract SortContext/SortWrapper of instance_list into utils https://review.openstack.org/518871 | 07:18 |
openstackgerrit | Yikun Jiang proposed openstack/nova master: Add cross cell sort support for get_migrations https://review.openstack.org/517273 | 07:18 |
openstackgerrit | Yikun Jiang proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406 | 07:18 |
*** sshwarts has joined #openstack-nova | 07:19 | |
*** fragatina has quit IRC | 07:20 | |
openstackgerrit | Nguyen Van Trung proposed openstack/nova-specs master: Add linters test to run spec validation https://review.openstack.org/519561 | 07:20 |
*** xinliang has quit IRC | 07:21 | |
openstackgerrit | Yikun Jiang proposed openstack/nova master: Implement query param schema for migration index https://review.openstack.org/518644 | 07:22 |
*** belmoreira has quit IRC | 07:23 | |
openstackgerrit | Yikun Jiang proposed openstack/nova master: Extract SortContext/SortWrapper of instance_list into utils https://review.openstack.org/518871 | 07:24 |
openstackgerrit | Yikun Jiang proposed openstack/nova master: Extract SortContext/SortWrapper of instance_list into utils https://review.openstack.org/518871 | 07:25 |
openstackgerrit | Yikun Jiang proposed openstack/nova master: Add cross cell sort support for get_migrations https://review.openstack.org/517273 | 07:25 |
openstackgerrit | Yikun Jiang proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406 | 07:26 |
*** hoonetorg has joined #openstack-nova | 07:29 | |
*** ratailor is now known as ratailor|Lu | 07:33 | |
*** ratailor|Lu is now known as ratailor|Lunch | 07:33 | |
*** xinliang has joined #openstack-nova | 07:34 | |
*** salv-orlando has quit IRC | 07:36 | |
*** salv-orlando has joined #openstack-nova | 07:36 | |
*** Tom-Tom has quit IRC | 07:39 | |
*** salv-orlando has quit IRC | 07:40 | |
*** salv-orlando has joined #openstack-nova | 07:47 | |
*** snasir has quit IRC | 07:47 | |
*** snasir has joined #openstack-nova | 07:48 | |
*** Tom-Tom has joined #openstack-nova | 07:51 | |
openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.54 - Enable reset keypair while rebuild https://review.openstack.org/519572 | 07:51 |
*** alexchadin has joined #openstack-nova | 07:54 | |
*** abhishekk has quit IRC | 07:58 | |
*** naichuans has quit IRC | 07:59 | |
*** naichuans has joined #openstack-nova | 07:59 | |
openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.55 - Enable cold migration with target host https://review.openstack.org/406707 | 08:06 |
*** takashin has left #openstack-nova | 08:07 | |
*** ralonsoh has joined #openstack-nova | 08:09 | |
*** yamamoto has quit IRC | 08:09 | |
*** pcaruana has joined #openstack-nova | 08:10 | |
*** yangyapeng has quit IRC | 08:10 | |
*** yamamoto has joined #openstack-nova | 08:12 | |
*** yangyapeng has joined #openstack-nova | 08:12 | |
*** yamamoto has quit IRC | 08:12 | |
openstackgerrit | Yikun Jiang proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406 | 08:13 |
*** tesseract has joined #openstack-nova | 08:15 | |
*** phuongnh has quit IRC | 08:15 | |
*** liusheng has quit IRC | 08:16 | |
*** Sukhdev has quit IRC | 08:16 | |
*** phuongnh has joined #openstack-nova | 08:17 | |
*** liusheng has joined #openstack-nova | 08:17 | |
*** links has quit IRC | 08:17 | |
openstackgerrit | Nguyen Van Trung proposed openstack/nova-specs master: Add linters test to run spec validation https://review.openstack.org/519561 | 08:23 |
*** salv-orlando has quit IRC | 08:24 | |
*** dikonoor has quit IRC | 08:25 | |
*** dikonoor has joined #openstack-nova | 08:26 | |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Change 'InstancePCIRequest' spec field https://review.openstack.org/449257 | 08:26 |
*** links has joined #openstack-nova | 08:31 | |
*** yingjun has joined #openstack-nova | 08:35 | |
*** qsyqian has joined #openstack-nova | 08:39 | |
*** yamamoto has joined #openstack-nova | 08:41 | |
*** salv-orlando has joined #openstack-nova | 08:42 | |
*** yamahata has quit IRC | 08:44 | |
*** AlexeyAbashkin has joined #openstack-nova | 08:47 | |
openstackgerrit | Saverio Proto proposed openstack/nova stable/pike: Correct log message when removing a security group https://review.openstack.org/519583 | 08:48 |
*** ratailor|Lunch is now known as ratailor | 08:48 | |
*** yamamoto has quit IRC | 08:50 | |
openstackgerrit | Saverio Proto proposed openstack/nova stable/pike: Correct log message when removing a security group https://review.openstack.org/519583 | 08:51 |
*** jpena|off is now known as jpena | 08:52 | |
openstackgerrit | Saverio Proto proposed openstack/nova stable/ocata: Correct log message when removing a security group https://review.openstack.org/519585 | 08:53 |
*** trungnv has quit IRC | 08:53 | |
*** priteau has joined #openstack-nova | 08:54 | |
openstackgerrit | Saverio Proto proposed openstack/nova stable/newton: Correct log message when removing a security group https://review.openstack.org/519586 | 08:54 |
*** sapd__ has quit IRC | 08:56 | |
*** sapd__ has joined #openstack-nova | 08:57 | |
*** rmart04 has joined #openstack-nova | 09:02 | |
*** Tom-Tom has quit IRC | 09:06 | |
*** Tom-Tom has joined #openstack-nova | 09:06 | |
openstackgerrit | Li Yingjun proposed openstack/nova master: Versioned notifications for service create and delete https://review.openstack.org/519588 | 09:07 |
*** links has quit IRC | 09:07 | |
*** Tom-Tom has quit IRC | 09:10 | |
*** Tom-Tom has joined #openstack-nova | 09:13 | |
*** cheneydc has quit IRC | 09:13 | |
*** cheneydc has joined #openstack-nova | 09:14 | |
*** trungnv has joined #openstack-nova | 09:15 | |
*** qsyqian has quit IRC | 09:15 | |
*** salv-orlando has quit IRC | 09:20 | |
*** markvoelker has quit IRC | 09:21 | |
*** links has joined #openstack-nova | 09:21 | |
*** salv-orlando has joined #openstack-nova | 09:22 | |
*** cheneydc has quit IRC | 09:23 | |
*** jianghuaw has quit IRC | 09:26 | |
*** salv-orlando has quit IRC | 09:26 | |
*** Tom-Tom has quit IRC | 09:26 | |
*** akekane has joined #openstack-nova | 09:27 | |
*** Tom-Tom has joined #openstack-nova | 09:27 | |
*** akekane is now known as abhishekk | 09:27 | |
*** qsyqian has joined #openstack-nova | 09:29 | |
*** slaweq has joined #openstack-nova | 09:31 | |
*** Tom-Tom has quit IRC | 09:31 | |
*** trungnv has quit IRC | 09:32 | |
openstackgerrit | Yikun Jiang proposed openstack/nova master: Add pagination and changes-since for instance-actions https://review.openstack.org/326326 | 09:33 |
*** yingjun has quit IRC | 09:34 | |
*** Tom-Tom has joined #openstack-nova | 09:35 | |
*** derekh has joined #openstack-nova | 09:39 | |
*** lucas-afk is now known as lucasagomes | 09:39 | |
*** andreas__ has joined #openstack-nova | 09:41 | |
*** andreas_s has quit IRC | 09:45 | |
*** trinaths has left #openstack-nova | 09:53 | |
*** Tom-Tom has quit IRC | 09:54 | |
*** jianghuaw has joined #openstack-nova | 09:55 | |
*** yangyapeng has quit IRC | 09:56 | |
*** yangyapeng has joined #openstack-nova | 09:57 | |
*** abhishekk has quit IRC | 09:57 | |
*** jianghuaw has quit IRC | 09:57 | |
*** jianghuaw has joined #openstack-nova | 09:58 | |
*** yamamoto has joined #openstack-nova | 10:01 | |
*** dikonoor has quit IRC | 10:01 | |
*** dikonoor has joined #openstack-nova | 10:01 | |
*** namnh has quit IRC | 10:04 | |
*** annp has quit IRC | 10:04 | |
*** diga has quit IRC | 10:05 | |
*** yangyapeng has quit IRC | 10:05 | |
*** salv-orlando has joined #openstack-nova | 10:08 | |
*** salv-orlando has quit IRC | 10:11 | |
*** salv-orlando has joined #openstack-nova | 10:11 | |
*** tssurya has joined #openstack-nova | 10:14 | |
*** felipemonteiro has joined #openstack-nova | 10:16 | |
*** qsyqian has quit IRC | 10:17 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test alloc_cands with indirectly sharing RPs https://review.openstack.org/519601 | 10:17 |
openstackgerrit | Rajesh Tailor proposed openstack/nova master: Host addition host-aggregate should be case-sensitive https://review.openstack.org/498334 | 10:17 |
*** Tom-Tom has joined #openstack-nova | 10:18 | |
gibi | efried: thanks for the rebase of my rp test patch it make sense to move that | 10:18 |
gibi | efried, jaypipes: I discovered a debatable edge case with shared RPs in https://review.openstack.org/#/c/519601/ | 10:19 |
gibi | efried, jaypipes: it is all about indirectly sharing resources through a non sharing RP | 10:19 |
openstackgerrit | Rajesh Tailor proposed openstack/nova master: Fix case-sensitivity for metadata keys https://review.openstack.org/504885 | 10:20 |
*** markvoelker has joined #openstack-nova | 10:21 | |
*** Tom-Tom has quit IRC | 10:22 | |
*** diga has joined #openstack-nova | 10:24 | |
*** lpetrut_ has joined #openstack-nova | 10:25 | |
*** ratailor has quit IRC | 10:26 | |
*** ratailor_ has joined #openstack-nova | 10:26 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test alloc_cands with indirectly sharing RPs https://review.openstack.org/519601 | 10:27 |
*** akekane has joined #openstack-nova | 10:27 | |
*** akekane is now known as abhishekk | 10:27 | |
*** sdague has joined #openstack-nova | 10:32 | |
*** qsyqian has joined #openstack-nova | 10:33 | |
*** yamamoto has quit IRC | 10:36 | |
*** yamamoto has joined #openstack-nova | 10:36 | |
*** qsyqian has quit IRC | 10:37 | |
*** claudiub has joined #openstack-nova | 10:39 | |
*** FL1SK has quit IRC | 10:41 | |
*** alexchadin has quit IRC | 10:42 | |
*** alexchadin has joined #openstack-nova | 10:42 | |
*** yangyapeng has joined #openstack-nova | 10:42 | |
*** yangyapeng has quit IRC | 10:51 | |
*** andreas__ has quit IRC | 10:53 | |
*** yangyapeng has joined #openstack-nova | 10:53 | |
*** jianghuaw has quit IRC | 10:56 | |
*** sridharg has quit IRC | 10:58 | |
*** andreas_s has joined #openstack-nova | 10:58 | |
*** udesale has quit IRC | 11:00 | |
*** ratailor_ has quit IRC | 11:00 | |
*** abhishekk has quit IRC | 11:02 | |
*** andreas_s has quit IRC | 11:02 | |
*** dikonoor has quit IRC | 11:02 | |
*** alexchadin has quit IRC | 11:02 | |
*** dikonoor has joined #openstack-nova | 11:03 | |
*** andreas_s has joined #openstack-nova | 11:03 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test alloc_cands with one RP shared between two RPs https://review.openstack.org/519617 | 11:06 |
*** andreas_s has quit IRC | 11:07 | |
gibi | efried, jaypipes: One more set of test case now for CN1 <- SS -> CN2 type of setup. Simple case works but a more complicated one mixes unconnected RPs: https://review.openstack.org/#/c/519617/ | 11:08 |
*** andreas_s has joined #openstack-nova | 11:08 | |
*** alexchadin has joined #openstack-nova | 11:10 | |
*** gouthamr has joined #openstack-nova | 11:10 | |
*** andreas_s has quit IRC | 11:12 | |
*** andreas_s has joined #openstack-nova | 11:13 | |
*** salv-orlando has quit IRC | 11:19 | |
*** salv-orlando has joined #openstack-nova | 11:19 | |
*** dtantsur|afk is now known as dtantsur | 11:20 | |
*** qsyqian has joined #openstack-nova | 11:22 | |
*** andreas_s has quit IRC | 11:22 | |
*** jianghuaw has joined #openstack-nova | 11:26 | |
*** andreas_s has joined #openstack-nova | 11:27 | |
*** akekane has joined #openstack-nova | 11:27 | |
jianghuaw | bauzas: Can we make this common patch be merged or still need further actions? https://review.openstack.org/#/c/512580/ | 11:28 |
*** akekane has quit IRC | 11:28 | |
*** qsyqian has quit IRC | 11:30 | |
*** andreas_s has quit IRC | 11:32 | |
moshele | stephenfin: hi | 11:33 |
*** andreas_s has joined #openstack-nova | 11:35 | |
*** gszasz has joined #openstack-nova | 11:35 | |
moshele | ramishra | 11:37 |
*** felipemonteiro has quit IRC | 11:38 | |
*** qsyqian has joined #openstack-nova | 11:43 | |
*** bhagyashri_s is now known as bhagyashris | 11:43 | |
*** andreas_s has quit IRC | 11:43 | |
*** andreas_s has joined #openstack-nova | 11:44 | |
*** bkopilov has quit IRC | 11:44 | |
openstackgerrit | Jianghua Wang proposed openstack/nova master: vgpu: add enabled white list https://review.openstack.org/512580 | 11:51 |
openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: get vGPU stats from hypervisor https://review.openstack.org/512965 | 11:51 |
openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: provide vGPU inventory in compute node https://review.openstack.org/516217 | 11:51 |
openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: create vGPU for instance https://review.openstack.org/516899 | 11:51 |
*** tbachman has quit IRC | 11:54 | |
openstackgerrit | Yikun Jiang proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406 | 11:55 |
openstackgerrit | Merged openstack/nova master: Handle InstanceNotFound when setting password via metadata https://review.openstack.org/472933 | 11:58 |
openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: create vGPU for instance https://review.openstack.org/516899 | 11:59 |
*** ratailor has joined #openstack-nova | 12:00 | |
*** phuongnh has quit IRC | 12:01 | |
*** alexchadin has quit IRC | 12:02 | |
*** alexchadin has joined #openstack-nova | 12:03 | |
*** nsingh has quit IRC | 12:05 | |
*** gouthamr has quit IRC | 12:05 | |
*** lucasagomes is now known as lucas-hungry | 12:06 | |
*** niraj_singh has joined #openstack-nova | 12:06 | |
*** udesale has joined #openstack-nova | 12:07 | |
openstackgerrit | Elod Illes proposed openstack/nova master: Deduplicate server_group samples https://review.openstack.org/519629 | 12:09 |
*** dikonoor has quit IRC | 12:09 | |
*** yangyapeng has quit IRC | 12:10 | |
openstackgerrit | Matthew Booth proposed openstack/nova stable/pike: libvirt: bandwidth param should be set in guest migrate https://review.openstack.org/519631 | 12:10 |
openstackgerrit | Matthew Booth proposed openstack/nova stable/pike: libvirt: Don't VIR_MIGRATE_NON_SHARED_INC without migrate_disks https://review.openstack.org/519632 | 12:10 |
*** yangyapeng has joined #openstack-nova | 12:10 | |
*** smatzek has joined #openstack-nova | 12:10 | |
*** yangyapeng has quit IRC | 12:18 | |
*** gouthamr has joined #openstack-nova | 12:20 | |
openstackgerrit | Matthew Booth proposed openstack/nova stable/ocata: libvirt: bandwidth param should be set in guest migrate https://review.openstack.org/519635 | 12:22 |
openstackgerrit | Matthew Booth proposed openstack/nova stable/ocata: libvirt: Don't VIR_MIGRATE_NON_SHARED_INC without migrate_disks https://review.openstack.org/519636 | 12:22 |
*** sbezverk has joined #openstack-nova | 12:24 | |
*** qsyqian has quit IRC | 12:33 | |
*** ratailor has quit IRC | 12:35 | |
openstackgerrit | Merged openstack/nova master: block_device_mapping_v2.bus_type is missing from api-ref https://review.openstack.org/499488 | 12:37 |
openstackgerrit | Merged openstack/nova master: Fix test_get_volume_config method https://review.openstack.org/489467 | 12:37 |
*** ttsiouts has quit IRC | 12:37 | |
openstackgerrit | Merged openstack/nova master: Fix wrapping of neutron forbidden error https://review.openstack.org/508802 | 12:37 |
*** tssurya has quit IRC | 12:40 | |
*** sridharg has joined #openstack-nova | 12:41 | |
*** ricardoas has quit IRC | 12:41 | |
*** rcernin has joined #openstack-nova | 12:44 | |
efried | gibi Thanks for the new tests, will look shortly. | 12:45 |
*** zhurong has joined #openstack-nova | 12:46 | |
*** sean-k-mooney has joined #openstack-nova | 12:48 | |
*** FL1SK has joined #openstack-nova | 12:50 | |
*** belmoreira has joined #openstack-nova | 12:50 | |
*** bkopilov has joined #openstack-nova | 12:53 | |
*** gouthamr has quit IRC | 12:53 | |
*** tssurya has joined #openstack-nova | 12:55 | |
*** lucas-hungry is now known as lucasagomes | 12:55 | |
*** links has quit IRC | 12:56 | |
*** gszasz has quit IRC | 12:59 | |
*** gouthamr has joined #openstack-nova | 13:02 | |
openstackgerrit | Eric Fried proposed openstack/nova master: placement: add nested resource providers https://review.openstack.org/377138 | 13:03 |
openstackgerrit | Eric Fried proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215 | 13:03 |
openstackgerrit | Eric Fried proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | 13:03 |
openstackgerrit | Eric Fried proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | 13:03 |
openstackgerrit | Eric Fried proposed openstack/nova master: Reformat _get_all_with_shared https://review.openstack.org/512799 | 13:03 |
efried | gibi ^ includes re-inserting that busted dependency in the middle. | 13:03 |
*** liverpooler has joined #openstack-nova | 13:04 | |
*** gszasz has joined #openstack-nova | 13:04 | |
*** tssurya has quit IRC | 13:05 | |
openstackgerrit | Eric Fried proposed openstack/nova master: placement: add nested resource providers https://review.openstack.org/377138 | 13:06 |
openstackgerrit | Eric Fried proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215 | 13:06 |
openstackgerrit | Eric Fried proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | 13:06 |
openstackgerrit | Eric Fried proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | 13:06 |
openstackgerrit | Eric Fried proposed openstack/nova master: Reformat _get_all_with_shared https://review.openstack.org/512799 | 13:06 |
efried | Dang, missed a rename, fixed ^ | 13:06 |
*** vladikr has joined #openstack-nova | 13:07 | |
*** yangyapeng has joined #openstack-nova | 13:08 | |
gibi | efried: your https://review.openstack.org/#/c/514092 bounced back from the gate as it needs a rebase and a unit test fix | 13:08 |
*** andreas_s has quit IRC | 13:08 | |
efried | gibi Thanks, on it. | 13:09 |
*** diga has quit IRC | 13:10 | |
*** jpena is now known as jpena|lunch | 13:11 | |
*** tbachman has joined #openstack-nova | 13:12 | |
*** liverpooler has quit IRC | 13:12 | |
openstackgerrit | Eric Fried proposed openstack/nova master: placement: AllocCands.get_by_{filters => requests} https://review.openstack.org/514092 | 13:12 |
*** jaypipes has joined #openstack-nova | 13:12 | |
efried | gibi ^ hecho | 13:13 |
efried | jaypipes Welcome back! | 13:13 |
*** yangyapeng has quit IRC | 13:13 | |
*** liverpooler has joined #openstack-nova | 13:13 | |
*** amodi has joined #openstack-nova | 13:13 | |
jaypipes | efried: heh, thanks :) | 13:13 |
*** andreas_s has joined #openstack-nova | 13:13 | |
alex_xu | jaypipes: good morning! | 13:13 |
jaypipes | alex_xu: is it? :) | 13:13 |
efried | Friday jaypipes open patches: 13; merge conflicts: 12. Tuesday jaypipes open patches: 13; merge conflicts: 1 | 13:14 |
efried | I'd say that's a good morning. | 13:14 |
jaypipes | efried: oh, excellent :) | 13:14 |
alex_xu | sounds like all of us are wating for you :) | 13:14 |
jaypipes | alex_xu: yeah, had a "lovely" new-hire training session up in Tampa yesterday ... | 13:14 |
alex_xu | jaypipes: efried, just want to show you this patch https://review.openstack.org/519081 | 13:15 |
alex_xu | new-hire training is always lovely | 13:15 |
*** dave-mccowan has joined #openstack-nova | 13:15 | |
openstackgerrit | Eric Fried proposed openstack/nova master: placement: AllocCands.get_by_{filters => requests} https://review.openstack.org/514092 | 13:16 |
alex_xu | jaypipes: efried I didn't some test last week on 1w rps and probably 70% loads on each rp, then I found that sql is faster than the current one. The current one will spend 1.7s in my env, the new sql just spend 0.7s | 13:16 |
alex_xu | s/didn't/did/... | 13:16 |
*** Tom-Tom has joined #openstack-nova | 13:16 | |
*** edand has quit IRC | 13:17 | |
efried | alex_xu It was on my list to look at. I wanted to scrutinize the functional test a bit. | 13:17 |
jaypipes | alex_xu: what is 1w rps? | 13:17 |
*** tssurya has joined #openstack-nova | 13:17 | |
alex_xu | efried: ok, thanks | 13:18 |
alex_xu | jaypipes: 10000 resource providers... | 13:18 |
jaypipes | alex_xu: can you put your DB setup into a pastebin please? # rps, inventory records used per rc, number of allocations used per rp, etc | 13:21 |
*** edmondsw has joined #openstack-nova | 13:22 | |
alex_xu | jaypipes: ok | 13:22 |
*** yangyapeng has joined #openstack-nova | 13:23 | |
*** ttsiouts has joined #openstack-nova | 13:23 | |
*** artom_ has quit IRC | 13:24 | |
*** erlon has joined #openstack-nova | 13:25 | |
*** zhurong has quit IRC | 13:26 | |
alex_xu | jaypipes: here is https://pastebin.com/kPMtEcQb, i just copy the variables from my script | 13:26 |
*** yangyapeng has quit IRC | 13:27 | |
jaypipes | alex_xu: k, thank you sir :) I'll run benchmarks later today. | 13:27 |
*** andreas_s has quit IRC | 13:27 | |
*** andreas_s has joined #openstack-nova | 13:28 | |
alex_xu | jaypipes: cool | 13:29 |
*** gszasz has quit IRC | 13:30 | |
alex_xu | jaypipes: also a non-shared version at https://review.openstack.org/#/c/516778/6/nova/objects/resource_provider.py@2518 | 13:30 |
jaypipes | alex_xu: k | 13:31 |
efried | alex_xu I rebased your patch on top of the pending series - the one with all the extra tests in it - and it's failing some stuff. | 13:31 |
*** fragatina has joined #openstack-nova | 13:31 | |
*** ralonsoh has quit IRC | 13:32 | |
efried | alex_xu There are a number of cases where it's returning zero candidates. | 13:32 |
jaypipes | alex_xu: sum(invs.resource_class_id) is not what you're looking for there. I believe you're looking for count(distinct invs.resource_class_id) | 13:32 |
*** edand has joined #openstack-nova | 13:32 | |
*** tbachman has quit IRC | 13:32 | |
*** tbachman has joined #openstack-nova | 13:33 | |
alex_xu | jaypipes: oops, that is true | 13:35 |
alex_xu | jaypipes: but it shoudl works with count | 13:35 |
jaypipes | alex_xu: yep. | 13:35 |
alex_xu | efried: sorry, which patch? | 13:35 |
efried | https://review.openstack.org/#/c/519081/ | 13:35 |
jaypipes | alex_xu: I'll benchmark later today. I have a dentist appointment shortly and will set up benchmarks after that | 13:35 |
alex_xu | jaypipes: ok, no problem | 13:36 |
*** fragatina has quit IRC | 13:36 | |
alex_xu | efried: ok, I will try to debug them | 13:37 |
efried | alex_xu Note that some of the test cases are highlighting bugs, so the current 'expected' value is wrong - but the correct 'expected' value is in the comments, and it still ain't [] | 13:37 |
alex_xu | efried: got it, thanks | 13:38 |
*** gouthamr has quit IRC | 13:38 | |
*** links has joined #openstack-nova | 13:38 | |
*** salv-orlando has quit IRC | 13:40 | |
*** salv-orlando has joined #openstack-nova | 13:41 | |
sean-k-mooney | jaypipes: o/ | 13:42 |
gibi | efried: thanks for the fix on the get_by_requests refactor patch. I'm +2. | 13:43 |
sean-k-mooney | jaypipes: can i get you to weigh in on a matter of os-vif policy. specifcally https://review.openstack.org/#/c/484386/ and the extention of our public api. | 13:43 |
gibi | efried: Do you agree to move https://review.openstack.org/#/c/518633/ top of Jay's series as that will decrease the amount of visible bugs in the patch? | 13:43 |
sean-k-mooney | jaypipes: i -2'd the previous itteration that added a shared implementation to os-vif.common | 13:44 |
sean-k-mooney | jaypipes: it has now been split but sahid dislikes the code duplication in tree this causes | 13:44 |
sean-k-mooney | as a counter proposal the only path i see forward to not extend the publci api and avoid duplication is to have a os-vif.internal and use an assertion to ensure that no one else can import it like this http://paste.openstack.org/show/626267/. toughts? | 13:45 |
*** andreas_s has quit IRC | 13:46 | |
*** andreas_s has joined #openstack-nova | 13:48 | |
*** yangyapeng has joined #openstack-nova | 13:50 | |
efried | gibi I was just looking at that. In general I'm in favor of moving any test patches on top of that series, as it does seem to be resolving *some* issues. | 13:51 |
efried | gibi As we were discussing above, I'm also in favor of putting things like refactors on top of that series because the extra tests will flush out more bugs. | 13:52 |
efried | (refactors of the prod code) | 13:52 |
gibi | efried: cool. If you move that there I can quickly re-review and +2 it :) | 13:52 |
*** edmondsw_ has joined #openstack-nova | 13:53 | |
*** esberglu has joined #openstack-nova | 13:54 | |
*** lyan has joined #openstack-nova | 13:54 | |
*** gouthamr has joined #openstack-nova | 13:54 | |
*** edmondsw has quit IRC | 13:55 | |
*** szaher has quit IRC | 13:58 | |
*** yangyapeng has quit IRC | 13:58 | |
*** edmondsw_ is now known as edmondsw | 13:59 | |
*** edmondsw has quit IRC | 13:59 | |
*** lajoskatona has quit IRC | 13:59 | |
*** mriedem has joined #openstack-nova | 13:59 | |
*** andreas_s has quit IRC | 14:01 | |
*** szaher has joined #openstack-nova | 14:02 | |
*** jianghuaw_ has joined #openstack-nova | 14:03 | |
*** rmart04 has quit IRC | 14:04 | |
*** rmart04 has joined #openstack-nova | 14:04 | |
*** andreas_s has joined #openstack-nova | 14:05 | |
*** rmart04 has quit IRC | 14:05 | |
*** tbachman has quit IRC | 14:08 | |
*** jpena|lunch is now known as jpena | 14:08 | |
*** awaugama has joined #openstack-nova | 14:09 | |
*** alexchadin has quit IRC | 14:10 | |
*** hshiina has quit IRC | 14:11 | |
*** gszasz has joined #openstack-nova | 14:12 | |
*** edmondsw has joined #openstack-nova | 14:12 | |
*** edleafe- is now known as edleafe | 14:13 | |
*** efried has quit IRC | 14:14 | |
* edleafe returns to Novaland | 14:14 | |
*** Oku_OS has quit IRC | 14:17 | |
gibi | edleafe: welcome back | 14:19 |
bauzas | edleafe: hope you didn't get sick and your IRC bouncer is fine, unlike me | 14:20 |
* bauzas still struggling with redeploying another ZNC instance | 14:21 | |
bauzas | oh, and my corp phone died too | 14:21 |
*** Oku_OS has joined #openstack-nova | 14:22 | |
edleafe | bauzas: znc was humming along nicely in my absence | 14:23 |
edleafe | don't know if I can read back that much :) | 14:23 |
bauzas | hah | 14:23 |
*** hamzy_ has quit IRC | 14:24 | |
*** ralonsoh has joined #openstack-nova | 14:24 | |
*** smatzek has quit IRC | 14:25 | |
sean-k-mooney | has anyone stacked on centos 7.4 recently | 14:25 |
*** efried has joined #openstack-nova | 14:27 | |
*** yangyapeng has joined #openstack-nova | 14:29 | |
kashyap | Do people do that often? I thought most do it only with Debian / Ubuntu / Fedora | 14:30 |
openstackgerrit | Eric Fried proposed openstack/nova master: Test allocation candidates: multiple aggregates https://review.openstack.org/518633 | 14:31 |
efried | gibi ^ That was... interesting. | 14:31 |
*** gbarros has joined #openstack-nova | 14:31 | |
sean-k-mooney | kashyap: stack with centos? | 14:36 |
*** tbachman has joined #openstack-nova | 14:36 | |
bhagyashris | mriedem: Hi, I have one doubt regarding the instance snapshot. When I create the instance snapshot using the nova image-create <server-id> <snapshot-name> the it only creates the plain image that means it only calls the image-create and not image-upload and the image remains in queued status so How can I create instance snapshot in active state. | 14:36 |
sean-k-mooney | kashyap: i normally do all my dev on ubuntu but i need to use centos currently because reasons.... | 14:36 |
*** amodi has quit IRC | 14:36 | |
*** moshele has quit IRC | 14:36 | |
kashyap | sean-k-mooney: Yeah | 14:37 |
gibi | efried: did you encountered you type of problems during the rebase? | 14:38 |
kashyap | sean-k-mooney: Might be also want check on #openstack-qa channel | 14:38 |
sean-k-mooney | kashyap: fedora is rather fragile as its a bit too bleading edge | 14:38 |
*** yangyapeng has quit IRC | 14:38 | |
efried | gibi The test results changed rather dramatically. | 14:38 |
kashyap | sean-k-mooney: Yeah, bled this morning, trying to shave a yak. | 14:38 |
efried | gibi The good news is that all the results now at least seem to have the proper resources in them. | 14:38 |
efried | gibi I think the problems are now limited to crossing aggregates. | 14:39 |
sean-k-mooney | kashyap: im currently hitting a issue with nova that is proably unrelated to centos to be honest just have not seen it last time i stacked on ubuntu. devstack is not finding the nova compute agent in the cells db and devstack is failing because of this | 14:39 |
gibi | efried: I feel that is actually a good news. From having multiple problems we managed to go done to one single issue | 14:40 |
efried | gibi Oh, definitely agree. | 14:41 |
efried | Just made for a tough rebase :) | 14:41 |
kashyap | sean-k-mooney: I see, afraid I don't know the cause of it. | 14:42 |
mriedem | bhagyashris: something probably failed server-side, because nova-compute will upload the image data to glance | 14:42 |
*** markvoelker has quit IRC | 14:43 | |
*** markvoelker has joined #openstack-nova | 14:43 | |
sean-k-mooney | kashyap: im restacking with reclone=true to make sure eveything is uptodate but basically im seeing openstack --os-cloud devstack-admin --os-region RegionOne compute service list --host silpixa00385232 --service nova-compute -c ID -f value fail. | 14:44 |
*** Nil_ has joined #openstack-nova | 14:44 | |
kashyap | sean-k-mooney: In the past 'reclone=true' screwed me up. Let me dig up the reason, I recall writing it down why it's "bad" | 14:44 |
kashyap | IIRC, it messes up with Git remotes | 14:44 |
sean-k-mooney | kashyap: hehe i have been using devstack for 4 years i know why it can be bad but in this case its safe enough as it will just do a git pull on all the repos in /opt/stack | 14:45 |
bhagyashris | mriedem: but there is no optional parameter --file at the time of creation of instance snapshot so thats why the snapshot created at glance is in queued state | 14:45 |
kashyap | sean-k-mooney: Ah, okay, you're well-scarred then. | 14:45 |
mriedem | bhagyashris: i think you might be misunderstanding what the nova image-create operation does | 14:46 |
mriedem | it's not the same as glance image-create and image-upload | 14:46 |
*** mlavalle has joined #openstack-nova | 14:46 | |
sean-k-mooney | kashyap: enough so to be imune to the fact it will happliy destroy your system if you dont speak to it firmly form time to time yes | 14:46 |
kashyap | sean-k-mooney: I use this little ugly script as a 'hook' to update all the repos -- http://paste.openstack.org/show/626273/ | 14:47 |
kashyap | sean-k-mooney: Heh, indeed | 14:47 |
*** Yingxin has quit IRC | 14:48 | |
*** Yingxin has joined #openstack-nova | 14:48 | |
sean-k-mooney | kashyap: yes that is basically what reclone=true does in the git_clone fuction https://github.com/openstack-dev/devstack/blob/master/functions-common#L493-L566 but it also support gerrit references,branches and tags | 14:48 |
bhagyashris | bhagyashris: I will go through nova image-create code | 14:48 |
kashyap | sean-k-mooney: Yeah, that's why I use a minimal script that uses `virt-builder` that sets up DevStack env from scratch, if I'm taking too much time to debug the darn thing. (Should clean up the script) -- https://kashyapc.fedorapeople.org/virt/openstack/setup-devstack-vm.bash | 14:49 |
bhagyashris | mriedem: I will go through nova image-create code | 14:49 |
kashyap | sean-k-mooney: Ah, noted; okay I'll be a bit less fearful of it, and give it a whirl next time | 14:49 |
mriedem | bhagyashris: nova-api creates an image in glance and then casts to the nova-compute service to actually snapshot the disk and upload the file to glance for the snapshot image, | 14:50 |
mriedem | so if the image was stuck in queued state forever, something probably failed on the nova-compute side | 14:50 |
*** lpetrut_ has quit IRC | 14:51 | |
jianghuaw_ | bauzas, jaypipes: are you around? | 14:51 |
sean-k-mooney | reclone=true is safe, PIP_UPGRADE=ture is not and should be a last resort as it ignores the upper-constriants file | 14:51 |
*** lpetrut_ has joined #openstack-nova | 14:51 | |
jianghuaw_ | bauzas, jaypipes: Have some questions on how to report nested resource providers and traits from drivers. | 14:52 |
bhagyashris | mriedem: ohh ok. Thank you :) | 14:52 |
*** gbarros has quit IRC | 14:53 | |
*** smatzek has joined #openstack-nova | 14:53 | |
*** yamahata has joined #openstack-nova | 14:53 | |
*** READ10 has joined #openstack-nova | 14:53 | |
*** smatzek has quit IRC | 14:53 | |
bauzas | jianghuaw: over the phone, sorry | 14:53 |
*** smatzek has joined #openstack-nova | 14:53 | |
jianghuaw_ | bauzas, ok. May chat with you later. | 14:54 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Handle InstanceNotFound when setting password via metadata https://review.openstack.org/519659 | 14:56 |
openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: Unit tests must mock os_xenapi calls https://review.openstack.org/499573 | 15:00 |
*** rcernin has quit IRC | 15:00 | |
*** mdnadeem has quit IRC | 15:01 | |
jaypipes | jianghuaw_: just got back. what's up? | 15:04 |
*** cfriesen has joined #openstack-nova | 15:04 | |
jianghuaw_ | jaypipes, my problem is how to report the nested resource providers and traits from drivers to resource provider. Should we use the existing get_inventory()? | 15:05 |
jaypipes | sean-k-mooney: what is stephenfin's opinion? | 15:05 |
jianghuaw_ | jaypipes, or should we define new interfaces? | 15:06 |
jaypipes | jianghuaw_: no, we will need to define a new update_inventory() interface to the virt driver API. | 15:06 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add FlavorPayload.description for versioned notifications https://review.openstack.org/514782 | 15:06 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add microversion to allow setting flavor description https://review.openstack.org/515213 | 15:06 |
sean-k-mooney | jaypipes: i think he as ok with either version. but stephenfin if you are around maybe you can comment | 15:07 |
jianghuaw_ | what's the data structure return by update_inventory()? | 15:07 |
*** eharney has joined #openstack-nova | 15:08 | |
sean-k-mooney | jaypipes: there are two patches from rodolfo that are effect by this. https://review.openstack.org/#/c/487405/ and https://review.openstack.org/#/c/484386/ | 15:08 |
jaypipes | jianghuaw_: I was thinking a boolean. The update_inventory() would accept a ProviderTree object and the virt driver would update the providers in the ProviderTree appropriately, returning whether anything changed. | 15:08 |
jaypipes | sean-k-mooney: understood. I will try to get to those today. | 15:09 |
sean-k-mooney | jaypipes: thanks jay. and no worries it would be good to come to a dession but soon but i also want to make the right one so lets not rush. | 15:10 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Validate new image via scheduler during rebuild https://review.openstack.org/519662 | 15:10 |
jianghuaw_ | jaypipes, so everything is done in driver. resource tracker just trigger the update. right? | 15:10 |
efried | jaypipes So I'm clear, does ProviderTree include the resources, or just the provider hierarchy? | 15:12 |
openstackgerrit | Ilya Shakhat proposed openstack/nova master: Initialize osprofiler in uWSGI application https://review.openstack.org/519664 | 15:12 |
sean-k-mooney | efried: i think the full tree with the resouces. at least that is what i had assumed at the PTG | 15:13 |
jianghuaw_ | jaypipes, do you have a rough plan on when will start to add the interface of update_inventory()? | 15:13 |
*** gouthamr has quit IRC | 15:13 | |
jaypipes | jianghuaw_: for now, yes, that's the plan. eventually, a generic device manager may also get called... but that's later on. | 15:13 |
efried | sean-k-mooney That would make sense if the method is called update_inventory | 15:13 |
jaypipes | efried: no, inventory records are kept in the ProviderTree for each resource class the provider provides | 15:13 |
jianghuaw_ | jaypipes, just wonder if it's proper time to start n-rp work for vgpu in XenAPI. | 15:14 |
*** yangyapeng has joined #openstack-nova | 15:14 | |
*** Tom-Tom_ has joined #openstack-nova | 15:15 | |
openstackgerrit | Xuanzhou Perry Dong proposed openstack/nova master: Check for volume size https://review.openstack.org/519666 | 15:16 |
*** gouthamr has joined #openstack-nova | 15:16 | |
efried | jianghuaw_ If you do, you'll have to base it on the series in progress, starting at https://review.openstack.org/#/c/377138/ | 15:16 |
*** psachin has quit IRC | 15:16 | |
*** psachin has joined #openstack-nova | 15:17 | |
*** Tom-Tom has quit IRC | 15:17 | |
jianghuaw_ | efried, thanks. yes I have been looking at this series of patches. | 15:17 |
efried | jianghuaw_ But also note that we (jaypipes, alex_xu, me, etc.) haven't yet written the GET /allocation_candidates affordance for nrp. | 15:18 |
efried | Which could make the scheduling results... unpredictable. | 15:18 |
*** armax has joined #openstack-nova | 15:19 | |
*** yangyapeng has quit IRC | 15:19 | |
*** amodi has joined #openstack-nova | 15:19 | |
jianghuaw_ | efried, I see. I had some testing today. I understood n-rp has not been completed yet. But wonder if we can do things in parallel. | 15:19 |
sean-k-mooney | efried: is the numbered traits syntax for associating tratis and resouce requests that was discussed at the summit still going ahead for pike. | 15:20 |
sean-k-mooney | sorry queens | 15:20 |
*** hongbin has joined #openstack-nova | 15:20 | |
efried | jianghuaw_ To some extent, in the sense that more than one person can be proposing patches for the various pieces of functionality; but in this case, they should probably be proposed on that same series. Which can get confusing for rebases. | 15:20 |
efried | sean-k-mooney Yes, and good progress has been made there. At this point it's stalled waiting for the big refactor series starting at https://review.openstack.org/#/c/516778/ | 15:21 |
sean-k-mooney | efried: good to hear is the requesting traits via flavours work integrated with this also | 15:21 |
jianghuaw_ | efried, understood. | 15:21 |
*** litao__ has quit IRC | 15:22 | |
efried | sean-k-mooney Yes. The code to parse granular resources & traits from flavor extra specs, pass to the scheduler, pass to GET /allocation_candidates, parse in placement from the querystring, and pass to the internal methods - all that is done. | 15:22 |
efried | sean-k-mooney The remaining piece is to make said internal methods actually Do The Right Thing with that information, which will bump the placement microversion; and then switch on the support from the scheduler side. | 15:23 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Validate new image via scheduler during rebuild https://review.openstack.org/519672 | 15:23 |
efried | sean-k-mooney You can see that stuff here: https://review.openstack.org/#/q/topic:bp/granular-resource-requests | 15:24 |
sean-k-mooney | efried: ah ok, so yes good progress is certenly apt. that will make my verified boot friend very happy to hear once i translate it for them. | 15:25 |
sean-k-mooney | jaypipes: on that note im going to split all the stateful traits into seperate patches. i know currently we do not want to standardies stateful traits so until we come to a dession on how to model the state of capablities ill advise the use of CUSTOM_ traits | 15:27 |
jaypipes | jianghuaw_: I'd appreciate another week to get more n-r-p stuff merged. | 15:30 |
jaypipes | sean-k-mooney: there's no such thing as a stateful trait... I'll respond to you on those patches. | 15:30 |
jianghuaw_ | jaypipes: cool. Thanks. | 15:31 |
sean-k-mooney | jaypipes: :) i know but without inventing another entity to tag ever resouce provider with and request every where we request a trait im inventing that term untill we have something better | 15:31 |
sean-k-mooney | jaypipes: but for now ill reduce the scope of those patches to just capabilities with no state implied as you suggesed in the patch reviews previously | 15:32 |
jaypipes | sean-k-mooney: whether or not a particular trait is applied to a specific resource provider is not relevant to what the trait represents. | 15:32 |
jaypipes | sean-k-mooney: that's all I'm saying. | 15:32 |
jaypipes | sean-k-mooney: if a node supports UEFI boot mode, then it supports UEFI boot mode. Whether or not that node is *booted* using UEFI boot mode is entirely a different thing. | 15:33 |
jianghuaw_ | jaypipes, I'd appreciate if you can have time to review these patches again: https://review.openstack.org/#/c/512965 https://review.openstack.org/#/c/516217/ | 15:34 |
sean-k-mooney | jaypipes: yep understood traits solves the fomer. i had hoped it could be used to solve the later but if we want to use something else for that then that is fine too so long as we can eventually solve both | 15:34 |
*** bkopilov has quit IRC | 15:37 | |
*** bkopilov has joined #openstack-nova | 15:38 | |
*** bkopilov has quit IRC | 15:38 | |
*** janki has quit IRC | 15:39 | |
*** bkopilov has joined #openstack-nova | 15:39 | |
crlb | Hello jaypipes, can I ask you about change #516796? | 15:41 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: Validate new image via scheduler during rebuild https://review.openstack.org/519681 | 15:41 |
efried | sean-k-mooney What patches are we talking about? | 15:41 |
sean-k-mooney | efried: https://review.openstack.org/#/q/status:open+project:openstack/os-traits+branch:master+topic:verified_boot | 15:42 |
efried | sean-k-mooney Thanks. We had a discussion along these lines with the ironic folks too. Did you see the summary in the ML? | 15:43 |
sean-k-mooney | efried: also https://review.openstack.org/#/c/514737/ but im less concured about that one but it does better illistrate the type of configuration associated with the capablities that we would like to model eventurally | 15:43 |
sean-k-mooney | efried: no but i was aware that disscution was going to take place | 15:44 |
*** jmlowe has joined #openstack-nova | 15:44 | |
efried | sean-k-mooney Here's the summary -- and there's a link to the video recording if you want to sit through the whole thing (and see costumes): http://lists.openstack.org/pipermail/openstack-dev/2017-November/124200.html | 15:45 |
sean-k-mooney | efried: oh cool thanks. ya i will try and let it play in the background to get up to speed. that said im not sure how much time i will have on this or any topic going forword | 15:46 |
*** dtantsur is now known as dtantsur|brb | 15:46 | |
efried | understood | 15:46 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/newton: Validate new image via scheduler during rebuild https://review.openstack.org/519684 | 15:49 |
mriedem | dansmith: bauzas: https://review.openstack.org/#/q/I11746d1ea996a0f18b7c54b4c9c21df58cc4714b | 15:49 |
mriedem | tristanC: any other way you want to advertise this? | 15:50 |
*** artom_ has joined #openstack-nova | 15:50 | |
tristanC | mriedem: could we get +2 on the patches as well as the related change on stable/newton? | 15:51 |
*** Tom-Tom_ has quit IRC | 15:51 | |
jianghuaw_ | mriedem, could you suggest what I should do to move forward these two patches? https://review.openstack.org/#/q/topic:vdiStream+(status:open+OR+status:merged)+project:+openstack/nova | 15:51 |
mriedem | tristanC: that's why i pinged dan and sylvain | 15:52 |
mriedem | they are stable core | 15:52 |
mriedem | and master branch core for that matter... | 15:52 |
*** moshele has joined #openstack-nova | 15:52 | |
tristanC | does the pre-OSSA update looks good to you? | 15:52 |
mriedem | tristanC: the etherpad? | 15:53 |
tristanC | yes, https://etherpad.openstack.org/p/ofe4voc7sg | 15:53 |
mriedem | etherpad looks fine | 15:53 |
sean-k-mooney | hum... on the plus side my openstack --os-cloud devstack-admin --os-region RegionOne compute service list --host vhost-user --service nova-compute -c ID -f value issue also happens on ubuntu. on the negitive side it also happens on ubuntu ... | 15:53 |
mriedem | sean-k-mooney: what issue? | 15:53 |
tristanC | mriedem: thanks! | 15:54 |
mriedem | jianghuaw_: do you mean regarding the blueprint? | 15:54 |
sean-k-mooney | mriedem: devstack is failing for me with "Didn't find service registered by hostname after 60 seconds" checking for the nova compute service to be registered | 15:54 |
jianghuaw_ | mriedem, yes. | 15:55 |
jaypipes | sean-k-mooney: apologies, keep getting ripped away :( | 15:55 |
mriedem | sean-k-mooney: does the hostname match what's getting created via the discover_hosts CLI? | 15:55 |
mriedem | discover_hosts CLI is what creates the host mappings in the cell in the api db | 15:55 |
sean-k-mooney | mriedem: ill check the conductor console auth and scheduer are registered | 15:55 |
mriedem | jianghuaw_: i wish you would have asked several weeks ago, before the spec freeze | 15:55 |
jaypipes | crlb: yessir | 15:56 |
openstackgerrit | Eric M Gonzalez (tasker) proposed openstack/nova master: unquiesce instance on volume snapshot failure https://review.openstack.org/519464 | 15:56 |
*** yangyapeng has joined #openstack-nova | 15:56 | |
jianghuaw_ | mriedem, that's a shame. I thought I asked here to get reviewing on the patches. | 15:56 |
jianghuaw_ | mriedem, I thought that's spec-less BP. | 15:57 |
mriedem | jianghuaw_: it is a specless bp, but i generally consider new bp approval freeze to be at the same time as the spec freeze, which was october 19 | 15:58 |
mriedem | nearly a month ago | 15:58 |
mriedem | it's my fault for not being clear about that, | 15:58 |
*** ratailor has joined #openstack-nova | 15:58 | |
mriedem | but also, specless blueprints are supposed to be brought up in the weekly nova meeting to get discussion and approval | 15:58 |
jianghuaw_ | mriedem, sorry. That's my fault. | 15:58 |
tssurya | dansmith : thanks for the review I will address the comments soon, and yes you are right about the integrity issue with regards to us ending up with a different list of uuids than what we actually deleted, the same problem also exists between the list of records inserted into the shadow tables and the list archived - we had a bug reported ; https://bugs.launchpad.net/nova/+bug/1727266 | 15:58 |
openstack | Launchpad bug 1727266 in OpenStack Compute (nova) "archive_deleted_instances is not atomic for insert/delete" [Undecided,In progress] - Assigned to Surya Seetharaman (tssurya) | 15:58 |
tssurya | dansmith : maybe we need to first query for the list of records whose deleted flag is set; use that list to insert into the shadow tables, archive from the actual tables and also return the same list for deleting the instance_mappings and request specs ? | 15:59 |
mriedem | jianghuaw_: https://docs.openstack.org/nova/pike/contributor/blueprints.html#specs "When a blueprint does not require a spec it still needs to be approved before the code which implements the blueprint is merged. Specless blueprints are discussed and potentially approved during the Open Discussion portion of the weekly nova IRC meeting. See trivial specifications for more details." | 15:59 |
*** janki has joined #openstack-nova | 15:59 | |
dansmith | tssurya: just about to jump on a call so I can't really process that in my head at the moment, so can you put it in the review for me to read when I circle back? | 16:00 |
*** links has quit IRC | 16:00 | |
jianghuaw_ | mriedem, got it. Thanks. | 16:00 |
tssurya | dansmith : yes of course | 16:00 |
crlb | jaypipes I'm running tox locally and getting no errors. However, on the review, Zuul, Dell, and Xen, appear to give errors that I don't understand. Is there something else I should be doing? | 16:00 |
dansmith | tssurya: thanks | 16:00 |
mriedem | tristanC: +2s on https://review.openstack.org/#/q/I11746d1ea996a0f18b7c54b4c9c21df58cc4714b | 16:01 |
tristanC | dansmith: thanks, appreciated the fast review! | 16:01 |
dansmith | mriedem: tristanC I just hit the last one | 16:01 |
dansmith | heh, yep | 16:01 |
mriedem | dansmith: thanks | 16:01 |
mriedem | i will assume bauzas is on the same call | 16:02 |
*** yangyapeng has quit IRC | 16:02 | |
jaypipes | crlb: how are you running tox locally? | 16:02 |
jaypipes | crlb: do this: tox -epy27,pep8 | 16:02 |
jaypipes | crlb: and you'll see the same errors. | 16:02 |
tristanC | i'm waiting for vmt +2 on the ossa (https://review.openstack.org/519686) to send the advisory | 16:02 |
mriedem | ack | 16:04 |
*** itlinux has joined #openstack-nova | 16:04 | |
bauzas | mriedem: thanks for helping with https://review.openstack.org/#/c/519662/ | 16:05 |
sean-k-mooney | mriedem: so looking at the nova_api db the host_mappins is empty as is the compute_nodes table in the cell1 db however the devstack@n-cpu.service is running on the hsot according to systemctl | 16:05 |
bauzas | and thanks for the tests | 16:05 |
mriedem | tristanC: small typo in your patch | 16:06 |
*** Oku_OS is now known as Oku_OS-away | 16:08 | |
*** salv-orlando has quit IRC | 16:09 | |
sean-k-mooney | mriedem: huh i just realised there is a giant Guru Meditation report dump in the nova-compute journalctl log. is there a partcalar part that i should read in that to figure out what failed | 16:10 |
mriedem | i think that happens when you kill the service? | 16:11 |
mriedem | i'd check for errors in the n-cpu logs about why the compute node record wasn't created | 16:11 |
mriedem | are the conductor services running? | 16:12 |
*** yangyapeng has joined #openstack-nova | 16:12 | |
jaypipes | sean-k-mooney: the guru part. | 16:12 |
bauzas | mriedem: dansmith: FWIW, I'm having a big problem with my IRC bouncer so for the moment, I'm not getting a notify if someone asks me | 16:14 |
bauzas | mriedem: but yeah, just saw your pings | 16:14 |
*** yangyapeng has quit IRC | 16:16 | |
sean-k-mooney | mriedem: according to openstack compute service list yes. that said on ubuntu i actully have a tracback and a libvirt issue. on centos it prints no No Traceback! in one of the green tread imediatly after connecting to libvirt | 16:17 |
mriedem | efried: remember that ksa adapter bug i pointed out to you on saturday? did that amount to anything? | 16:18 |
mriedem | person was having failures connecting to glance | 16:18 |
openstackgerrit | Matthew Booth proposed openstack/nova stable/pike: libvirt: bandwidth param should be set in guest migrate https://review.openstack.org/519631 | 16:18 |
openstackgerrit | Matthew Booth proposed openstack/nova stable/pike: libvirt: Don't VIR_MIGRATE_NON_SHARED_INC without migrate_disks https://review.openstack.org/519632 | 16:18 |
efried | mriedem I don't believe it was a ksa adapter thing. I responded in the bug report. I think they had either their conf or their catalog misconfigured. | 16:19 |
mriedem | ok you must have invalidated the bug? | 16:19 |
efried | mriedem I marked it as Incomplete | 16:19 |
mriedem | oh ok | 16:19 |
efried | mriedem From what I could tell, they were running pike, so it would be before my ksa adapter code, FWIW. | 16:20 |
efried | I mean, still going through ksa adapters & sessions ultimately | 16:20 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: fix link to creating unit tests in contributor guide https://review.openstack.org/519482 | 16:23 |
mriedem | bauzas: don't forget the two changes below https://review.openstack.org/#/c/519684/ | 16:23 |
bauzas | done | 16:24 |
mriedem | thanks. dansmith ^ when you're off your call | 16:25 |
mriedem | tristanC: changes are all approved, just need to approve the dependencies on the newton change | 16:25 |
mriedem | i'll request releases on all stable branches once those are merged | 16:25 |
openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: create vGPU for instance https://review.openstack.org/516899 | 16:26 |
mriedem | and then we'll eol newton | 16:26 |
*** andreas_s has quit IRC | 16:29 | |
*** salv-orlando has joined #openstack-nova | 16:31 | |
*** prometheanfire has joined #openstack-nova | 16:32 | |
prometheanfire | mriedem: https://review.openstack.org/#/c/519672/ doesn't apply to 16.0.2 cleanly, are we expecting releases soon with the fix (for pike/ocata at least)? | 16:32 |
*** yangyapeng has joined #openstack-nova | 16:33 | |
*** andreas_s has joined #openstack-nova | 16:33 | |
mriedem | prometheanfire: "(10:25:59 AM) mriedem: i'll request releases on all stable branches once those are merged" | 16:33 |
prometheanfire | thanks, just rejoined the channel at 10:33 | 16:34 |
mriedem | doh | 16:34 |
prometheanfire | :P | 16:34 |
dansmith | mriedem: got them sorry | 16:36 |
mriedem | thanks, np | 16:36 |
mriedem | welcome back to the land of the (koala) free | 16:37 |
*** andreas_s has quit IRC | 16:38 | |
*** gyee_ has joined #openstack-nova | 16:38 | |
*** ratailor has quit IRC | 16:40 | |
dansmith | you mean "welcome back to the land of internet" | 16:40 |
dansmith | and yes, it's good to be back on the right side of the wet noodle | 16:40 |
*** hemna_ has joined #openstack-nova | 16:41 | |
*** yangyapeng has quit IRC | 16:41 | |
*** psachin has quit IRC | 16:42 | |
*** artom_ is now known as artom | 16:43 | |
mriedem | tonyb: would you be against changing this release note on stable/pike to be 'fixes' or 'other' rather than 'features'? https://review.openstack.org/#/c/513717/4/releasenotes/notes/bug-1721179-87bc7b64215944c0.yaml | 16:43 |
*** jmlowe has quit IRC | 16:43 | |
mriedem | i don't want to alarm people reading stable patch release notes | 16:44 |
*** belmoreira has quit IRC | 16:44 | |
sean-k-mooney | efried: just comming to the end of the video on Ironic-Nova traits discussion. given the audiunce and topic it makes sense that only the ironic case was discussed but everything discussed also need to work with just plain nova | 16:44 |
efried | sean-k-mooney Yup. | 16:45 |
efried | sean-k-mooney Somehow... | 16:45 |
*** READ10 has quit IRC | 16:45 | |
sean-k-mooney | efried: so with nova i need to be able to request the vm to be placed on a hypervior with secure boot enabled on the host not the vm | 16:45 |
*** itlinux has quit IRC | 16:45 | |
efried | sean-k-mooney Does enabling secure boot on the hypervisor entail bouncing the compute service? | 16:46 |
sean-k-mooney | efried: i expcitly do not want to enabled it. i want to find a hypervisor with it enabled | 16:47 |
sean-k-mooney | efried: the config map thing iss irrelevent in this case as i do dont want to change configuration | 16:47 |
sean-k-mooney | efried: but yes it would require a host reboot to chage secureboot and it would require the host os is signed and the signing key is sored in the firmware | 16:48 |
*** pcaruana has quit IRC | 16:48 | |
efried | sean-k-mooney Okay, great. What I'm getting at is: if you enable secure boot on a hypervisor, and you have to restart the host (or at least restart the compute service) then we can make the case that the virt driver should set the SECURE_BOOT_CAPABLE trait on that host's RP. | 16:48 |
efried | sean-k-mooney Then you can just make your flavor require the SECURE_BOOT_CAPABLE trait and you're good. | 16:49 |
sean-k-mooney | efried: nope that was specifically reject at teh ptg. that is what i proposed | 16:49 |
efried | sean-k-mooney But if secure boot was something you could just flip on and off at will, then jaypipes would freak out if you had the virt driver detect that setting and add/remove the trait on the fly. | 16:49 |
sean-k-mooney | efried: no in a nova compute case its done once when you deploy the compute node. or infrequeently as a maintence envent on the compute node by the operator | 16:50 |
efried | sean-k-mooney As far as I'm concerned, if you have to reboot the whole host, all bets are off as far as traits are concerned. That's not a "dynamic trait". You could be adding or removing all kinds of capabilities across that reboot. | 16:50 |
sean-k-mooney | efried: yep i could have change the cpus or ram or anything | 16:51 |
efried | sean-k-mooney That's MHO; jaypipes or dansmith may wish to take a firmer stance. | 16:51 |
sean-k-mooney | efried: but specifcally dicovering if secure boot was enable or even the boot mode was reject as something the nova compute agent can do at the PTG | 16:51 |
*** Tom-Tom has joined #openstack-nova | 16:51 | |
openstackgerrit | Matthew Booth proposed openstack/nova stable/ocata: libvirt: bandwidth param should be set in guest migrate https://review.openstack.org/519635 | 16:52 |
openstackgerrit | Matthew Booth proposed openstack/nova stable/ocata: libvirt: Don't VIR_MIGRATE_NON_SHARED_INC without migrate_disks https://review.openstack.org/519636 | 16:52 |
efried | sean-k-mooney By compute "agent", do you mean the virt driver? | 16:52 |
sean-k-mooney | i mean nova compute agent e.g. devstack@n-cpu.service which includes the virt dirver | 16:53 |
efried | sean-k-mooney Hum, okay, that doesn't seem like the right thing. If not the compute agent deciding what traits to set on the host RP, then who? | 16:53 |
sean-k-mooney | efried: see lines 548-558 https://etherpad.openstack.org/p/nova-ptg-queens | 16:54 |
sean-k-mooney | efried: the stading policy on https://docs.openstack.org/nova/latest/contributor/policies.html#metrics-gathering was sighted as the reson to not allow this | 16:54 |
*** vladikr has quit IRC | 16:55 | |
mriedem | gibi: dansmith: do you remember fixing a bug in the target_cell fixture stuff where it was swallowing exceptions? | 16:56 |
*** Tom-Tom has quit IRC | 16:56 | |
dansmith | I think so | 16:56 |
mriedem | i thought that was fixed as part of the instance list series | 16:56 |
mriedem | ok i'll dig | 16:56 |
efried | sean-k-mooney Metrics seems like a poor analogy | 16:56 |
*** Vek has joined #openstack-nova | 16:57 | |
gibi | mriedem: yes, it rings a bell | 16:57 |
mriedem | https://github.com/openstack/nova/commit/af05d6208b4f95f1d9b882ed376552b75bb87506#diff-04a5f91f2963947e47ab25a8b291617a | 16:57 |
mriedem | consider the bell rung | 16:57 |
dansmith | mriedem: https://review.openstack.org/#/c/506312/ | 16:58 |
*** sapcc-bot has quit IRC | 16:58 | |
dansmith | heh | 16:58 |
*** sapcc-bot2 has joined #openstack-nova | 16:58 | |
mriedem | yup cool | 16:58 |
sean-k-mooney | efried: i agree. i really do think this should be something the compute agent/virt driver discovers hence my original proposal but the ask was to have an external enity other then nova set these traits on the compute node | 16:58 |
mriedem | part of that is going to pike | 16:58 |
*** carthaca_ has joined #openstack-nova | 16:58 | |
*** tpatzig_ has joined #openstack-nova | 16:58 | |
*** dgonzalez_ has joined #openstack-nova | 16:58 | |
efried | sean-k-mooney What external entity?? | 16:58 |
efried | sean-k-mooney The operator?? | 16:58 |
efried | sean-k-mooney I guess in the case of ironic they said it would be the ironic inspector and/or operator manually adding traits. | 16:59 |
sean-k-mooney | efried: anything that was not nova. i was planning to use a systemd script that invoked the openstack client as a workaround | 16:59 |
efried | sean-k-mooney That just seems like unnecessary work for something that could so easily be automated in the virt driver. | 17:00 |
sean-k-mooney | efried: yes in the ironic case it would be ironic itself, feed by the inspector | 17:00 |
*** tpatzig_ has quit IRC | 17:00 | |
*** carthaca_ has quit IRC | 17:00 | |
*** dgonzalez_ has quit IRC | 17:00 | |
*** itlinux has joined #openstack-nova | 17:01 | |
sean-k-mooney | efried: yes jaypipes dansmith strongly express a preference to not have the virt driver do this though. it was 4 oclock on fiday and everyone wanted to leave at that point so maybe in light of the ironic call there opipion has changed | 17:01 |
efried | sean-k-mooney Presumably it's not illegal for virt to set *any* traits on the host RP. So now we've got multiple different sources for traits. Who's the source of truth? If the operator deletes a trait that was set by virt, does it stay gone, or does virt restore it next time the pt runs? | 17:02 |
*** slaweq has quit IRC | 17:02 | |
*** jmlowe has joined #openstack-nova | 17:02 | |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Handle InstanceNotFound when setting password via metadata https://review.openstack.org/519659 | 17:02 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Fix CellDatabases fixture swallowing exceptions https://review.openstack.org/519702 | 17:02 |
sean-k-mooney | efried: i would assume that the virt driver would restore it as it can tell the differece between new hardware was added and the operator deleted it | 17:02 |
efried | right | 17:03 |
sean-k-mooney | efried: the virt driver would have to have a blacklist to support operators removing traits in this case | 17:03 |
efried | ugh | 17:03 |
sean-k-mooney | e.g. in the nova.conf | 17:03 |
*** jmlowe has quit IRC | 17:04 | |
mriedem | efried: replied to your question in https://review.openstack.org/#/c/515418/ about stable policy for fixing 500 errors | 17:04 |
efried | sean-k-mooney Is one idea that the operator sets host traits via nova.conf? | 17:04 |
sean-k-mooney | efried: im not sure you sould want to remove traits as an operator | 17:04 |
efried | mriedem Thanks, updated to +1 | 17:05 |
sean-k-mooney | efried: there was a proposal to allow the operator to add traits via nova.conf but not to set them. | 17:05 |
efried | sean-k-mooney Sorry, I didn't follow that. add but not set ? | 17:05 |
*** sahid has quit IRC | 17:05 | |
sean-k-mooney | e.g. so they could lable nic with things like hardware offload capable, which would be unioned with the set of traits discovered by the virt dirver | 17:06 |
sean-k-mooney | so append traits but override via config | 17:06 |
sean-k-mooney | this is all tided to your generic device manager work though so it will change as that evolves. | 17:07 |
mriedem | hongbin: can you add the conflicts note to the commit message for this backport? https://review.openstack.org/#/c/515418/1 | 17:07 |
mriedem | see details inline | 17:07 |
*** vladikr has joined #openstack-nova | 17:09 | |
*** eharney has quit IRC | 17:10 | |
*** janki has quit IRC | 17:10 | |
*** eharney has joined #openstack-nova | 17:11 | |
*** udesale has quit IRC | 17:11 | |
efried | sean-k-mooney Okay, yeah, I remember talking about a conf file-based black/white/trait-list for devices. So the multiple-sources-for-traits thing is something we'll have to deal with one way or another. | 17:12 |
efried | sean-k-mooney But the point remains that there will be some traits the virt driver should set, and some that should be set "externally" (be it from conf, osc, other service, magic wand, whatever). | 17:13 |
sean-k-mooney | efried: yes mid term we should have resouce providers and traits created by neutron for bandwith based scheuling in rocky | 17:13 |
efried | sean-k-mooney And (I guess on a case by case basis) we'll have to decide which of those is appropriate for which. | 17:13 |
efried | sean-k-mooney So for the case of secure boot capability, I contend that one should be okay for the virt driver to detect. | 17:13 |
*** lpetrut_ has quit IRC | 17:13 | |
efried | Because it's something that's either on or off for the entire life of the n-cpu process; and it's something that the virt driver can easily detect. | 17:14 |
sean-k-mooney | efried: yes. i would agree. if we can rely on the presence to also indicate that its enabled we remove the need for states at least for the nova case | 17:14 |
*** clarkb has quit IRC | 17:14 | |
*** yangyapeng has joined #openstack-nova | 17:14 | |
sean-k-mooney | efried: for ironic that will not work however | 17:14 |
efried | sean-k-mooney This may be a case where the difference is made in one's choice of words. | 17:15 |
*** yamamoto has quit IRC | 17:15 | |
*** yamahata has quit IRC | 17:15 | |
sean-k-mooney | well ideally i should use the same trait to represent secure boot will be used regradless of if its ironic | 17:16 |
sean-k-mooney | or nova vms | 17:16 |
sean-k-mooney | or zun containers for that matter | 17:16 |
efried | sean-k-mooney The host is capable of hosting secure boot VMs. If you say it like that, I think you can get away with it. | 17:16 |
efried | sean-k-mooney But if you say, "The host is capable of hosting secure boot VMs, but only if the secure boot feature is enabled," that's where you'll get into trouble. | 17:16 |
sean-k-mooney | efried: that not what im asking for. im asking for a host that iteslf is useing secure boot that can host my vm. | 17:17 |
efried | sean-k-mooney A host that was brought up without secure boot enablement is NOT CAPABLE OF HOSTING SECURE BOOT VMS. Ergo it should not have the trait. | 17:17 |
sean-k-mooney | the contraint is on the host not the vm. the vm could be bios booting | 17:17 |
efried | Yeah, I understand. | 17:17 |
sean-k-mooney | secure boot vms could imply that the vm is also useing uefi secure boot too | 17:18 |
efried | sean-k-mooney That's a sidetrack to the discussion. | 17:19 |
efried | sean-k-mooney What we're trying to get at here is that you want to be able to specify in your flavor: SECURE_BOOT_CAPABLE - and have your VM land on a host that's secure boot capable. | 17:19 |
crlb | jaypipes tox -epy27,pep8 | tee ../nova-patched-tox.log | 17:19 |
crlb | py27: commands succeeded | 17:19 |
crlb | pep8: commands succeeded | 17:19 |
*** lucasagomes is now known as lucas-afk | 17:19 | |
*** salv-orl_ has joined #openstack-nova | 17:20 | |
sean-k-mooney | efried: or i specify SECURE_BOOT_CAPABLE and i land on a hypvervior(in the xen/kvm meaning) that can excute the vm kernel using a uefi bootload with secureboot enabel in the vm | 17:20 |
sean-k-mooney | we have to be able to treet both seperatly | 17:21 |
*** yamamoto has joined #openstack-nova | 17:21 | |
sean-k-mooney | but i think we are violently agreeing mostly :) | 17:21 |
efried | sean-k-mooney That's fine; so they're separate traits | 17:21 |
efried | sean-k-mooney What you *don't* want is to have the scheduler decide on a host, send the boot request, and then have virt punt because it can't do secure boot, resulting in retries, which are limited to the cell, etc. etc. | 17:21 |
sean-k-mooney | one one trait againts too different things e.g. traits against the host and traits againts the virtulisation context. | 17:22 |
sean-k-mooney | efried: yes that would be bad | 17:22 |
*** salv-orlando has quit IRC | 17:22 | |
*** yangyapeng has quit IRC | 17:22 | |
*** clarkb has joined #openstack-nova | 17:22 | |
sean-k-mooney | * that was ment to be or one trait. | 17:23 |
mriedem | bauzas: dansmith: sdague: i'm thinking we want these in for ocata and newton before eol too, since i'll be cutting a release today or tomorrow for that cve https://review.openstack.org/#/q/Icb418f2be575bb2ba82756fdeb67b24a28950746 | 17:23 |
*** Apoorva has joined #openstack-nova | 17:23 | |
mriedem | fixes bug 1675676 | 17:23 |
openstack | bug 1719730 in OpenStack Compute (nova) ocata "duplicate for #1675676 Reschedule after the late affinity check fails with "'NoneType' object is not iterable"" [High,In progress] https://launchpad.net/bugs/1719730 - Assigned to Matt Riedemann (mriedem) | 17:23 |
efried | sean-k-mooney Ah, so now we're talking about overloading the trait semantic to say, "not only do I want to make sure I wind up on a SECURE_BOOT_CAPABLE host, I also want this vm to be secure boot." | 17:23 |
efried | sean-k-mooney And *that* is the cardinal sin according to the PTB. | 17:24 |
*** sridharg has quit IRC | 17:24 | |
efried | sean-k-mooney They would have you specify that second thing via some completely separate mechanism. | 17:24 |
efried | sean-k-mooney Perhaps image metadata; perhaps a different extra_spec; perhaps some other confy thing... | 17:24 |
sean-k-mooney | efried: the traith that you use for secure boot should not change based on the compute context you choose | 17:24 |
sean-k-mooney | e.g. it should be the same for vm,baremetal and contianer | 17:25 |
sean-k-mooney | in the ironic case the compute context is the same as the host | 17:25 |
efried | sean-k-mooney I'm on board with that. | 17:25 |
sean-k-mooney | in the containter and vm case the are not | 17:25 |
sdague | mriedem: make it so | 17:26 |
sdague | i.e. +A | 17:26 |
*** yamamoto has quit IRC | 17:26 | |
*** moshele has quit IRC | 17:26 | |
mriedem | thanks | 17:26 |
sean-k-mooney | efried: personally i would see these are two differnt traits request. one is find me a host with X and the second is find me a hypervisor capable of emulating Y and then we take the intersection of both | 17:26 |
openstackgerrit | Elod Illes proposed openstack/nova master: Use FakeLiveMigrateDriver in notification test https://review.openstack.org/519405 | 17:30 |
openstackgerrit | Eric Fried proposed openstack/nova master: Test helper: validate provider summaries https://review.openstack.org/518982 | 17:30 |
*** edand has quit IRC | 17:30 | |
efried | gibi I rebased this on top of the series, and added affordance for traits too ^ | 17:30 |
*** chyka has joined #openstack-nova | 17:33 | |
*** sshwarts has quit IRC | 17:36 | |
*** yamamoto has joined #openstack-nova | 17:36 | |
*** bauzas_ has joined #openstack-nova | 17:36 | |
*** fragatina has joined #openstack-nova | 17:37 | |
*** fragatina has quit IRC | 17:37 | |
*** fragatina has joined #openstack-nova | 17:38 | |
*** bauzas_ has quit IRC | 17:39 | |
*** yamamoto_ has joined #openstack-nova | 17:40 | |
*** gyee has quit IRC | 17:40 | |
*** yamamoto has quit IRC | 17:41 | |
*** AlexeyAbashkin has quit IRC | 17:41 | |
*** gbarros has joined #openstack-nova | 17:43 | |
*** gyee has joined #openstack-nova | 17:44 | |
*** Apoorva has quit IRC | 17:44 | |
*** salv-orl_ has quit IRC | 17:44 | |
*** Apoorva has joined #openstack-nova | 17:44 | |
*** dtantsur|brb is now known as dtantsur | 17:45 | |
*** bauzas_ has joined #openstack-nova | 17:45 | |
*** bauzas has left #openstack-nova | 17:45 | |
*** yamamoto_ has quit IRC | 17:45 | |
*** bauzas_ has quit IRC | 17:45 | |
*** bauzas_ has joined #openstack-nova | 17:46 | |
*** bauzas_ is now known as bauzas | 17:46 | |
*** fragatina has quit IRC | 17:48 | |
*** Swami has joined #openstack-nova | 17:48 | |
*** yangyapeng has joined #openstack-nova | 17:49 | |
*** jdillaman has quit IRC | 17:50 | |
bauzas | yay, I got a new bouncer \o/ | 17:50 |
*** salv-orlando has joined #openstack-nova | 17:52 | |
*** yamamoto has joined #openstack-nova | 17:52 | |
*** stelucz has joined #openstack-nova | 17:52 | |
*** derekh has quit IRC | 17:54 | |
*** salv-orlando has quit IRC | 17:54 | |
*** salv-orlando has joined #openstack-nova | 17:55 | |
*** yangyapeng has quit IRC | 17:57 | |
hongbin | mriedem: ack | 17:58 |
*** salv-orl_ has joined #openstack-nova | 17:58 | |
*** fragatina has joined #openstack-nova | 17:58 | |
*** abalutoiu has joined #openstack-nova | 17:58 | |
*** yamahata has joined #openstack-nova | 17:59 | |
*** Tom-Tom has joined #openstack-nova | 18:01 | |
*** salv-orlando has quit IRC | 18:01 | |
*** yamamoto has quit IRC | 18:03 | |
*** fragatina has quit IRC | 18:04 | |
mriedem | alaski: didn't you at one point have a doc on your thoughts for the tasks api, or was that just a patch that never merged? | 18:04 |
mriedem | i found a really old juno spec https://review.openstack.org/#/c/86938/ | 18:05 |
mriedem | oh maybe https://specs.openstack.org/openstack/nova-specs/specs/backlog/approved/instance-tasks.html | 18:06 |
*** Tom-Tom has quit IRC | 18:06 | |
*** claudiub|2 has joined #openstack-nova | 18:08 | |
*** claudiub has quit IRC | 18:11 | |
mordred | mriedem: questoin about scheduler hints ... the API ref https://developer.openstack.org/api-ref/compute/#create-server seems to indicate that os:scheduler_hints can be put as a key on the server dict in the request, or that OS-SCH-HNT:scheduler_hints can be used outside of the server dict. | 18:12 |
mordred | mriedem: but there's an old guide https://docs.openstack.org/kilo/config-reference/content/section_compute-scheduler.html that shows using os:scheduler_hints at the top level ... and i just got a patch: https://review.openstack.org/#/c/519705/2/shade/openstackcloud.py to move using os:scheduler_hints to outside of the server dict | 18:12 |
mordred | mriedem: is the story that it always goes into the top level of the payload and that either key works? | 18:13 |
mriedem | i think the latter, i think it's a top-level key https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/schemas/scheduler_hints.py#L75 | 18:16 |
mriedem | and either works | 18:16 |
mriedem | https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/scheduler_hints.py#L18 | 18:16 |
sean-k-mooney | for consitnecy sake shade and osc should proably pass scheduler hints the same way | 18:17 |
mriedem | i don't think you can send it in the server body | 18:17 |
mriedem | it's a top-level key | 18:17 |
mriedem | the server create api-ref in nova is just confusing | 18:17 |
mriedem | "The dictionary of data to send to the scheduler. Alternatively, you can specify OS-SCH-HNT:scheduler_hints as the key in the request body." | 18:18 |
mriedem | that's just saying, you can use either key | 18:18 |
mriedem | i don't know why we have 2 keys, i'm assuming v2 api extension fun | 18:18 |
mriedem | gmann, sdague or oomichi might remember | 18:18 |
mriedem | mordred: yeah e.g. https://docs.openstack.org/nova/latest/admin/configuration/schedulers.html#differenthostfilter | 18:20 |
mriedem | "With the API, use the os:scheduler_hints key. For example:" | 18:20 |
mordred | ok. cool. so it just shows up weird in the api ref docs ... | 18:20 |
mordred | since it's in the same block as key_name and whatnot | 18:20 |
mriedem | yeah, we should be clear that it's a top-level key | 18:20 |
mriedem | yup | 18:20 |
mriedem | i, or someone else, can amend the doc | 18:21 |
mordred | cool. thanks! | 18:22 |
*** fragatina has joined #openstack-nova | 18:22 | |
openstackgerrit | Elod Illes proposed openstack/nova master: Transform instance-live_migration_pre notification https://review.openstack.org/482070 | 18:23 |
*** salv-orl_ has quit IRC | 18:24 | |
mriedem | the example request shows it's top-level too | 18:24 |
mriedem | https://developer.openstack.org/api-ref/compute/#create-server | 18:24 |
mriedem | so that helps somewhat | 18:24 |
*** salv-orlando has joined #openstack-nova | 18:24 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: make a note about os:scheduler_hints being a top-level key https://review.openstack.org/519717 | 18:27 |
mriedem | mordred: see if this floats your boat ^ | 18:27 |
jaypipes | crlb: I'll pull your patch and run locally and see what's up. | 18:28 |
*** salv-orlando has quit IRC | 18:28 | |
mordred | mriedem: yes! that's super helpful | 18:29 |
*** stelucz has quit IRC | 18:29 | |
*** yamamoto has joined #openstack-nova | 18:29 | |
*** stelucz has joined #openstack-nova | 18:29 | |
mriedem | cburgess: semi interesting live migration abort scenario for you | 18:30 |
cburgess | OK shoot | 18:30 |
mriedem | cburgess: today you can list in-progress live migrations for a given server | 18:30 |
mriedem | https://github.com/openstack/nova/blob/16.0.0/nova/db/sqlalchemy/api.py#L4789 | 18:30 |
mriedem | if it's in one of those 4 states ^ | 18:30 |
mriedem | however, you can only abort a running migration https://github.com/openstack/nova/blob/16.0.0/nova/compute/api.py#L3952 | 18:30 |
mriedem | presumably because the migration transitions fast enough that it gets to the hypervisor and starts running | 18:31 |
mriedem | however, | 18:31 |
*** stelucz has quit IRC | 18:31 | |
mriedem | we have this max_concurrent_live_migrations config option in compute which defaults to 1, | 18:31 |
*** stelucz has joined #openstack-nova | 18:31 | |
mriedem | so you could be trying to move 10 instances off a host via live migration, and by default those are all going to queue up, | 18:31 |
mriedem | let's say you wanted to abort half of those (1 in 10 are running) | 18:31 |
mriedem | the abort would fail | 18:31 |
cburgess | mriedem Give me a few... sorry had someone talking to me | 18:32 |
*** stelucz has quit IRC | 18:32 | |
mriedem | one solution is just bump up your max_concurrent_live_migrations value to 10, but then you hit issues with running a bunch of migrations at once | 18:32 |
*** stelucz has joined #openstack-nova | 18:32 | |
mriedem | np, just looking for some ops feedback | 18:32 |
alaski | mriedem: there was http://git.openstack.org/cgit/openstack/nova-specs/tree/specs/backlog/approved/instance-tasks.rst | 18:33 |
*** stelucz has quit IRC | 18:34 | |
mriedem | alaski: yup i found it later via google, i was mistakenly searching the nova docs | 18:34 |
alaski | cool | 18:34 |
*** stelucz has joined #openstack-nova | 18:35 | |
*** yamamoto has quit IRC | 18:35 | |
*** stelucz has quit IRC | 18:35 | |
*** prometheanfire has left #openstack-nova | 18:35 | |
*** yamamoto has joined #openstack-nova | 18:36 | |
*** tesseract has quit IRC | 18:37 | |
*** yangyapeng has joined #openstack-nova | 18:37 | |
openstackgerrit | Hongbin Lu proposed openstack/nova stable/pike: Handle exception on adding secgroup https://review.openstack.org/515418 | 18:38 |
openstackgerrit | Hongbin Lu proposed openstack/nova stable/ocata: Handle exception on adding secgroup https://review.openstack.org/515421 | 18:40 |
jaypipes | crlb: https://pastebin.com/maKwN7pM -- failures are reproduceable for me. Only suggestion I have is do an rm -rf .tox/ and then retry the tox -epy27,pep8. Other than that... perhaps you forgot to git review and push your latest changes to gerrit? | 18:42 |
*** yangyapeng has quit IRC | 18:43 | |
efried | crlb If you run tox with -r it'll rebuild the venv. | 18:43 |
openstackgerrit | Merged openstack/nova stable/newton: Functional regression test for evacuate with a target https://review.openstack.org/516118 | 18:43 |
efried | easier than rm -rf .tox | 18:43 |
openstackgerrit | Hongbin Lu proposed openstack/nova stable/pike: Handle exception on adding secgroup https://review.openstack.org/515418 | 18:45 |
openstackgerrit | Hongbin Lu proposed openstack/nova stable/pike: Handle exception on adding secgroup https://review.openstack.org/515418 | 18:46 |
*** stelucz has joined #openstack-nova | 18:46 | |
*** READ10 has joined #openstack-nova | 18:47 | |
openstackgerrit | Hongbin Lu proposed openstack/nova stable/ocata: Handle exception on adding secgroup https://review.openstack.org/515421 | 18:48 |
*** snasir has quit IRC | 18:49 | |
*** snasir has joined #openstack-nova | 18:50 | |
*** gfhellma has joined #openstack-nova | 18:55 | |
*** lpetrut_ has joined #openstack-nova | 18:55 | |
*** yangyapeng has joined #openstack-nova | 18:56 | |
*** jpena is now known as jpena|off | 18:59 | |
*** moshele has joined #openstack-nova | 18:59 | |
*** yangyapeng has quit IRC | 19:00 | |
*** tjones has joined #openstack-nova | 19:01 | |
*** tasker has quit IRC | 19:03 | |
*** moshele has quit IRC | 19:10 | |
*** yamamoto has quit IRC | 19:11 | |
openstackgerrit | Hongbin Lu proposed openstack/nova master: Handle not found error on taking snapshot https://review.openstack.org/511074 | 19:12 |
*** yangyapeng has joined #openstack-nova | 19:17 | |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova master: Add error message on metadata API https://review.openstack.org/517152 | 19:17 |
crlb | jaypipes: checked pastebin. The test_versions.py that you are running is not the latest, but patch set 2 updated it and dealt with the pep8 errors. If i try it review, I get: git review | 19:20 |
crlb | Creating a git remote called "gerrit" that maps to: | 19:20 |
crlb | ssh://crlb@review.openstack.org:29418/openstack/nova.git | 19:20 |
crlb | No changes between HEAD and gerrit/master. Submitting for review would | 19:20 |
crlb | be pointless. | 19:20 |
*** dave-mccowan has quit IRC | 19:21 | |
*** tjones has quit IRC | 19:21 | |
mnaser | crlb: did you commit anything on your local branch? | 19:22 |
mnaser | also, puppet fun, we suspect that we're seeing some weird timeout in puppet openstack intergration due to this change - https://github.com/openstack/puppet-nova/commit/df638e2526d2d957318519dfcfb9098cb7726095 | 19:22 |
mnaser | we go from multiple threads in a single process, to single thread in multiple processes | 19:23 |
*** AlexeyAbashkin has joined #openstack-nova | 19:23 | |
mnaser | but it seems to have spawned some strange timeouts with floating ip assignment. what's the nova team stance on this in terms of "best practice" ? | 19:23 |
*** yangyapeng has quit IRC | 19:25 | |
efried | jaypipes Can I do stuff to this series? https://review.openstack.org/#/c/516778/ | 19:25 |
*** Sukhdev has joined #openstack-nova | 19:26 | |
mnaser | looks like devstack uses API_WORKERS for # of processes in uwsgi, and has an 'enable-threads' feature | 19:27 |
*** AlexeyAbashkin has quit IRC | 19:28 | |
*** abalutoiu has quit IRC | 19:32 | |
*** gbarros has quit IRC | 19:35 | |
crlb | jaypipes. I'm sorry, I didn't realize that subsequent git reviews would give you different change numbers. The 2nd and 3rd patch sets are on change #518119. | 19:36 |
*** READ10 has quit IRC | 19:43 | |
*** owalsh has quit IRC | 19:43 | |
*** owalsh has joined #openstack-nova | 19:44 | |
*** dtantsur is now known as dtantsur|afk | 19:45 | |
mriedem | dansmith: i reckon i should put out a release note for that cve on the stable branches, so i guess i'll do that right quick | 19:45 |
mriedem | and we can ram it through | 19:45 |
efried | crlb To be sure you are modifying the same review, the Change-Id in the commit message needs to remain the same. If you run `git commit` with the `--amend` flag, it'll make sure you're staying on the same change set. Otherwise (without `--amend`) it'll create a new change on top of the previous one and you'll wind up with separate reviews. | 19:46 |
*** ralonsoh has quit IRC | 19:46 | |
efried | crlb At this point I believe you'll want to squash these changes together, assuming they're going after the same thing. | 19:48 |
*** itlinux has quit IRC | 19:54 | |
*** Sukhdev has quit IRC | 20:01 | |
*** gabor_antal has quit IRC | 20:06 | |
*** gabor_antal has joined #openstack-nova | 20:06 | |
*** gszasz has quit IRC | 20:10 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add security release note for OSSA-2017-005 https://review.openstack.org/519738 | 20:12 |
mriedem | sdague: dansmith: ^ if this looks ok, let's get it in and i'll do backports | 20:12 |
*** yamamoto has joined #openstack-nova | 20:12 | |
*** pcaruana has joined #openstack-nova | 20:12 | |
*** dikonoor has joined #openstack-nova | 20:15 | |
*** yangyapeng has joined #openstack-nova | 20:20 | |
dansmith | mriedem: I dunno if that's common for a cve or not (to mention the cve but not describe it) | 20:21 |
mriedem | that's what the link is for | 20:21 |
dansmith | but if it is, it's cool with me | 20:21 |
dansmith | yeah, just seems sparse compared to what we'd normally do | 20:22 |
mriedem | let me find an old one | 20:22 |
mriedem | https://docs.openstack.org/releasenotes/nova/newton.html#security-issues | 20:22 |
dansmith | normally -> for a normal bug | 20:22 |
mriedem | https://docs.openstack.org/releasenotes/nova/newton.html#id15 | 20:23 |
dansmith | mriedem: even that example is more descriptive | 20:23 |
mriedem | ok i'll wordsmith this a bit | 20:23 |
*** Tom-Tom has joined #openstack-nova | 20:23 | |
*** AlexeyAbashkin has joined #openstack-nova | 20:23 | |
*** yamamoto has quit IRC | 20:23 | |
dansmith | I don't care that much, I would just expect it to say "this includes a fix for cve 123, which could result in instances rebuilt with images not normally allowed on their host" or something | 20:24 |
*** yangyapeng has quit IRC | 20:25 | |
*** dikonoor has quit IRC | 20:26 | |
openstackgerrit | Merged openstack/nova stable/pike: Correct log message when removing a security group https://review.openstack.org/519583 | 20:26 |
*** dikonoor has joined #openstack-nova | 20:26 | |
openstackgerrit | Merged openstack/nova master: Validate new image via scheduler during rebuild https://review.openstack.org/519662 | 20:27 |
openstackgerrit | Merged openstack/nova stable/pike: Validate new image via scheduler during rebuild https://review.openstack.org/519672 | 20:27 |
openstackgerrit | Merged openstack/nova master: Add FlavorPayload.description for versioned notifications https://review.openstack.org/514782 | 20:27 |
openstackgerrit | Merged openstack/nova stable/pike: Pick ironic nodes without VCPU set https://review.openstack.org/513678 | 20:27 |
openstackgerrit | Merged openstack/nova stable/ocata: Set group_members when converting to legacy request spec https://review.openstack.org/517860 | 20:27 |
*** Tom-Tom has quit IRC | 20:27 | |
*** yangyapeng has joined #openstack-nova | 20:27 | |
openstackgerrit | Merged openstack/nova stable/ocata: Validate new image via scheduler during rebuild https://review.openstack.org/519681 | 20:27 |
*** AlexeyAbashkin has quit IRC | 20:28 | |
*** eharney has quit IRC | 20:28 | |
*** yangyapeng has quit IRC | 20:32 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add security release note for OSSA-2017-005 https://review.openstack.org/519738 | 20:32 |
mriedem | dansmith: ^ added more detail | 20:32 |
dansmith | mriedem: thanks | 20:33 |
mriedem | jaypipes: melwitt: can you hit https://review.openstack.org/#/c/519738/ so i can start backports? | 20:34 |
dansmith | mriedem: melwitt is out for a couple weeks, AFAIK | 20:34 |
mriedem | orly | 20:34 |
*** tojuvone has quit IRC | 20:34 | |
mriedem | plural? | 20:34 |
dansmith | this week and part of next week I think | 20:35 |
mriedem | ok | 20:35 |
*** tojuvone has joined #openstack-nova | 20:35 | |
dansmith | through Nov 20 | 20:35 |
mriedem | i'll give my jay and sean pings 5 more minutes and then i say we just approve | 20:35 |
dansmith | I can just fast approve. it's a reno. | 20:35 |
mriedem | ok | 20:35 |
mriedem | the fixes for newton are failing devstack, presumably because neutron is eol and things are just not going to work there anymore | 20:36 |
mriedem | if only we had LTS support... | 20:37 |
dansmith | um... | 20:37 |
mriedem | yeah, fudge, devstack in newton can't find stable/newton the branch, so it pip installs from master | 20:37 |
mriedem | farq | 20:37 |
dansmith | for nova? | 20:38 |
mriedem | no, devstack fails to setup | 20:38 |
mriedem | http://logs.openstack.org/19/516119/1/gate/legacy-tempest-dsvm-neutron-full/f24b24b/logs/devstacklog.txt.gz#_2017-11-14_17_24_29_768 | 20:38 |
mriedem | b/c newton is eol | 20:38 |
dansmith | that's because neutron's branch is gone right? | 20:39 |
dansmith | any one project deletes their branch and devstack stops working yes? | 20:39 |
mriedem | yes | 20:39 |
mriedem | well, anything we need for our base install | 20:39 |
dansmith | so that's what I meant by "for nova?" ... meaning "for which project?" | 20:40 |
mriedem | i could remove neutron from ENABLED_SERVICES but then nova just won't work, unless nova-net | 20:40 |
mriedem | which in newton works w/o cellsv1.... | 20:40 |
mriedem | :) | 20:40 |
mriedem | i think | 20:40 |
dansmith | or ask for the branch to be re-created based on the discussion at summit | 20:40 |
dansmith | mordred said it was easy to just resurrect that | 20:40 |
mriedem | it is, but then it would be EOL'ed next week ago | 20:41 |
mriedem | *again | 20:41 |
dansmith | or we start disabling jobs that don't work, per the discussion :) | 20:41 |
mordred | whatdididonow? | 20:41 |
mriedem | in this case it's easier probably to just disable the job | 20:41 |
mriedem | mordred: we have a cve fix for newton that can't pass the gate b/c dsvm jobs fail trying to install neutron, | 20:42 |
mriedem | because neutron newton is eol | 20:42 |
dansmith | mordred: we're trying to backport a cve fix to nova's newton branch and can't because neutron has eol'd theirs | 20:42 |
mordred | ah. yah. | 20:42 |
tonyb | mriedem: s/features/fixes/ on that backport is fine by me. | 20:43 |
mriedem | um, cinder newton must be eol too | 20:43 |
mriedem | http://logs.openstack.org/19/516119/1/gate/legacy-tempest-dsvm-cells/9e11a37/logs/devstacklog.txt.gz#_2017-11-14_17_21_45_400 | 20:43 |
tonyb | Oh rats :( | 20:43 |
mriedem | it is | 20:43 |
smcginnis | mriedem: Everyone is EOL on Newton I thought. | 20:43 |
mriedem | so yeah, dsvm just won't work on netwon | 20:43 |
mriedem | smcginnis: except nova | 20:44 |
smcginnis | mriedem: You snooze you lose. | 20:44 |
tonyb | Yeah everyone except nova, ironic and cycle-trainling | 20:44 |
mriedem | tonyb: ok so looking at https://review.openstack.org/#/c/516119/ | 20:44 |
dansmith | mriedem: the question comes up if we should even release newton if we never ran a devstack job against it | 20:44 |
mriedem | tonyb: i'd have to skip these jobs | 20:44 |
mriedem | legacy-tempest-dsvm-cellsFAILURE in 19m 44slegacy-tempest-dsvm-multinode-live-migrationFAILURE in 19m 43slegacy-tempest-dsvm-neutron-fullFAILURE in 18m 41s | 20:44 |
openstackgerrit | Merged openstack/nova stable/ocata: Correct log message when removing a security group https://review.openstack.org/519585 | 20:44 |
mriedem | dansmith: well this patch passed dsvm on 10/30 https://review.openstack.org/#/c/516119/ | 20:45 |
dansmith | mriedem: but you can't get a dsvm job run on the actual cve fix anymore right? | 20:45 |
mriedem | somehow https://review.openstack.org/#/c/519684/ did run legacy-tempest-dsvm-neutron-full earlier today | 20:45 |
mriedem | ^ did | 20:46 |
tonyb | mriedem: Yeah we only have 2 options patch devstack to work with eol-tags or accept the risk and drop the dsvm tests | 20:46 |
* tonyb messed that one up sorry | 20:46 | |
mriedem | i could try patching devstack, but it might be awhile | 20:47 |
dansmith | mriedem: hmm, how did that work? | 20:48 |
mriedem | dansmith: no idea | 20:48 |
mriedem | not sure if there are mirrors involved | 20:48 |
tonyb | I can also try to see how bad it is but not until after I've dropped the kids at school | 20:48 |
mriedem | ok, so i'm goign to get the reno backported for pike and ocata, then get the release requests up for those branches and will deal with newton | 20:48 |
mriedem | s/school/pool/ | 20:48 |
tonyb | ;P | 20:48 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Add security release note for OSSA-2017-005 https://review.openstack.org/519752 | 20:50 |
tonyb | newton is xenial right? | 20:50 |
* tonyb needs to build a fresh VM for testing the devstack change | 20:50 | |
mriedem | yes | 20:50 |
tonyb | cool | 20:50 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: Add security release note for OSSA-2017-005 https://review.openstack.org/519753 | 20:51 |
*** gouthamr has quit IRC | 20:53 | |
dansmith | mriedem: tonyb: +2 on the reno backports | 20:55 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/newton: Pass requested_destination in filter_properties https://review.openstack.org/516119 | 20:55 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/newton: Validate new image via scheduler during rebuild https://review.openstack.org/519684 | 20:55 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/newton: Add security release note for OSSA-2017-005 https://review.openstack.org/519755 | 20:55 |
*** smatzek has quit IRC | 20:59 | |
tonyb | looks like they're good to go on > newton | 21:00 |
mriedem | yes | 21:00 |
mriedem | i'll propose releases for pike and ocata once the reno is merged | 21:00 |
openstackgerrit | Eric Fried proposed openstack/nova master: begin refactor AllocCandidates._get_by_filters() https://review.openstack.org/516778 | 21:00 |
openstackgerrit | Eric Fried proposed openstack/nova master: build ProviderSummary objects in sep function https://review.openstack.org/516779 | 21:00 |
openstackgerrit | Eric Fried proposed openstack/nova master: build alloc request resources for shared resources https://review.openstack.org/516780 | 21:00 |
openstackgerrit | Eric Fried proposed openstack/nova master: create allocation request for single provider https://review.openstack.org/516781 | 21:00 |
openstackgerrit | Eric Fried proposed openstack/nova master: finish refactor AllocCandidates._get_by_filters() https://review.openstack.org/516782 | 21:00 |
openstackgerrit | Eric Fried proposed openstack/nova master: required traits for no sharing providers https://review.openstack.org/517027 | 21:00 |
openstackgerrit | Eric Fried proposed openstack/nova master: Add ProviderSummary.resource_class_names @property https://review.openstack.org/517646 | 21:00 |
openstackgerrit | Eric Fried proposed openstack/nova master: handle traits with sharing providers https://review.openstack.org/517119 | 21:00 |
openstackgerrit | Eric Fried proposed openstack/nova master: Test alloc_cands with non overlapping sharing RPs https://review.openstack.org/519380 | 21:00 |
openstackgerrit | Eric Fried proposed openstack/nova master: Test alloc_cands with one RP shared between two RPs https://review.openstack.org/519617 | 21:00 |
openstackgerrit | Eric Fried proposed openstack/nova master: Test allocation candidates: multiple aggregates https://review.openstack.org/518633 | 21:00 |
openstackgerrit | Eric Fried proposed openstack/nova master: Test helper: validate provider summaries https://review.openstack.org/518982 | 21:00 |
efried | jaypipes ^ Addressed most of the comments in this series. | 21:00 |
efried | jaypipes There are still bugs; but I think alex_xu might be willing to +2 the bottom one at this point. | 21:01 |
*** smatzek has joined #openstack-nova | 21:03 | |
tonyb | mriedem: cool, I'll keep an eye out | 21:03 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif stable/pike: Updated from global requirements https://review.openstack.org/493146 | 21:04 |
*** moshele has joined #openstack-nova | 21:05 | |
*** amodi has quit IRC | 21:06 | |
*** smatzek has quit IRC | 21:07 | |
*** jdillaman has joined #openstack-nova | 21:12 | |
efried | dansmith stephenfin When you get a chance, would you please put eyeballs back on https://review.openstack.org/#/c/377138/ -- at this point it's just in a sisyphean rebase cycle. | 21:15 |
*** salv-orlando has joined #openstack-nova | 21:18 | |
moshele | stephenfin: hi | 21:20 |
*** lyan has quit IRC | 21:21 | |
oomichi | mriedem: scheduler-hints is one of gross API designs. Do we have more related patches except https://review.openstack.org/#/c/519717 ? | 21:22 |
*** gouthamr has joined #openstack-nova | 21:23 | |
*** acormier has joined #openstack-nova | 21:23 | |
moshele | ramishra: hi | 21:24 |
*** slaweq has joined #openstack-nova | 21:25 | |
*** AlexeyAbashkin has joined #openstack-nova | 21:25 | |
mriedem | oomichi: nope | 21:25 |
mriedem | dansmith: here is my devstack newton-eol trickery https://review.openstack.org/519758 | 21:26 |
mriedem | thanks to google for providing bash substring tips | 21:26 |
dansmith | mriedem: wow that's... special | 21:26 |
mriedem | heh | 21:26 |
mriedem | eol_tag=$(echo ${git_ref:7:${#git_ref}})-eol | 21:27 |
mriedem | is the best piece of coding i've done in ages | 21:27 |
dansmith | mriedem: heh | 21:27 |
efried | mriedem What's the 7-char prefix there? | 21:27 |
mriedem | stable/ | 21:28 |
dansmith | stable/ | 21:28 |
mriedem | strip that off | 21:28 |
efried | Then eol_tag=${git_ref#stable/}-eol would work too | 21:28 |
efried | You know, if you're a shell guy rather than a python guy. | 21:28 |
mriedem | so it doies | 21:29 |
mriedem | i am not a shell guy | 21:29 |
*** AlexeyAbashkin has quit IRC | 21:29 | |
efried | Furthermore, that syntax will work all the way back to ksh88 | 21:29 |
efried | Which is important. | 21:29 |
efried | In case you're running this script on AIX | 21:29 |
mriedem | fuck i wish | 21:29 |
mriedem | updated, thanks for the tip | 21:32 |
cburgess | mriedem OK sorry... things got.. crazy here. I'm reading what your wrote hours ago now. | 21:32 |
efried | mriedem Sure thing. But also x=foo is the same as x=$(echo foo) | 21:33 |
cburgess | mriedem OK so... back to the live migration cancel thing, I think its probably reasonable to be able to cancel queued or running. | 21:34 |
*** jmlowe has joined #openstack-nova | 21:35 | |
*** felipemonteiro has joined #openstack-nova | 21:35 | |
*** lyan has joined #openstack-nova | 21:38 | |
*** lpetrut_ has quit IRC | 21:41 | |
*** rcernin has joined #openstack-nova | 21:44 | |
mriedem | cburgess: ack | 21:45 |
mriedem | good to know | 21:45 |
cburgess | I agree there is a bit of race condition though. | 21:46 |
mriedem | apparently our operators are annoyed that they queue up a bunch of live migrations and can't abort some of them before they hit the hypervisor | 21:46 |
*** oomichi is now known as oomichi_afk | 21:46 | |
mriedem | tonyb: ok redoing this newton series with the depends-on | 21:46 |
cburgess | As in you want the API to fast fail if the state is wrong, but that state can change by the time you hit the compute node. | 21:46 |
mriedem | and had to slip another funtional test backport in the mix | 21:46 |
mriedem | cburgess: yeah i think we'd just have to ignore it if we got to the compute and something changed, | 21:47 |
cburgess | mriedem Yeah that makes sense. So I think queued and running are the 2 states that aren't really transitory. So just block action on the transitory states. | 21:47 |
mriedem | although if it went from queued->running, we should be able to abort in the compute | 21:47 |
mriedem | as normal | 21:47 |
cburgess | mriedem Right thats my thought. Fast fail on the API, and then another check on compute and fail if you aren't in an allowed state on compute. | 21:47 |
*** MasterOfBugs has joined #openstack-nova | 21:47 | |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/newton: Pass requested_destination in filter_properties https://review.openstack.org/516119 | 21:47 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/newton: Validate new image via scheduler during rebuild https://review.openstack.org/519684 | 21:47 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/newton: Add security release note for OSSA-2017-005 https://review.openstack.org/519755 | 21:47 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/newton: Fix non-parameterized service id in hypervisors sample tests https://review.openstack.org/519763 | 21:47 |
*** acormier has quit IRC | 21:48 | |
*** acormier has joined #openstack-nova | 21:48 | |
*** gouthamr has quit IRC | 21:49 | |
* tonyb takes kids to school be back in a bit | 21:51 | |
*** acormier has quit IRC | 21:53 | |
*** artom has quit IRC | 21:53 | |
*** pramodrj07 has joined #openstack-nova | 21:56 | |
*** smatzek has joined #openstack-nova | 21:58 | |
*** adreznec_ has joined #openstack-nova | 21:58 | |
*** adreznec has quit IRC | 21:59 | |
*** adreznec_ is now known as adreznec | 21:59 | |
*** zigo has quit IRC | 21:59 | |
*** MasterOfBugs has quit IRC | 22:00 | |
*** zigo has joined #openstack-nova | 22:01 | |
*** smatzek has quit IRC | 22:02 | |
*** pcaruana has quit IRC | 22:02 | |
*** pcaruana has joined #openstack-nova | 22:06 | |
*** gfhellma_ has joined #openstack-nova | 22:06 | |
*** pcaruana has quit IRC | 22:07 | |
*** gfhellma has quit IRC | 22:08 | |
openstackgerrit | Merged openstack/nova master: placement: AllocCands.get_by_{filters => requests} https://review.openstack.org/514092 | 22:09 |
*** threestrands has joined #openstack-nova | 22:13 | |
*** pramodrj07 has quit IRC | 22:13 | |
openstackgerrit | Merged openstack/nova master: Deduplicate server_group samples https://review.openstack.org/519629 | 22:16 |
*** jmlowe has quit IRC | 22:19 | |
*** lpetrut_ has joined #openstack-nova | 22:20 | |
openstackgerrit | Merged openstack/nova stable/pike: Add security release note for OSSA-2017-005 https://review.openstack.org/519752 | 22:20 |
openstackgerrit | Merged openstack/nova stable/ocata: Add security release note for OSSA-2017-005 https://review.openstack.org/519753 | 22:20 |
mriedem | i think the devstack hack is working | 22:21 |
mriedem | tempest is running on that patch now | 22:21 |
mriedem | so the install was good | 22:21 |
*** threestrands has quit IRC | 22:21 | |
openstackgerrit | Eric Fried proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | 22:24 |
openstackgerrit | Eric Fried proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | 22:24 |
openstackgerrit | Eric Fried proposed openstack/nova master: Reformat _get_all_with_shared https://review.openstack.org/512799 | 22:24 |
tonyb | mriedem: \o/ I +2'd the devstack change | 22:24 |
efried | jaypipes Addressed the couple of comments remaining in here --^ | 22:25 |
*** dikonoor has quit IRC | 22:25 | |
mriedem | tonyb: you don't think that devstack change needs to go on master first and backported? | 22:25 |
mriedem | ah i see your comment | 22:25 |
tonyb | mriedem: Yeah that thing | 22:26 |
*** yangyapeng has joined #openstack-nova | 22:26 | |
*** dikonoor has joined #openstack-nova | 22:26 | |
*** gouthamr has joined #openstack-nova | 22:26 | |
* tonyb trusts mriedem will do the follow-up work (or just tell me to do it ;P) | 22:27 | |
mriedem | oh fo sho | 22:27 |
mriedem | begin preparations for ocata and pike releases | 22:28 |
* mriedem likes to talk like a captain | 22:28 | |
*** yangyapeng has quit IRC | 22:34 | |
tonyb | mriedem: You're just missing a "make it so" or "engage!" in there somewhere | 22:37 |
tonyb | mriedem: perhaps you could take that as a challenge? | 22:37 |
mriedem | i thought about it | 22:37 |
mriedem | sdague said engage earlier today | 22:37 |
mriedem | oh no he said the other | 22:38 |
mriedem | make it so | 22:38 |
*** Swami has quit IRC | 22:38 | |
mriedem | ok pike release https://review.openstack.org/#/c/519799/ and ocata release https://review.openstack.org/#/c/519808/ | 22:38 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/newton: Correct log message when removing a security group https://review.openstack.org/519586 | 22:40 |
mriedem | dansmith: tonyb: want to bless this https://review.openstack.org/#/c/519763/ ? | 22:41 |
*** gbarros has joined #openstack-nova | 22:43 | |
tonyb | mriedem: +2'd | 22:43 |
dansmith | tonyb: there's one above it too | 22:44 |
dansmith | (the important one) | 22:44 |
tonyb | dansmith: Yeah I'm looking that the zuul consoles for that series and the pike/ocata releases | 22:47 |
dansmith | okay | 22:48 |
*** priteau has quit IRC | 22:48 | |
*** priteau has joined #openstack-nova | 22:49 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/519806 | 22:49 |
*** eharney has joined #openstack-nova | 22:50 | |
openstackgerrit | Merged openstack/nova master: Add error message on metadata API https://review.openstack.org/517152 | 22:51 |
*** awaugama has quit IRC | 22:52 | |
mriedem | thanks - at this hour, i'll just fast re-approve anything that was already approved earlier todya | 22:52 |
mriedem | once ci is happy | 22:52 |
*** priteau has quit IRC | 22:53 | |
efried | jaypipes Taking on the latest merge conflict... | 22:54 |
mriedem | tonyb: i think the devstack patch isn't actually doing the job | 22:57 |
mriedem | http://logs.openstack.org/58/519758/3/check/legacy-tempest-dsvm-neutron-full/1676068/logs/devstacklog.txt.gz#_2017-11-14_21_45_19_885 | 22:57 |
mriedem | i'm not sure why the job passed though | 22:57 |
tonyb | ergh moar debugging | 22:58 |
mriedem | well, i assume my regex compare in bash is wrong | 22:58 |
mriedem | i can update to what efried suggested | 22:58 |
mriedem | although | 22:59 |
mriedem | http://logs.openstack.org/58/519758/3/check/legacy-tempest-dsvm-neutron-full/1676068/logs/pip2-freeze.txt.gz | 22:59 |
mriedem | neutron==9.4.2.dev21 | 22:59 |
*** lpetrut_ has quit IRC | 22:59 | |
mriedem | that is newton for neutron | 22:59 |
efried | mriedem I only glanced at that line, just noticed you were using a regex when a simple glob compare would do. | 22:59 |
*** gfhellma_ has quit IRC | 23:00 | |
tonyb | Yeah /me is confused | 23:00 |
mriedem | it clearly shows it not checking out the newton-eol tag though | 23:01 |
*** Apoorva_ has joined #openstack-nova | 23:01 | |
*** gfhellma_ has joined #openstack-nova | 23:02 | |
efried | mriedem It appears you need to whack the double quotes around the regex side. | 23:03 |
efried | mriedem Or change to the simpler form | 23:03 |
*** dikonoo has joined #openstack-nova | 23:03 | |
yankcrime | webdev | 23:03 |
*** edmondsw has quit IRC | 23:04 | |
*** Apoorva has quit IRC | 23:04 | |
*** edmondsw has joined #openstack-nova | 23:04 | |
efried | efried@efried-ThinkPad-W520:~$ [[ "stable/whatever" =~ stable/.* ]]; echo $? | 23:04 |
efried | 0 | 23:04 |
efried | efried@efried-ThinkPad-W520:~$ [[ "stable/whatever" =~ "stable/.*" ]]; echo $? | 23:04 |
efried | 1 | 23:04 |
*** Sukhdev has joined #openstack-nova | 23:04 | |
*** erlon has quit IRC | 23:04 | |
efried | then there's | 23:05 |
efried | efried@efried-ThinkPad-W520:~$ [[ "stable/whatever" == stable/* ]]; echo $? | 23:05 |
efried | 0 | 23:05 |
mriedem | yup, i copied something from d-g incorrectly | 23:05 |
efried | I believe they're trying to be helpful with =~ in bash and make it so you can include quotes more "easily" as match tokens in your actual regex. | 23:06 |
tonyb | mriedem: So http://logs.openstack.org/58/519758/3/check/legacy-tempest-dsvm-neutron-full/1676068/logs/devstacklog.txt.gz#_2017-11-14_21_45_19_892 | 23:06 |
tonyb | shows that for reasons I don't grok you got newton-eol | 23:06 |
*** edmondsw_ has joined #openstack-nova | 23:07 | |
mriedem | yeah i don't either | 23:07 |
mriedem | but i've updated the devstack patch | 23:07 |
*** dikonoor has quit IRC | 23:07 | |
mriedem | and added efried as co-author since he's fixed all my mistakes | 23:08 |
efried | :) | 23:08 |
*** edmondsw has quit IRC | 23:08 | |
*** imacdonn has quit IRC | 23:09 | |
*** imacdonn has joined #openstack-nova | 23:09 | |
openstackgerrit | Merged openstack/nova master: Add security release note for OSSA-2017-005 https://review.openstack.org/519738 | 23:10 |
mriedem | ok newton stuff is all rechecked | 23:13 |
*** edmondsw_ has quit IRC | 23:13 | |
mriedem | this is going to take an hour so i'm gonna be afk for a bit | 23:14 |
*** erlon has joined #openstack-nova | 23:14 | |
tonyb | okay | 23:15 |
*** gyee_ has quit IRC | 23:23 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608 | 23:31 |
*** takashin has joined #openstack-nova | 23:32 | |
*** jaypipes has quit IRC | 23:32 | |
*** felipemonteiro has quit IRC | 23:34 | |
*** gbarros has quit IRC | 23:35 | |
*** edmondsw has joined #openstack-nova | 23:36 | |
*** eharney has quit IRC | 23:36 | |
*** gyee_ has joined #openstack-nova | 23:36 | |
*** gbarros has joined #openstack-nova | 23:36 | |
*** edmondsw has quit IRC | 23:40 | |
*** slaweq has quit IRC | 23:41 | |
*** slaweq has joined #openstack-nova | 23:42 | |
*** dave-mccowan has joined #openstack-nova | 23:42 | |
*** slaweq has quit IRC | 23:46 | |
openstackgerrit | Eric Fried proposed openstack/nova master: begin refactor AllocCandidates._get_by_filters() https://review.openstack.org/516778 | 23:49 |
openstackgerrit | Eric Fried proposed openstack/nova master: build ProviderSummary objects in sep function https://review.openstack.org/516779 | 23:49 |
openstackgerrit | Eric Fried proposed openstack/nova master: build alloc request resources for shared resources https://review.openstack.org/516780 | 23:49 |
openstackgerrit | Eric Fried proposed openstack/nova master: create allocation request for single provider https://review.openstack.org/516781 | 23:49 |
openstackgerrit | Eric Fried proposed openstack/nova master: finish refactor AllocCandidates._get_by_filters() https://review.openstack.org/516782 | 23:49 |
openstackgerrit | Eric Fried proposed openstack/nova master: required traits for no sharing providers https://review.openstack.org/517027 | 23:49 |
openstackgerrit | Eric Fried proposed openstack/nova master: Add ProviderSummary.resource_class_names @property https://review.openstack.org/517646 | 23:49 |
openstackgerrit | Eric Fried proposed openstack/nova master: handle traits with sharing providers https://review.openstack.org/517119 | 23:49 |
openstackgerrit | Eric Fried proposed openstack/nova master: Test alloc_cands with non overlapping sharing RPs https://review.openstack.org/519380 | 23:49 |
openstackgerrit | Eric Fried proposed openstack/nova master: Test alloc_cands with one RP shared between two RPs https://review.openstack.org/519617 | 23:49 |
openstackgerrit | Eric Fried proposed openstack/nova master: Test allocation candidates: multiple aggregates https://review.openstack.org/518633 | 23:49 |
openstackgerrit | Eric Fried proposed openstack/nova master: Test helper: validate provider summaries https://review.openstack.org/518982 | 23:49 |
efried | For the record, I suspect a couple of the patches in the middle of this ^ will fail functional tests, because auto-merge. I'll deal with that tomorrow. (jaypipes ^ in case you're eavesdropping) | 23:50 |
*** lyan has quit IRC | 23:51 | |
*** gouthamr has quit IRC | 23:52 | |
*** claudiub|2 has quit IRC | 23:53 | |
*** moshele has quit IRC | 23:56 | |
*** Ankit has joined #openstack-nova | 23:58 | |
*** yangyapeng has joined #openstack-nova | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!