| *** hongbin has quit IRC | 00:05 | |
| *** TxGirlGeek has quit IRC | 00:05 | |
| *** shaohe_feng has quit IRC | 00:08 | |
| *** shaohe_feng has joined #openstack-nova | 00:08 | |
| *** med_ has quit IRC | 00:15 | |
| *** shaohe_feng has quit IRC | 00:18 | |
| *** shaohe_feng has joined #openstack-nova | 00:19 | |
| *** takashin has joined #openstack-nova | 00:19 | |
| *** vladikr has quit IRC | 00:19 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform metrics.update notification https://review.openstack.org/580567 | 00:20 |
|---|---|---|
| *** shaohe_feng has quit IRC | 00:28 | |
| *** antosh has quit IRC | 00:29 | |
| *** shaohe_feng has joined #openstack-nova | 00:29 | |
| *** Dinesh_Bhor has joined #openstack-nova | 00:30 | |
| *** vladikr has joined #openstack-nova | 00:38 | |
| *** shaohe_feng has quit IRC | 00:39 | |
| *** shaohe_feng has joined #openstack-nova | 00:39 | |
| *** licanwei has joined #openstack-nova | 00:39 | |
| *** threestrands has joined #openstack-nova | 00:40 | |
| *** brault has joined #openstack-nova | 00:48 | |
| *** shaohe_feng has quit IRC | 00:49 | |
| openstackgerrit | Brin Zhang proposed openstack/nova stable/queens: Add unshelve instance error info to fault table https://review.openstack.org/583418 | 00:50 |
| *** shaohe_feng has joined #openstack-nova | 00:51 | |
| *** hiro-kobayashi has joined #openstack-nova | 00:52 | |
| *** brault has quit IRC | 00:52 | |
| *** dave-mccowan has joined #openstack-nova | 00:57 | |
| *** shaohe_feng has quit IRC | 00:59 | |
| *** itlinux has quit IRC | 00:59 | |
| *** shaohe_feng has joined #openstack-nova | 01:01 | |
| *** chyka has joined #openstack-nova | 01:05 | |
| *** shaohe_feng has quit IRC | 01:09 | |
| *** dave-mccowan has quit IRC | 01:09 | |
| *** chyka has quit IRC | 01:09 | |
| *** shaohe_feng has joined #openstack-nova | 01:10 | |
| *** slaweq has joined #openstack-nova | 01:11 | |
| *** alexpilotti has quit IRC | 01:13 | |
| *** harlowja has quit IRC | 01:13 | |
| *** erlon has quit IRC | 01:15 | |
| *** slaweq has quit IRC | 01:15 | |
| *** med_ has joined #openstack-nova | 01:16 | |
| *** erlon has joined #openstack-nova | 01:16 | |
| openstackgerrit | Brin Zhang proposed openstack/nova stable/ocata: Add unshelve instance error info to fault table https://review.openstack.org/583422 | 01:16 |
| *** oomichi has quit IRC | 01:16 | |
| *** dave-mccowan has joined #openstack-nova | 01:17 | |
| *** shaohe_feng has quit IRC | 01:20 | |
| *** shaohe_feng has joined #openstack-nova | 01:20 | |
| *** mrsoul has joined #openstack-nova | 01:22 | |
| *** dave-mcc_ has joined #openstack-nova | 01:22 | |
| *** mrsoul_ has joined #openstack-nova | 01:22 | |
| *** alexpilotti has joined #openstack-nova | 01:24 | |
| *** dave-mccowan has quit IRC | 01:24 | |
| *** mrsoul` has quit IRC | 01:24 | |
| *** phuongnh has joined #openstack-nova | 01:24 | |
| *** mriedem has quit IRC | 01:24 | |
| *** mschuppert has quit IRC | 01:25 | |
| *** alexpilotti has quit IRC | 01:29 | |
| *** shaohe_feng has quit IRC | 01:30 | |
| *** alexpilotti has joined #openstack-nova | 01:30 | |
| *** tiendc has joined #openstack-nova | 01:32 | |
| *** alexpilotti has quit IRC | 01:35 | |
| *** shaohe_feng has joined #openstack-nova | 01:35 | |
| openstackgerrit | Brin Zhang proposed openstack/nova stable/pike: Add unshelve instance error info to fault table https://review.openstack.org/583426 | 01:35 |
| *** alexpilotti has joined #openstack-nova | 01:36 | |
| *** BrinZhang has joined #openstack-nova | 01:39 | |
| *** shaohe_feng has quit IRC | 01:40 | |
| *** shaohe_feng has joined #openstack-nova | 01:41 | |
| *** med_ has quit IRC | 01:43 | |
| *** annp has quit IRC | 01:43 | |
| *** alexpilotti has quit IRC | 01:44 | |
| *** Guest1853 has quit IRC | 01:45 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Wrong href link returned when providing non-existed version in GET version API https://review.openstack.org/486850 | 01:47 |
| *** shaohe_feng has quit IRC | 01:50 | |
| *** shaohe_feng has joined #openstack-nova | 01:52 | |
| *** lei-zh has joined #openstack-nova | 01:53 | |
| *** tianhui_ has quit IRC | 01:57 | |
| *** hongbin has joined #openstack-nova | 01:57 | |
| *** shaohe_feng has quit IRC | 02:01 | |
| *** shaohe_feng has joined #openstack-nova | 02:01 | |
| *** alexpilotti has joined #openstack-nova | 02:03 | |
| *** alexpilotti has quit IRC | 02:07 | |
| *** shaohe_feng has quit IRC | 02:11 | |
| *** slaweq has joined #openstack-nova | 02:11 | |
| *** shaohe_feng has joined #openstack-nova | 02:12 | |
| *** slaweq has quit IRC | 02:16 | |
| *** logan- has quit IRC | 02:20 | |
| *** logan- has joined #openstack-nova | 02:21 | |
| *** shaohe_feng has quit IRC | 02:21 | |
| *** huanhongda has joined #openstack-nova | 02:21 | |
| *** gcb has joined #openstack-nova | 02:22 | |
| *** shaohe_feng has joined #openstack-nova | 02:22 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Change deprecated policies to policy https://review.openstack.org/583434 | 02:26 |
| *** jichen has joined #openstack-nova | 02:31 | |
| *** alex_xu has quit IRC | 02:31 | |
| *** shaohe_feng has quit IRC | 02:31 | |
| *** shaohe_feng has joined #openstack-nova | 02:33 | |
| *** alex_xu has joined #openstack-nova | 02:33 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Change deprecated policies to policy https://review.openstack.org/583434 | 02:33 |
| *** yamahata has quit IRC | 02:33 | |
| *** alexpilotti has joined #openstack-nova | 02:39 | |
| *** dpawlik has joined #openstack-nova | 02:41 | |
| *** shaohe_feng has quit IRC | 02:42 | |
| *** shaohe_feng has joined #openstack-nova | 02:42 | |
| *** tidwellr has joined #openstack-nova | 02:43 | |
| *** yedongcan has joined #openstack-nova | 02:45 | |
| *** dpawlik has quit IRC | 02:45 | |
| *** alexpilotti has quit IRC | 02:51 | |
| *** shaohe_feng has quit IRC | 02:52 | |
| *** tidwellr has quit IRC | 02:52 | |
| *** shaohe_feng has joined #openstack-nova | 02:53 | |
| *** lei-zh1 has joined #openstack-nova | 02:56 | |
| *** lei-zh has quit IRC | 02:56 | |
| *** shaohe_feng has quit IRC | 03:02 | |
| *** shaohe_feng has joined #openstack-nova | 03:03 | |
| *** annp has joined #openstack-nova | 03:11 | |
| *** shaohe_feng has quit IRC | 03:12 | |
| *** alexpilotti has joined #openstack-nova | 03:13 | |
| *** shaohe_feng has joined #openstack-nova | 03:13 | |
| *** alexpilotti has quit IRC | 03:18 | |
| *** trozet has quit IRC | 03:19 | |
| *** alexpilotti has joined #openstack-nova | 03:19 | |
| *** trozet has joined #openstack-nova | 03:19 | |
| *** shaohe_feng has quit IRC | 03:23 | |
| *** mdnadeem has joined #openstack-nova | 03:23 | |
| *** alexpilotti has quit IRC | 03:24 | |
| *** udesale has joined #openstack-nova | 03:24 | |
| *** shaohe_feng has joined #openstack-nova | 03:24 | |
| *** mdnadeem has quit IRC | 03:25 | |
| *** alexpilotti has joined #openstack-nova | 03:26 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Wrong href link returned when providing non-existed version in GET version API https://review.openstack.org/486850 | 03:26 |
| openstackgerrit | Lei Zhang proposed openstack/nova master: Add method to get cpu traits https://review.openstack.org/560317 | 03:29 |
| openstackgerrit | Lei Zhang proposed openstack/nova master: Docs for reporting CPU traits feature https://review.openstack.org/582902 | 03:29 |
| *** alexpilotti has quit IRC | 03:31 | |
| *** shaohe_feng has quit IRC | 03:33 | |
| *** alexpilotti has joined #openstack-nova | 03:33 | |
| *** shaohe_feng has joined #openstack-nova | 03:33 | |
| openstackgerrit | Merged openstack/nova master: Use ThreadPoolExecutor for max_concurrent_live_migrations https://review.openstack.org/563505 | 03:34 |
| openstackgerrit | Merged openstack/nova master: Compute: add support to abort queued live migration https://review.openstack.org/568542 | 03:34 |
| openstackgerrit | Merged openstack/nova master: Fix ServerMigrationSampleJsonTestsV2_24 to use its own sample file https://review.openstack.org/581562 | 03:34 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Change deprecated policies to policy https://review.openstack.org/583434 | 03:37 |
| *** alexpilotti has quit IRC | 03:38 | |
| *** alexpilotti has joined #openstack-nova | 03:39 | |
| openstackgerrit | Merged openstack/nova master: objects: Add NUMATopologyLimits.network_metadata https://review.openstack.org/575486 | 03:43 |
| *** shaohe_feng has quit IRC | 03:43 | |
| openstackgerrit | Merged openstack/nova master: Add "bind_ports_to_host" neutron API method https://review.openstack.org/523604 | 03:43 |
| openstackgerrit | Merged openstack/nova master: Add "delete_port_binding" network API method https://review.openstack.org/552170 | 03:43 |
| openstackgerrit | Merged openstack/nova master: Add "activate_port_binding" neutron API method https://review.openstack.org/555947 | 03:43 |
| openstackgerrit | Merged openstack/nova master: Delete port bindings in setup_networks_on_host if teardown=True https://review.openstack.org/556333 | 03:44 |
| *** alexpilotti has quit IRC | 03:44 | |
| yikun | mriedem , yeah, thanks for reminder, I remove left polices in https://review.openstack.org/#/c/583434/ | 03:45 |
| *** shaohe_feng has joined #openstack-nova | 03:45 | |
| *** alexpilotti has joined #openstack-nova | 03:46 | |
| *** hongbin has quit IRC | 03:49 | |
| *** lpetrut has joined #openstack-nova | 03:51 | |
| *** cfriesen_ has quit IRC | 03:51 | |
| *** alexpilotti has quit IRC | 03:51 | |
| *** alexpilotti has joined #openstack-nova | 03:53 | |
| *** shaohe_feng has quit IRC | 03:53 | |
| *** shaohe_feng has joined #openstack-nova | 03:54 | |
| *** alexpilotti has quit IRC | 03:58 | |
| *** alexpilotti has joined #openstack-nova | 03:59 | |
| *** pchavva has quit IRC | 04:00 | |
| *** shaohe_feng has quit IRC | 04:04 | |
| *** alexpilotti has quit IRC | 04:04 | |
| *** shaohe_feng has joined #openstack-nova | 04:04 | |
| *** Bhujay has joined #openstack-nova | 04:06 | |
| *** alexpilotti has joined #openstack-nova | 04:06 | |
| *** lei-zh1 has quit IRC | 04:06 | |
| *** alexpilotti has quit IRC | 04:10 | |
| *** slaweq has joined #openstack-nova | 04:11 | |
| *** shaohe_feng has quit IRC | 04:14 | |
| *** shaohe_feng has joined #openstack-nova | 04:14 | |
| *** slaweq has quit IRC | 04:16 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform libvirt.error notification https://review.openstack.org/484851 | 04:17 |
| *** alexpilotti has joined #openstack-nova | 04:19 | |
| *** alexpilotti has quit IRC | 04:24 | |
| *** shaohe_feng has quit IRC | 04:24 | |
| *** shaohe_feng has joined #openstack-nova | 04:25 | |
| *** alexpilotti has joined #openstack-nova | 04:26 | |
| *** alexpilotti has quit IRC | 04:30 | |
| *** chyka has joined #openstack-nova | 04:32 | |
| *** alexpilotti has joined #openstack-nova | 04:32 | |
| openstackgerrit | fupingxie proposed openstack/nova master: Delete allocations when it is re-allocated https://review.openstack.org/582899 | 04:32 |
| *** shaohe_feng has quit IRC | 04:34 | |
| *** shaohe_feng has joined #openstack-nova | 04:35 | |
| *** mdnadeem has joined #openstack-nova | 04:36 | |
| *** alexpilotti has quit IRC | 04:36 | |
| *** chyka has quit IRC | 04:37 | |
| *** alexpilotti has joined #openstack-nova | 04:39 | |
| *** janki has joined #openstack-nova | 04:39 | |
| *** lpetrut has quit IRC | 04:39 | |
| *** lpetrut has joined #openstack-nova | 04:42 | |
| *** alexpilotti has quit IRC | 04:44 | |
| *** shaohe_feng has quit IRC | 04:45 | |
| *** alexpilotti has joined #openstack-nova | 04:45 | |
| *** shaohe_feng has joined #openstack-nova | 04:45 | |
| openstackgerrit | Merged openstack/nova master: Use hard coded values in schema than reference https://review.openstack.org/581288 | 04:47 |
| *** alexpilotti has quit IRC | 04:49 | |
| *** alexpilotti has joined #openstack-nova | 04:51 | |
| *** lpetrut has quit IRC | 04:54 | |
| *** shaohe_feng has quit IRC | 04:55 | |
| *** udesale has quit IRC | 04:55 | |
| *** udesale has joined #openstack-nova | 04:55 | |
| *** alexpilotti has quit IRC | 04:56 | |
| *** trungnv has quit IRC | 04:57 | |
| *** tiendc has quit IRC | 04:57 | |
| *** trungnv has joined #openstack-nova | 04:57 | |
| *** tiendc has joined #openstack-nova | 04:57 | |
| *** shaohe_feng has joined #openstack-nova | 04:58 | |
| *** alexpilotti has joined #openstack-nova | 04:58 | |
| *** abhishekk has joined #openstack-nova | 04:59 | |
| *** sridharg has joined #openstack-nova | 04:59 | |
| *** alexpilotti has quit IRC | 05:03 | |
| *** alexpilotti has joined #openstack-nova | 05:04 | |
| *** shaohe_feng has quit IRC | 05:05 | |
| *** shaohe_feng has joined #openstack-nova | 05:06 | |
| *** alexpilotti has quit IRC | 05:08 | |
| *** vabada_ has joined #openstack-nova | 05:10 | |
| *** MrWatson has joined #openstack-nova | 05:10 | |
| *** openstackgerrit has quit IRC | 05:10 | |
| *** alexpilotti has joined #openstack-nova | 05:11 | |
| *** slaweq has joined #openstack-nova | 05:11 | |
| *** openstackgerrit has joined #openstack-nova | 05:12 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge server create for multiple-create extension https://review.openstack.org/580017 | 05:12 |
| *** lucasagomes_ has joined #openstack-nova | 05:12 | |
| *** chason_ has joined #openstack-nova | 05:14 | |
| *** shaohe_feng has quit IRC | 05:15 | |
| *** links has joined #openstack-nova | 05:15 | |
| *** alexpilotti has quit IRC | 05:16 | |
| *** Bhujay has quit IRC | 05:16 | |
| *** efried has quit IRC | 05:16 | |
| *** jmlowe has quit IRC | 05:16 | |
| *** vabada has quit IRC | 05:16 | |
| *** bauzas has quit IRC | 05:16 | |
| *** icey has quit IRC | 05:16 | |
| *** hamzy has quit IRC | 05:16 | |
| *** NostawRm has quit IRC | 05:16 | |
| *** serverascode has quit IRC | 05:16 | |
| *** mwhahaha has quit IRC | 05:16 | |
| *** auggy has quit IRC | 05:16 | |
| *** johnsom has quit IRC | 05:16 | |
| *** ameade has quit IRC | 05:16 | |
| *** simondodsley has quit IRC | 05:16 | |
| *** coreycb has quit IRC | 05:16 | |
| *** pas-ha has quit IRC | 05:16 | |
| *** jamespage has quit IRC | 05:16 | |
| *** NobodyCam has quit IRC | 05:16 | |
| *** DuncanT has quit IRC | 05:16 | |
| *** lucasagomes has quit IRC | 05:16 | |
| *** Roamer` has quit IRC | 05:16 | |
| *** chason has quit IRC | 05:16 | |
| *** tristanC has quit IRC | 05:16 | |
| *** slaweq has quit IRC | 05:16 | |
| *** shaohe_feng has joined #openstack-nova | 05:17 | |
| *** icey has joined #openstack-nova | 05:17 | |
| *** alexpilotti has joined #openstack-nova | 05:17 | |
| *** takashin has quit IRC | 05:18 | |
| *** sambetts_ has quit IRC | 05:18 | |
| *** toabctl has quit IRC | 05:18 | |
| *** takashin has joined #openstack-nova | 05:20 | |
| *** toabctl has joined #openstack-nova | 05:21 | |
| *** alexpilotti has quit IRC | 05:22 | |
| *** sambetts_ has joined #openstack-nova | 05:23 | |
| *** alexpilotti has joined #openstack-nova | 05:25 | |
| *** shaohe_feng has quit IRC | 05:26 | |
| *** shaohe_feng has joined #openstack-nova | 05:26 | |
| *** krtaylor has quit IRC | 05:27 | |
| *** alexpilotti has quit IRC | 05:29 | |
| *** alexpilotti has joined #openstack-nova | 05:32 | |
| *** janki has quit IRC | 05:34 | |
| *** Bhujay has joined #openstack-nova | 05:35 | |
| *** efried has joined #openstack-nova | 05:35 | |
| *** jmlowe has joined #openstack-nova | 05:35 | |
| *** bauzas has joined #openstack-nova | 05:35 | |
| *** hamzy has joined #openstack-nova | 05:35 | |
| *** serverascode has joined #openstack-nova | 05:35 | |
| *** mwhahaha has joined #openstack-nova | 05:35 | |
| *** auggy has joined #openstack-nova | 05:35 | |
| *** johnsom has joined #openstack-nova | 05:35 | |
| *** ameade has joined #openstack-nova | 05:35 | |
| *** simondodsley has joined #openstack-nova | 05:35 | |
| *** coreycb has joined #openstack-nova | 05:35 | |
| *** pas-ha has joined #openstack-nova | 05:35 | |
| *** jamespage has joined #openstack-nova | 05:35 | |
| *** NobodyCam has joined #openstack-nova | 05:35 | |
| *** DuncanT has joined #openstack-nova | 05:35 | |
| *** Roamer` has joined #openstack-nova | 05:35 | |
| *** ratailor has joined #openstack-nova | 05:35 | |
| *** shaohe_feng has quit IRC | 05:36 | |
| *** alexpilotti has quit IRC | 05:36 | |
| *** shaohe_feng has joined #openstack-nova | 05:37 | |
| *** jangutter has quit IRC | 05:38 | |
| *** alexpilotti has joined #openstack-nova | 05:39 | |
| *** jangutter has joined #openstack-nova | 05:39 | |
| *** s1061123 has quit IRC | 05:40 | |
| *** s1061123 has joined #openstack-nova | 05:41 | |
| *** alexpilotti has quit IRC | 05:43 | |
| *** Luzi has joined #openstack-nova | 05:45 | |
| *** alexpilotti has joined #openstack-nova | 05:46 | |
| *** shaohe_feng has quit IRC | 05:46 | |
| *** shaohe_feng has joined #openstack-nova | 05:47 | |
| *** alexpilotti has quit IRC | 05:50 | |
| *** krtaylor has joined #openstack-nova | 05:52 | |
| *** alexpilotti has joined #openstack-nova | 05:53 | |
| *** mrsoul_ is now known as mschuppert | 05:54 | |
| *** damien_r has joined #openstack-nova | 05:55 | |
| *** shaohe_feng has quit IRC | 05:56 | |
| *** alexpilotti has quit IRC | 05:58 | |
| *** shaohe_feng has joined #openstack-nova | 05:58 | |
| gmann | nova api office hour time | 05:59 |
| gmann | #startmeeting nova api | 06:00 |
| openstack | Meeting started Wed Jul 18 06:00:12 2018 UTC and is due to finish in 60 minutes. The chair is gmann. Information about MeetBot at http://wiki.debian.org/MeetBot. | 06:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 06:00 |
| *** openstack changes topic to " (Meeting topic: nova api)" | 06:00 | |
| openstack | The meeting name has been set to 'nova_api' | 06:00 |
| gmann | PING List: gmann, alex_xu | 06:00 |
| gmann | who all here today ? | 06:00 |
| *** lei-zh1 has joined #openstack-nova | 06:00 | |
| alex_xu | o/ | 06:00 |
| *** alexpilotti has joined #openstack-nova | 06:01 | |
| huanhongda | I am here | 06:01 |
| gmann | alex_xu: hogepodge hi | 06:02 |
| alex_xu | gmann: hi | 06:02 |
| gmann | let's start | 06:02 |
| openstackgerrit | fupingxie proposed openstack/nova master: Delete allocations when it is re-allocated https://review.openstack.org/582899 | 06:02 |
| gmann | #topic Priorities | 06:03 |
| *** openstack changes topic to "Priorities (Meeting topic: nova api)" | 06:03 | |
| *** bkopilov has joined #openstack-nova | 06:03 | |
| gmann | #link https://etherpad.openstack.org/p/rocky-nova-priorities-tracking | 06:03 |
| gmann | L57 in above etherpad | 06:03 |
| gmann | 1. Servers Ips non-unique network names | 06:03 |
| gmann | i sent mail to Maciej Kucia last week but no response and also no response on patch. | 06:04 |
| gmann | anyone knows Maciej Kucia ? | 06:04 |
| gmann | otherwise i can plan to push the code update | 06:05 |
| gmann | FF is close and we should have code up for review by this week max | 06:05 |
| *** alexpilotti has quit IRC | 06:06 | |
| alex_xu | only one week left | 06:06 |
| gmann | yeah | 06:06 |
| gmann | ok i will update the code for that | 06:07 |
| *** shaohe_feng has quit IRC | 06:07 | |
| gmann | #action gmann to update code for 1. Servers Ips non-unique network names | 06:07 |
| gmann | 2.Abort live migration in queued state: | 06:07 |
| gmann | this is almost done | 06:07 |
| gmann | #link https://review.openstack.org/#/q/topic:bp/abort-live-migration-in-queued-status+(status:open+OR+status:merged) | 06:07 |
| alex_xu | gmann: the main part already merge? | 06:08 |
| alex_xu | gmann: for the non-unique network names | 06:08 |
| gmann | alex_xu: no, only spec is merged | 06:08 |
| *** shaohe_feng has joined #openstack-nova | 06:08 | |
| *** alexpilotti has joined #openstack-nova | 06:09 | |
| alex_xu | gmann: ok, not sure we have enough time get the patch ready and merge | 06:09 |
| gmann | alex_xu: i will check how much code is needed | 06:09 |
| alex_xu | gmann: cool, thanks | 06:09 |
| gmann | on abort live migration | 06:10 |
| gmann | Kevin_Zheng: i think nova client changes are left ? | 06:11 |
| gmann | after that we can mark this complete | 06:11 |
| *** slaweq has joined #openstack-nova | 06:11 | |
| Kevin_Zheng | gmann yeah, I will work on that today | 06:11 |
| gmann | alex_xu: trivial one -https://review.openstack.org/#/c/583212/ | 06:11 |
| gmann | Kevin_Zheng: thanks | 06:11 |
| alex_xu | gmann: will check | 06:12 |
| gmann | alex_xu: thanks | 06:12 |
| gmann | 3. Complex anti-affinity policies: | 06:12 |
| alex_xu | done...super trivial | 06:12 |
| gmann | alex_xu: :) | 06:12 |
| gmann | anti-affinity one are merged | 06:13 |
| gmann | 1 patch left on nova #link https://review.openstack.org/#/c/583434/ | 06:13 |
| *** alexpilotti has quit IRC | 06:13 | |
| gmann | and nova client change | 06:13 |
| gmann | #link https://review.openstack.org/#/c/578261/ | 06:13 |
| gmann | same for this, after those 2 patches it can be marked as complete | 06:13 |
| gmann | 4. Volume multiattach enhancements: | 06:14 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Wrong href link returned when providing non-existed version in GET version API https://review.openstack.org/486850 | 06:15 |
| gmann | this one is no progress and matt mentioned to work on this in coming weeks may be | 06:15 |
| gmann | 5. API Extensions merge work | 06:15 |
| gmann | #link https://review.openstack.org/#/q/project:openstack/nova+branch:master+topic:bp/api-extensions-merge-rocky | 06:15 |
| gmann | update few patches on this. | 06:15 |
| *** alexpilotti has joined #openstack-nova | 06:16 | |
| gmann | will push all the remaining code by friday | 06:16 |
| *** slaweq has quit IRC | 06:16 | |
| alex_xu | gmann: how many we left? | 06:16 |
| *** shaohe_feng has quit IRC | 06:17 | |
| gmann | alex_xu: 3 on server_create and then response one(~9) | 06:17 |
| *** shaohe_feng has joined #openstack-nova | 06:18 | |
| alex_xu | gmann: cool | 06:18 |
| alex_xu | gmann: so we left the policy deprecation to the next release? | 06:18 |
| alex_xu | sorry, the policy remove | 06:18 |
| gmann | alex_xu: yeah, i thought removing in this cycle is little early. what you say? | 06:19 |
| alex_xu | gmann: sounds ok for me | 06:19 |
| alex_xu | just ensure the plan | 06:19 |
| gmann | alex_xu: ok. in Stein early we can remove them | 06:20 |
| alex_xu | gmann: yeah, cool | 06:20 |
| *** alexpilotti has quit IRC | 06:20 | |
| gmann | 6. Handling a down cell | 06:20 |
| gmann | #link https://review.openstack.org/#/q/topic:bp/handling-down-cell+(status:open+OR+status:merged) | 06:21 |
| gmann | code is up for review and matt already +2 on few patches | 06:21 |
| gmann | i have not looked into that yet. | 06:21 |
| *** lpetrut has joined #openstack-nova | 06:22 | |
| gmann | i will target this fro today | 06:22 |
| gmann | from | 06:22 |
| *** alexpilotti has joined #openstack-nova | 06:23 | |
| gmann | that's all on priority things from me | 06:23 |
| gmann | anything else to discuss or any priority review | 06:23 |
| alex_xu | nothing from me | 06:24 |
| gmann | ok let's move next | 06:25 |
| gmann | #topic Bug Triage/Discussion | 06:25 |
| *** openstack changes topic to "Bug Triage/Discussion (Meeting topic: nova api)" | 06:25 | |
| gmann | #link https://etherpad.openstack.org/p/nova-api-weekly-bug-report | 06:25 |
| gmann | weekly report | 06:25 |
| gmann | total bugs 69 | 06:25 |
| gmann | i reviewed i think 4-5 patches of in-progress bugs | 06:26 |
| gmann | #link https://review.openstack.org/#/c/582674/1 | 06:26 |
| gmann | #link https://review.openstack.org/#/c/566143/ | 06:26 |
| gmann | ^^ these are in good shape | 06:26 |
| gmann | and easy one | 06:27 |
| *** shaohe_feng has quit IRC | 06:27 | |
| gmann | no new or untriaged bug. | 06:28 |
| *** alexpilotti has quit IRC | 06:28 | |
| gmann | we will keep reviewing the in-progress patches and get the number down | 06:29 |
| gmann | any bug anyone want to discuss ? otherwise we can move next may be | 06:29 |
| *** tristanC has joined #openstack-nova | 06:30 | |
| *** alexpilotti has joined #openstack-nova | 06:30 | |
| *** damien_r has quit IRC | 06:30 | |
| alex_xu | I saw Kevin_Zheng updated this one https://review.openstack.org/486850 | 06:30 |
| gmann | k | 06:31 |
| Kevin_Zheng | yeah, trying to clean up my bugs :) | 06:31 |
| gmann | so it is stripping the after version part now? | 06:32 |
| *** shaohe_feng has joined #openstack-nova | 06:32 | |
| gmann | anyway let's review on patch | 06:34 |
| *** alexpilotti has quit IRC | 06:34 | |
| alex_xu | yea | 06:35 |
| *** pcaruana has joined #openstack-nova | 06:35 | |
| gmann | let's move next | 06:36 |
| gmann | #topic Open Discussion | 06:36 |
| *** openstack changes topic to "Open Discussion (Meeting topic: nova api)" | 06:36 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Wrong href link returned when providing non-existed version in GET version API https://review.openstack.org/486850 | 06:36 |
| gmann | i do not have anything to discuss. | 06:36 |
| *** alexchadin has joined #openstack-nova | 06:36 | |
| alex_xu | me too | 06:36 |
| gmann | ok | 06:36 |
| gmann | let's close early today and back to work. | 06:37 |
| gmann | thanks for joining. | 06:37 |
| gmann | alex_xu: and happy holiday :) saw your tweet of lucky draw | 06:37 |
| gmann | #endmeeting | 06:37 |
| *** openstack changes topic to "Current runways: abort-live-migration-in-queued-status / add-zvm-driver-rocky / glance-image-traits -- This channel is for Nova development. For support of Nova deployments, please use #openstack." | 06:37 | |
| openstack | Meeting ended Wed Jul 18 06:37:43 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 06:37 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/nova_api/2018/nova_api.2018-07-18-06.00.html | 06:37 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/nova_api/2018/nova_api.2018-07-18-06.00.txt | 06:37 |
| *** shaohe_feng has quit IRC | 06:37 | |
| openstack | Log: http://eavesdrop.openstack.org/meetings/nova_api/2018/nova_api.2018-07-18-06.00.log.html | 06:37 |
| *** shaohe_feng has joined #openstack-nova | 06:39 | |
| *** lei-zh1 has quit IRC | 06:39 | |
| *** cfriesen_ has joined #openstack-nova | 06:40 | |
| *** lei-zh1 has joined #openstack-nova | 06:40 | |
| *** alexpilotti has joined #openstack-nova | 06:44 | |
| *** cfriesen_ has quit IRC | 06:45 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge server create for keypair extension https://review.openstack.org/583203 | 06:47 |
| *** shaohe_feng has quit IRC | 06:48 | |
| openstackgerrit | Chen Hanxiao proposed openstack/nova master: libvirt: guest: introduce blockStats instead of domain.blockStats https://review.openstack.org/526833 | 06:48 |
| *** alexpilotti has quit IRC | 06:49 | |
| *** dpawlik has joined #openstack-nova | 06:49 | |
| *** damien_r has joined #openstack-nova | 06:49 | |
| *** shaohe_feng has joined #openstack-nova | 06:49 | |
| alex_xu | gmann: thanks :) | 06:51 |
| *** alexpilotti has joined #openstack-nova | 06:51 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix the incorrect description and sample https://review.openstack.org/528180 | 06:54 |
| *** slaweq has joined #openstack-nova | 06:54 | |
| *** alexpilotti has quit IRC | 06:56 | |
| *** shaohe_feng has quit IRC | 06:58 | |
| *** alexpilotti has joined #openstack-nova | 06:58 | |
| *** shaohe_feng has joined #openstack-nova | 06:59 | |
| *** ispp has joined #openstack-nova | 07:00 | |
| *** alexpilotti has quit IRC | 07:03 | |
| *** alexpilotti has joined #openstack-nova | 07:05 | |
| *** moshele has joined #openstack-nova | 07:08 | |
| *** shaohe_feng has quit IRC | 07:08 | |
| *** ccamacho has joined #openstack-nova | 07:08 | |
| *** peereb has joined #openstack-nova | 07:09 | |
| *** alexpilotti has quit IRC | 07:10 | |
| *** shaohe_feng has joined #openstack-nova | 07:11 | |
| *** alexpilotti has joined #openstack-nova | 07:13 | |
| *** brault has joined #openstack-nova | 07:17 | |
| *** alexpilotti has quit IRC | 07:18 | |
| *** tesseract has joined #openstack-nova | 07:18 | |
| *** shaohe_feng has quit IRC | 07:18 | |
| *** brault has quit IRC | 07:19 | |
| *** brault has joined #openstack-nova | 07:19 | |
| *** shaohe_feng has joined #openstack-nova | 07:19 | |
| *** alexpilotti has joined #openstack-nova | 07:25 | |
| *** rcernin has quit IRC | 07:28 | |
| *** shaohe_feng has quit IRC | 07:29 | |
| *** alexpilotti has quit IRC | 07:29 | |
| *** ragiman has joined #openstack-nova | 07:30 | |
| *** shaohe_feng has joined #openstack-nova | 07:30 | |
| *** alexpilotti has joined #openstack-nova | 07:31 | |
| *** vishakha has quit IRC | 07:34 | |
| *** dklyle has quit IRC | 07:35 | |
| *** alexpilotti has quit IRC | 07:35 | |
| *** alexpilotti has joined #openstack-nova | 07:36 | |
| *** janki has joined #openstack-nova | 07:37 | |
| *** sahid has joined #openstack-nova | 07:38 | |
| *** tssurya has joined #openstack-nova | 07:38 | |
| *** shaohe_feng has quit IRC | 07:39 | |
| *** alexpilotti has quit IRC | 07:41 | |
| *** shaohe_feng has joined #openstack-nova | 07:41 | |
| *** alexpilotti has joined #openstack-nova | 07:42 | |
| *** AlexeyAbashkin has joined #openstack-nova | 07:45 | |
| *** alexpilotti has quit IRC | 07:46 | |
| *** vishakha has joined #openstack-nova | 07:47 | |
| *** shaohe_feng has quit IRC | 07:49 | |
| *** shaohe_feng has joined #openstack-nova | 07:51 | |
| *** Bhujay has quit IRC | 07:52 | |
| *** ttsiouts has joined #openstack-nova | 07:53 | |
| *** alexpilotti has joined #openstack-nova | 07:53 | |
| *** tobasco_away is now known as tobasco | 07:54 | |
| slaweq | sean-k-mooney: hi | 07:56 |
| *** ttsiouts has quit IRC | 07:57 | |
| slaweq | sean-k-mooney: recently in some neutron jobs I saw failing test tempest.api.compute.admin.test_servers_on_multinodes.ServersOnMultiNodesTest.test_create_server_with_scheduler_hint_group_anti_affinity - it doesn't looks like related to neutron for me | 07:58 |
| slaweq | sean-k-mooney: maybe You are already aware of such issue? Can You take a look on it maybe? | 07:58 |
| slaweq | example of failure is e.g.: http://logs.openstack.org/08/555608/34/check/neutron-tempest-multinode-full/1d3aafb/logs/testr_results.html.gz | 07:58 |
| *** shaohe_feng has quit IRC | 07:59 | |
| *** shaohe_feng has joined #openstack-nova | 08:00 | |
| *** takashin has left #openstack-nova | 08:01 | |
| *** huanhongda has quit IRC | 08:03 | |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova master: Fix confusing log message in scheduler https://review.openstack.org/583486 | 08:03 |
| *** alexpilotti has quit IRC | 08:04 | |
| *** liuyulong_ has joined #openstack-nova | 08:04 | |
| *** alexpilotti has joined #openstack-nova | 08:05 | |
| *** markvoelker_ has quit IRC | 08:07 | |
| *** liuyulong has quit IRC | 08:08 | |
| *** alexpilotti has quit IRC | 08:09 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Remove Ocata comments which expires now https://review.openstack.org/583489 | 08:09 |
| *** shaohe_feng has quit IRC | 08:10 | |
| *** shaohe_feng has joined #openstack-nova | 08:10 | |
| *** alexpilotti has joined #openstack-nova | 08:11 | |
| *** GonZo2000 has joined #openstack-nova | 08:12 | |
| *** GonZo2000 has quit IRC | 08:12 | |
| *** GonZo2000 has joined #openstack-nova | 08:12 | |
| *** elod has quit IRC | 08:13 | |
| *** ttsiouts has joined #openstack-nova | 08:13 | |
| *** lucasagomes_ is now known as lucasagomes | 08:15 | |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova master: Fix confusing log message in scheduler https://review.openstack.org/583486 | 08:16 |
| *** alexpilotti has quit IRC | 08:16 | |
| *** shaohe_feng has quit IRC | 08:20 | |
| *** moshele has quit IRC | 08:20 | |
| *** do3meli has joined #openstack-nova | 08:22 | |
| openstackgerrit | jichenjc proposed openstack/nova master: clarify usage of upgrade_levels group https://review.openstack.org/583502 | 08:22 |
| *** alexpilotti has joined #openstack-nova | 08:22 | |
| *** MultipleCrashes has joined #openstack-nova | 08:23 | |
| *** shaohe_feng has joined #openstack-nova | 08:24 | |
| openstackgerrit | Stephen Finucane proposed openstack/os-vif master: convert os-vif docs to follow PTI https://review.openstack.org/559398 | 08:24 |
| *** cdent has joined #openstack-nova | 08:26 | |
| MultipleCrashes | Hello Guys, Looking for review of this to workflow - https://review.openstack.org/#/c/563418/ , Thanks! | 08:26 |
| *** alexpilotti has quit IRC | 08:27 | |
| *** alexpilotti has joined #openstack-nova | 08:30 | |
| *** shaohe_feng has quit IRC | 08:30 | |
| *** elod has joined #openstack-nova | 08:30 | |
| *** hiro-kobayashi has quit IRC | 08:30 | |
| *** MultipleCrashes has quit IRC | 08:32 | |
| *** shaohe_feng has joined #openstack-nova | 08:33 | |
| *** alexpilotti has quit IRC | 08:35 | |
| *** moshele has joined #openstack-nova | 08:36 | |
| *** alexpilotti has joined #openstack-nova | 08:36 | |
| *** lei-zh- has joined #openstack-nova | 08:38 | |
| *** derekh has joined #openstack-nova | 08:38 | |
| *** Bhujay has joined #openstack-nova | 08:39 | |
| *** shaohe_feng has quit IRC | 08:40 | |
| *** lei-zh1 has quit IRC | 08:41 | |
| *** alexpilotti has quit IRC | 08:41 | |
| *** shaohe_feng has joined #openstack-nova | 08:42 | |
| *** ttsiouts has quit IRC | 08:42 | |
| *** alexpilotti has joined #openstack-nova | 08:44 | |
| *** wwriverrat has quit IRC | 08:44 | |
| *** bkopilov has quit IRC | 08:44 | |
| *** alexpilotti has quit IRC | 08:48 | |
| *** lei-zh1 has joined #openstack-nova | 08:49 | |
| *** lei-zh- has quit IRC | 08:49 | |
| *** shaohe_feng has quit IRC | 08:51 | |
| *** threestrands has quit IRC | 08:51 | |
| openstackgerrit | Merged openstack/nova master: Handle HostMappingNotFound when deleting a compute service https://review.openstack.org/582674 | 08:52 |
| *** ttsiouts has joined #openstack-nova | 08:52 | |
| *** shaohe_feng has joined #openstack-nova | 08:53 | |
| *** alexpilotti has joined #openstack-nova | 08:54 | |
| *** vabada_ has quit IRC | 08:54 | |
| *** vabada has joined #openstack-nova | 08:54 | |
| *** alexpilotti has quit IRC | 08:58 | |
| *** alexpilotti has joined #openstack-nova | 08:59 | |
| *** shaohe_feng has quit IRC | 09:01 | |
| tssurya | alex_xu: good morning! are you around ? | 09:01 |
| *** shaohe_feng has joined #openstack-nova | 09:02 | |
| tssurya | if you have some minutes, could you please have a look at https://review.openstack.org/#/c/581243/ ? its a trivial nit fix for the specs | 09:02 |
| *** owalsh is now known as owalsh_afk | 09:07 | |
| *** Shilpa has quit IRC | 09:08 | |
| *** gongysh has joined #openstack-nova | 09:08 | |
| *** Shilpa has joined #openstack-nova | 09:08 | |
| *** shaohe_feng has quit IRC | 09:11 | |
| *** alexpilotti has quit IRC | 09:13 | |
| *** shaohe_feng has joined #openstack-nova | 09:13 | |
| *** alexpilotti has joined #openstack-nova | 09:17 | |
| *** dtantsur|afk is now known as dtantsur | 09:20 | |
| *** shaohe_feng has quit IRC | 09:21 | |
| *** alexpilotti has quit IRC | 09:21 | |
| *** MultipleCrashes has joined #openstack-nova | 09:22 | |
| *** shaohe_feng has joined #openstack-nova | 09:23 | |
| *** MultipleCrashes has quit IRC | 09:25 | |
| openstackgerrit | Merged openstack/nova master: Transform metrics.update notification https://review.openstack.org/580567 | 09:25 |
| *** jchhatbar has joined #openstack-nova | 09:29 | |
| *** sambetts_ is now known as sambetts | 09:30 | |
| *** alexpilotti has joined #openstack-nova | 09:31 | |
| *** janki has quit IRC | 09:31 | |
| *** shaohe_feng has quit IRC | 09:32 | |
| *** lei-zh1 has quit IRC | 09:32 | |
| *** alexpilotti has quit IRC | 09:33 | |
| *** shaohe_feng has joined #openstack-nova | 09:33 | |
| *** alexpilotti has joined #openstack-nova | 09:33 | |
| openstackgerrit | Stephen Finucane proposed openstack/os-traits master: Switch to stestr https://review.openstack.org/581265 | 09:33 |
| openstackgerrit | Stephen Finucane proposed openstack/os-traits master: tox: Remove '-constraints' targets https://review.openstack.org/583533 | 09:33 |
| openstackgerrit | Stephen Finucane proposed openstack/os-traits master: Follow the new PTI for document build https://review.openstack.org/583534 | 09:33 |
| stephenfin | jaypipes: When you're about https://review.openstack.org/583533 | 09:33 |
| *** hoonetorg has quit IRC | 09:34 | |
| *** GonZo2000 has quit IRC | 09:40 | |
| *** GonZo2000 has joined #openstack-nova | 09:41 | |
| *** GonZo2000 has quit IRC | 09:41 | |
| *** GonZo2000 has joined #openstack-nova | 09:41 | |
| *** vishakha has quit IRC | 09:41 | |
| *** shaohe_feng has quit IRC | 09:42 | |
| *** shaohe_feng has joined #openstack-nova | 09:43 | |
| *** brault has quit IRC | 09:44 | |
| *** brault has joined #openstack-nova | 09:45 | |
| *** JulienAtOvh has joined #openstack-nova | 09:47 | |
| *** hoonetorg has joined #openstack-nova | 09:48 | |
| *** JulienAtOvh has left #openstack-nova | 09:48 | |
| *** GonZo2000 has quit IRC | 09:49 | |
| *** JulienCornuwel has joined #openstack-nova | 09:49 | |
| *** GonZo2000 has joined #openstack-nova | 09:50 | |
| *** moshele has quit IRC | 09:51 | |
| *** moshele has joined #openstack-nova | 09:52 | |
| *** shaohe_feng has quit IRC | 09:52 | |
| *** shaohe_feng has joined #openstack-nova | 09:54 | |
| *** vishakha has joined #openstack-nova | 09:55 | |
| *** shaohe_feng has quit IRC | 10:02 | |
| *** abhishekk has quit IRC | 10:03 | |
| *** shaohe_feng has joined #openstack-nova | 10:03 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge server create for config_drive extension https://review.openstack.org/583538 | 10:06 |
| *** markvoelker has joined #openstack-nova | 10:08 | |
| *** alexchadin has quit IRC | 10:08 | |
| *** kukacz_ has quit IRC | 10:12 | |
| *** shaohe_feng has quit IRC | 10:13 | |
| *** kukacz_ has joined #openstack-nova | 10:13 | |
| *** shaohe_feng has joined #openstack-nova | 10:16 | |
| *** ttsiouts has quit IRC | 10:16 | |
| *** Bhujay has quit IRC | 10:16 | |
| *** ttsiouts has joined #openstack-nova | 10:17 | |
| *** ttsiouts has quit IRC | 10:21 | |
| *** alexchadin has joined #openstack-nova | 10:22 | |
| sean-k-mooney | stephenfin: you know that we dont use constraints in os-vif either. want me to fix that. | 10:22 |
| stephenfin | sean-k-mooney: Yeah, go for it | 10:23 |
| *** shaohe_feng has quit IRC | 10:23 | |
| *** shaohe_feng has joined #openstack-nova | 10:25 | |
| *** shaohe_feng has quit IRC | 10:33 | |
| *** shaohe_feng has joined #openstack-nova | 10:34 | |
| *** maciejjozefczyk has quit IRC | 10:38 | |
| *** maciejjozefczyk has joined #openstack-nova | 10:38 | |
| *** dpawlik has quit IRC | 10:39 | |
| *** Bhujay has joined #openstack-nova | 10:39 | |
| *** gongysh has quit IRC | 10:39 | |
| *** dpawlik has joined #openstack-nova | 10:41 | |
| *** markvoelker has quit IRC | 10:42 | |
| *** priteau has joined #openstack-nova | 10:42 | |
| *** shaohe_feng has quit IRC | 10:43 | |
| *** shaohe_feng has joined #openstack-nova | 10:46 | |
| *** gouthamr has quit IRC | 10:47 | |
| *** damien_r has quit IRC | 10:48 | |
| *** elod has quit IRC | 10:52 | |
| *** shaohe_feng has quit IRC | 10:54 | |
| *** elod has joined #openstack-nova | 10:54 | |
| *** bkopilov has joined #openstack-nova | 10:54 | |
| *** shaohe_feng has joined #openstack-nova | 10:54 | |
| *** dpawlik has quit IRC | 10:55 | |
| *** GonZo2000 has quit IRC | 10:55 | |
| *** dpawlik has joined #openstack-nova | 10:56 | |
| *** jichen has quit IRC | 10:58 | |
| *** elod has quit IRC | 10:59 | |
| *** gongysh has joined #openstack-nova | 10:59 | |
| *** wolverineav has joined #openstack-nova | 10:59 | |
| *** annp has quit IRC | 11:00 | |
| *** gouthamr has joined #openstack-nova | 11:01 | |
| *** shaohe_feng has quit IRC | 11:04 | |
| *** yedongcan has left #openstack-nova | 11:05 | |
| *** shaohe_feng has joined #openstack-nova | 11:08 | |
| slaweq | sean-k-mooney: hi, did You saw my question which I asked earlier? | 11:08 |
| sean-k-mooney | slaweq: no sorry | 11:08 |
| slaweq | sean-k-mooney: no problem, I can write it once again now :) | 11:09 |
| slaweq | so, recently in some neutron jobs I saw failing test tempest.api.compute.admin.test_servers_on_multinodes.ServersOnMultiNodesTest.test_create_server_with_scheduler_hint_group_anti_affinity - it doesn't looks like related to neutron for me | 11:09 |
| slaweq | maybe You are already aware of such issue? Can You take a look on it maybe? | 11:09 |
| slaweq | example of failure is e.g.: | 11:09 |
| slaweq | http://logs.openstack.org/08/555608/34/check/neutron-tempest-multinode-full/1d3aafb/logs/testr_results.html.gz | 11:09 |
| *** ttsiouts has joined #openstack-nova | 11:10 | |
| sean-k-mooney | yes so i was talking to matt riedemann about that job last night | 11:11 |
| sean-k-mooney | or rather that test | 11:11 |
| slaweq | sean-k-mooney: ok, so You are aware and it's not related to neutron, right? | 11:12 |
| sean-k-mooney | i think https://review.openstack.org/#/c/583347/2 will fix it | 11:12 |
| sean-k-mooney | the anti affinity filter is kindof broken because we now allow up to n server to land on the same host | 11:13 |
| slaweq | sean-k-mooney: ok, thx for info, that's enough for me :) | 11:13 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Add /reshaper handler for POST https://review.openstack.org/576927 | 11:13 |
| *** shaohe_feng has quit IRC | 11:14 | |
| sean-k-mooney | slaweq: there is an elastic recheck query for this here http://status.openstack.org/elastic-recheck/#1781710 matt was workin on this and i said i would try and help reproduce | 11:15 |
| *** shaohe_feng has joined #openstack-nova | 11:16 | |
| sean-k-mooney | slaweq: we could add a tempory skip if this is blocking some changes in neutron | 11:16 |
| slaweq | sean-k-mooney: no, it's not so big deal for us because it hits us in non voting jobs mostly | 11:16 |
| slaweq | but I saw it quite many times in last week so I just wanted to ask :) | 11:17 |
| sean-k-mooney | good to know. if it does become a problem feel free to flag it to us agian. | 11:18 |
| slaweq | sean-k-mooney: sure, thx | 11:19 |
| *** pchavva has joined #openstack-nova | 11:20 | |
| *** elod has joined #openstack-nova | 11:20 | |
| *** maciejjozefczyk has quit IRC | 11:21 | |
| *** tssurya has quit IRC | 11:21 | |
| *** maciejjozefczyk has joined #openstack-nova | 11:22 | |
| *** gongysh has quit IRC | 11:22 | |
| *** shaohe_feng has quit IRC | 11:24 | |
| *** maciejjozefczyk has quit IRC | 11:25 | |
| *** shaohe_feng has joined #openstack-nova | 11:25 | |
| *** tssurya has joined #openstack-nova | 11:32 | |
| *** shaohe_feng has quit IRC | 11:35 | |
| *** shaohe_feng has joined #openstack-nova | 11:35 | |
| *** ratailor has quit IRC | 11:38 | |
| *** markvoelker has joined #openstack-nova | 11:39 | |
| openstackgerrit | Merged openstack/os-vif master: convert os-vif docs to follow PTI https://review.openstack.org/559398 | 11:40 |
| *** maciejjozefczyk has joined #openstack-nova | 11:42 | |
| *** sridharg has quit IRC | 11:42 | |
| *** phuongnh has quit IRC | 11:43 | |
| *** maciejjozefczyk has quit IRC | 11:44 | |
| *** shaohe_feng has quit IRC | 11:45 | |
| *** shaohe_feng has joined #openstack-nova | 11:45 | |
| *** sridharg has joined #openstack-nova | 11:49 | |
| *** vladikr has quit IRC | 11:53 | |
| *** lyan has joined #openstack-nova | 11:54 | |
| *** Bhujay has quit IRC | 11:54 | |
| *** lyan is now known as Guest61292 | 11:54 | |
| *** shaohe_feng has quit IRC | 11:55 | |
| *** shaohe_feng has joined #openstack-nova | 11:57 | |
| openstackgerrit | xulei proposed openstack/nova master: Disable limits if force_hosts or force_nodes is set https://review.openstack.org/576693 | 11:59 |
| *** mdnadeem has quit IRC | 12:01 | |
| jaypipes | stephenfin: +2 on all in series. | 12:04 |
| *** markvoelker has quit IRC | 12:05 | |
| jaypipes | stephenfin: thx for spearheading that stuff. it's really not my cup o tea but I'm glad to be aligned with the rest of openstack on it now. | 12:05 |
| *** markvoelker has joined #openstack-nova | 12:05 | |
| *** vladikr has joined #openstack-nova | 12:05 | |
| *** shaohe_feng has quit IRC | 12:05 | |
| *** mdnadeem has joined #openstack-nova | 12:06 | |
| *** shaohe_feng has joined #openstack-nova | 12:06 | |
| *** Dinesh_Bhor has quit IRC | 12:06 | |
| *** tiendc has quit IRC | 12:07 | |
| *** pchavva has quit IRC | 12:07 | |
| *** edmondsw has joined #openstack-nova | 12:11 | |
| *** shaohe_feng has quit IRC | 12:16 | |
| *** jbcraig has joined #openstack-nova | 12:18 | |
| *** shaohe_feng has joined #openstack-nova | 12:21 | |
| *** owalsh_afk is now known as owalsh | 12:23 | |
| openstackgerrit | Merged openstack/nova master: hardware: Start accounting for networks in NUMA fitting https://review.openstack.org/564448 | 12:24 |
| openstackgerrit | Merged openstack/nova master: objects: Add RequestSpec.network_metadata https://review.openstack.org/564442 | 12:24 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge server create for bdm extensions https://review.openstack.org/583559 | 12:25 |
| *** shaohe_feng has quit IRC | 12:26 | |
| openstackgerrit | Merged openstack/os-traits master: tox: Remove '-constraints' targets https://review.openstack.org/583533 | 12:28 |
| openstackgerrit | Merged openstack/os-traits master: Follow the new PTI for document build https://review.openstack.org/583534 | 12:28 |
| *** shaohe_feng has joined #openstack-nova | 12:28 | |
| *** BrinZhang has quit IRC | 12:28 | |
| *** dpawlik has quit IRC | 12:30 | |
| openstackgerrit | Merged openstack/os-traits master: Switch to stestr https://review.openstack.org/581265 | 12:30 |
| *** jbcraig has quit IRC | 12:31 | |
| *** pchavva has joined #openstack-nova | 12:32 | |
| *** elod has quit IRC | 12:35 | |
| *** shaohe_feng has quit IRC | 12:36 | |
| *** elod has joined #openstack-nova | 12:38 | |
| *** shaohe_feng has joined #openstack-nova | 12:38 | |
| *** mriedem has joined #openstack-nova | 12:39 | |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: remove unused attribute driver for LibvirtConfigNodeDevice https://review.openstack.org/583246 | 12:41 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: add format_dom for ConfigNodeDevicePciSubFunctionCap https://review.openstack.org/583247 | 12:42 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: add missing type for LibvirtConfigNodeDevicePciCap https://review.openstack.org/583248 | 12:42 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: add format_dom() for config LibvirtConfigNodeDevicePciCap https://review.openstack.org/583249 | 12:42 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: add format_dom() method for LibvirtConfigNodeDevice https://review.openstack.org/583250 | 12:42 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: add generater of fake pci devices capabilities https://review.openstack.org/583251 | 12:42 |
| *** slaweq has quit IRC | 12:46 | |
| *** shaohe_feng has quit IRC | 12:46 | |
| sean-k-mooney | stephenfin: jaypipes efried so regarding https://review.openstack.org/#/c/583533/1 | 12:47 |
| *** maciejjozefczyk has joined #openstack-nova | 12:47 | |
| *** dpawlik has joined #openstack-nova | 12:47 | |
| sean-k-mooney | that will break if os-traits is added to upper-constraints | 12:47 |
| *** shaohe_feng has joined #openstack-nova | 12:47 | |
| sean-k-mooney | for libs you have to preprocess the upper-constratins and remove them from the list | 12:47 |
| *** maciejjozefczyk has quit IRC | 12:51 | |
| sean-k-mooney | ill create a script that works and propose it to both os-vif and os-traits | 12:51 |
| *** dpawlik has quit IRC | 12:52 | |
| openstackgerrit | sahid proposed openstack/nova master: hardware: fix memory check usage for small/large pages https://review.openstack.org/532168 | 12:52 |
| jaypipes | sean-k-mooney: I shall let you and stephenfin fight that out in a death match. | 12:54 |
| * jaypipes grabs popcorn | 12:54 | |
| *** slaweq has joined #openstack-nova | 12:55 | |
| sean-k-mooney | hehe well what stephenfin has will work. until the point that os-traits gets added to upperconstraints | 12:55 |
| sean-k-mooney | its exactly how nova works but nova is not a lib | 12:56 |
| *** shaohe_feng has quit IRC | 12:57 | |
| *** shaohe_feng has joined #openstack-nova | 12:57 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Handle HostMappingNotFound when deleting a compute service https://review.openstack.org/583566 | 12:59 |
| do3meli | someone able to review this one here: https://review.openstack.org/#/c/578040/ ? thanks | 13:00 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Handle HostMappingNotFound when deleting a compute service https://review.openstack.org/583567 | 13:01 |
| *** vladikr has quit IRC | 13:04 | |
| *** vladikr has joined #openstack-nova | 13:04 | |
| *** shaohe_feng has quit IRC | 13:07 | |
| *** shaohe_feng has joined #openstack-nova | 13:08 | |
| *** moshele has quit IRC | 13:14 | |
| *** shaohe_feng has quit IRC | 13:17 | |
| *** rmart04 has joined #openstack-nova | 13:18 | |
| *** lbragstad has quit IRC | 13:19 | |
| *** mchlumsky has joined #openstack-nova | 13:19 | |
| *** shaohe_feng has joined #openstack-nova | 13:21 | |
| *** vladikr has quit IRC | 13:24 | |
| *** vladikr has joined #openstack-nova | 13:24 | |
| *** eharney has joined #openstack-nova | 13:26 | |
| *** shaohe_feng has quit IRC | 13:27 | |
| JulienCornuwel | Hi, since the deprecation of nova hooks, I wonder what would be the correct way to handle my use case: I'm doing pci_passthrough of NVME drives to instances, and I want to wipe them clean when the customer destroys his instance. I can't use qemu hooks since they are run at stop/start and I don't want to wipe the drive on a simple reboot. Any pointers? | 13:27 |
| *** shaohe_feng has joined #openstack-nova | 13:28 | |
| *** gcb has quit IRC | 13:31 | |
| *** alexchadin has quit IRC | 13:34 | |
| gibi | JulienCornuwel: you can wait for instance.delete.end notifications from nova https://docs.openstack.org/nova/latest/reference/notifications.html | 13:34 |
| *** alexchadin has joined #openstack-nova | 13:34 | |
| *** alexchadin has quit IRC | 13:35 | |
| pooja_jadhav | dansmith: Hello | 13:35 |
| *** alexchadin has joined #openstack-nova | 13:35 | |
| JulienCornuwel | gibi: Is there a way to prevent nova from giving the drive to another customer until I'm done with the wiping? (takes about 30-40 seconds) I've done some tests, and if I run a nova boot during the wipe, nova just takes the device wether I'm done cleaning it or not… | 13:37 |
| efried | do3meli: Reviewed | 13:38 |
| do3meli | thx :efried. will work through it now | 13:38 |
| *** shaohe_feng has quit IRC | 13:38 | |
| efried | JulienCornuwel: Are you looking to write code to do this, or are you trying to do it under the existing codebase? | 13:38 |
| gibi | JulienCornuwel: that would require a way to reserve the given PCI device while you are wipeing it | 13:38 |
| stephenfin | sean-k-mooney: I saw that tox install script in a couple of oslo repos but not all of them. I thought mordred or someone had worked around this now, tbh | 13:39 |
| efried | pooja_jadhav: I think Dan is on vacation this week. | 13:39 |
| *** shaohe_feng has joined #openstack-nova | 13:40 | |
| pooja_jadhav | efried: Ohh.. thanks for update. I want to discuss about the patch https://review.openstack.org/#/c/580720/ | 13:40 |
| mordred | stephenfin: we still have tos intall scripts laying around? | 13:40 |
| stephenfin | mordred: aye, lemme root some out | 13:40 |
| sean-k-mooney | stephenfin: maybe but when i made the same change to os-vif it conmplained | 13:40 |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Microversion 2.64 - Use new format policy in server group https://review.openstack.org/578261 | 13:41 |
| mgariepy | hello, does "nova-manage cell_v2 map_instances" needs to be run on every upgrade ? It ran on my upgrade from newton to ocata and on the ocata to pike taking about 40 minutes each time. | 13:41 |
| efried | pooja_jadhav: Maybe someone here can help. What do you need? | 13:41 |
| JulienCornuwel | efried: We could write some code. But I was wondering if there was already a way that I was missing. | 13:41 |
| stephenfin | mordred: I see oslo.log, oslo.db, and oslo.cache locally | 13:42 |
| JulienCornuwel | gibi: exactly, is there a way to do that. | 13:42 |
| stephenfin | though there could be more that I don't have local copies of | 13:42 |
| *** munimeha1 has joined #openstack-nova | 13:42 | |
| sean-k-mooney | stephenfin: as far as i am aware all libs need to have the script | 13:42 |
| efried | JulienCornuwel: I can't think of anything. If you're writing code, you could do as gibi suggests and, in response to the notification, remove the device from inventory reporting (get_available_resource) until you're done with it. | 13:43 |
| mordred | stephenfin: wow. just when I thought we'd gotten rid of all of those :) | 13:43 |
| gibi | JulienCornuwel: I don't think we have current a way to reserver a PCI dev | 13:43 |
| pooja_jadhav | efried: Based on above patch https://review.openstack.org/#/c/580720/ submitted by Dan, When we boot an instance from volume then it should not consider local disk_gb. But I have applied patch and tried to tested the same but i can that local disk gb is getting used by seeing nova hypervisor-stats. | 13:44 |
| *** maciejjozefczyk has joined #openstack-nova | 13:44 | |
| sean-k-mooney | stephenfin: mordred this is what i get with the same change to os-vif | 13:44 |
| sean-k-mooney | http://paste.openstack.org/show/726197/ | 13:44 |
| efried | pooja_jadhav: libvirt, I assume? | 13:44 |
| pooja_jadhav | see* | 13:45 |
| stephenfin | sean-k-mooney: Stop overriding install_command | 13:45 |
| pooja_jadhav | efried: yes | 13:45 |
| stephenfin | sean-k-mooney: It should simply read 'install_command = pip install {opts} {packages}' (or possibly it can be unset - I'm not sure). mordred can probably explain why that breaks | 13:45 |
| sean-k-mooney | stephenfin: this is what i have install_command = pip install -U -c {env:UPPER_CONSTRAINTS_FILE:https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt} {opts} {packages} | 13:46 |
| stephenfin | sean-k-mooney: Yup, see above | 13:46 |
| *** lbragstad has joined #openstack-nova | 13:46 | |
| JulienCornuwel | efried: gibi: Thanks for the confirmation. We'll start working on a spec and thinking of a way to do that. | 13:46 |
| efried | pooja_jadhav: So the instance is actually booting from volume - i.e. truly not using the local disk - but nova hypervisor-stats is showing that capacity as having been applied from the local disk? | 13:46 |
| sean-k-mooney | oh you put the constatit in deps | 13:46 |
| tssurya | mgariepy: well the aim of map_instances is to just map the instances to the cells, so if you have done it once, and you are sure all instances are mapped, I don't see any reason to keep running it, unless you have new ones to populate | 13:46 |
| stephenfin | sean-k-mooney: Ah, I get it. You don't want to install your local package with constraints but you _do_ want to install your test requirements | 13:47 |
| stephenfin | ...with them | 13:47 |
| mgariepy | tssurya, https://github.com/openstack/openstack-ansible-os_nova/blob/stable/pike/tasks/nova_db_setup.yml#L58-L90 | 13:47 |
| stephenfin | sean-k-mooney: yuuup | 13:47 |
| efried | JulienCornuwel: Note that things involving the existing PCI passthrough subsystem are going to be difficult to get approval for, because we're trying to get rid of that whole thing. | 13:47 |
| sean-k-mooney | stephenfin: yep | 13:47 |
| mgariepy | so if the cell_v2 mapping was done in the upgrade, (and the create_cell returns 2, it would be right to skip the map_instances | 13:47 |
| *** shaohe_feng has quit IRC | 13:48 | |
| JulienCornuwel | efried: In favor of? Cyborg? | 13:48 |
| efried | JulienCornuwel: That said, I think you could do what you want without really hitting that code; and in such a way that it would be transferrable to placement-based device passthrough in the future. | 13:48 |
| *** r-daneel has joined #openstack-nova | 13:48 | |
| pooja_jadhav | efried: After looking at the code changes done by Dan, its not taking disk_gb resource. When allocation candidates returns the response. | 13:48 |
| efried | JulienCornuwel: Well, yes, but more generally placement. | 13:49 |
| mordred | stephenfin, sean-k-mooney the trick is putting -c{env:UPPER_CONSTRAINTS_FILE:https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt} into the deps list | 13:49 |
| mordred | not the install_command | 13:49 |
| tssurya | mgariepy: yes exactly | 13:49 |
| mordred | like http://git.openstack.org/cgit/openstack/openstacksdk/tree/tox.ini | 13:49 |
| sean-k-mooney | mordred: ya that is sneaky i like it | 13:49 |
| mgariepy | thanks :) | 13:49 |
| mordred | sean-k-mooney: it's the necessary trick for the self-constraint problem ;) | 13:50 |
| tssurya | mgariepy: no problem | 13:50 |
| efried | pooja_jadhav: It sounds like you should downvote the patch (it needs a rebase anyway, so you won't be the only thing holding it up) and explain what you're seeing. | 13:50 |
| *** burt has joined #openstack-nova | 13:50 | |
| efried | pooja_jadhav: What would be even better is if you could come up with a functional test that demonstrates the behavior. | 13:50 |
| mordred | tox installs the deps line separate from the current project - but it uses install_command for both things | 13:50 |
| *** dklyle has joined #openstack-nova | 13:50 | |
| sean-k-mooney | mordred: it should also work on cygwin which the install script version did not | 13:50 |
| mordred | so by putting constraints in deps, it winds up on the command line for the deps - but not for the install of the current project | 13:50 |
| mordred | sean-k-mooney: yup | 13:51 |
| *** shaohe_feng has joined #openstack-nova | 13:51 | |
| pooja_jadhav | efried: But at https://github.com/openstack/nova/blob/master/nova/objects/build_request.py#L64 I can see in this db_instance the root_gb=1 value is going. | 13:51 |
| *** gongysh has joined #openstack-nova | 13:51 | |
| *** dpawlik has joined #openstack-nova | 13:51 | |
| mordred | sean-k-mooney: there are many reasons we don't like those install script wrappers - we forget to communicate all of them :) | 13:51 |
| pooja_jadhav | efried: ok | 13:52 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform volume.usage notification https://review.openstack.org/580345 | 13:52 |
| JulienCornuwel | efried: thanks, we'll look into that | 13:53 |
| sean-k-mooney | mordred: well os-vif never used the install script because it broke my cygwin workflow at the time. | 13:53 |
| *** awaugama has joined #openstack-nova | 13:53 | |
| tssurya | jaypipes: you appear to be the only spec core online now; whenever you get the time could you please have a look at https://review.openstack.org/#/c/581243/ ? It a very easy not fix follow-up on a merged spec | 13:53 |
| tssurya | nit* | 13:53 |
| mordred | sean-k-mooney: \o/ | 13:54 |
| *** ttsiouts has quit IRC | 13:56 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform volume.usage notification https://review.openstack.org/580345 | 13:57 |
| *** shaohe_feng has quit IRC | 13:58 | |
| efried | JulienCornuwel: To be a bit more specific: your notification listener would conceivably start the cleanup process, and also register the device in some kind of cache ("reserved devices"). Add logic to your get_available_resource method to look at that cache and skip reporting any devices therein. | 13:59 |
| efried | So when we make the move to placement-based device reporting, the latter logic would go into update_provider_tree instead of get_available_resource, but everything else would stay the same. | 13:59 |
| efried | There may still be a race in this setup, because there may be a delay before get_available_resource/update_provider_tree is run. I suppose you could get around this by making your spawn method add the device to the "reserved" list, and then have your notification listener thingy only be responsible for removing it when done. | 13:59 |
| openstackgerrit | sean mooney proposed openstack/os-vif master: add upper_constraitns support https://review.openstack.org/583582 | 13:59 |
| sean-k-mooney | we should backport ^ | 14:00 |
| *** dklyle has quit IRC | 14:00 | |
| *** shaohe_feng has joined #openstack-nova | 14:01 | |
| sean-k-mooney | ok going to grab lunch brb | 14:01 |
| *** ttsiouts has joined #openstack-nova | 14:01 | |
| openstackgerrit | do3meli proposed openstack/nova master: docs: add nova host-evacuate command to evacuate documentation https://review.openstack.org/578040 | 14:02 |
| efried | sean-k-mooney: We should spell it right, though. | 14:06 |
| stephenfin | efried: It's the main reason we keep jaypipes around, tbh | 14:08 |
| sean-k-mooney | dambit hehe ill fix that when i get back | 14:08 |
| efried | sean-k-mooney: I'll push the edit right quick. | 14:08 |
| openstackgerrit | Eric Fried proposed openstack/os-vif master: add upper_constraints support https://review.openstack.org/583582 | 14:08 |
| stephenfin | spellchecker extraordinaire | 14:08 |
| *** shaohe_feng has quit IRC | 14:08 | |
| *** shaohe_feng has joined #openstack-nova | 14:09 | |
| efried | sean-k-mooney: +2 | 14:09 |
| openstackgerrit | Matt Riedemann proposed openstack/os-vif master: add upper_constraints support https://review.openstack.org/583582 | 14:09 |
| efried | mriedem: wtf, yo | 14:09 |
| *** ttsiouts has quit IRC | 14:10 | |
| stephenfin | efried: It's alright - we'd missed anyway | 14:10 |
| sean-k-mooney | mriedem: yes you are right ill fix those | 14:10 |
| efried | I'm still not clear that we care about upper constraints (or constraitns or constranits) for doc builds, but okay. | 14:11 |
| sean-k-mooney | oh the pti change has merged that why i missed the others | 14:12 |
| jaypipes | stephenfin: glad to be of service. :) | 14:12 |
| sean-k-mooney | ok got to run to shop back in 15 min ill fix it all up then | 14:12 |
| stephenfin | efried: Yeah, I'm on the fence myself about them, despite suggesting it in the first place | 14:13 |
| jaypipes | tssurya: done! | 14:13 |
| *** Bhujay has joined #openstack-nova | 14:17 | |
| *** shaohe_feng has quit IRC | 14:19 | |
| *** shaohe_feng has joined #openstack-nova | 14:19 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform libvirt.error notification https://review.openstack.org/484851 | 14:22 |
| mriedem | tssurya: the handling a down cell changes seem to be kind of all over the place, can you rebase those into a series? | 14:23 |
| mriedem | where the api changes with the microversion come at the end? | 14:23 |
| mriedem | i would do in this order: object change for queued_for_delete -> online data migration -> setting queued_for_delete -> handling a down cell + microversion | 14:24 |
| *** mlavalle has joined #openstack-nova | 14:24 | |
| *** Bhujay has quit IRC | 14:25 | |
| *** antosh has joined #openstack-nova | 14:25 | |
| *** udesale_ has joined #openstack-nova | 14:26 | |
| *** udesale has quit IRC | 14:27 | |
| efried | pooja_jadhav: Are you using DiskFilter? | 14:29 |
| *** shaohe_feng has quit IRC | 14:29 | |
| *** shaohe_feng has joined #openstack-nova | 14:29 | |
| pooja_jadhav | pooja_jadhav: I have checked nova.conf, also nova-cpu.conf.. in that in enable_filters there is no any DiskFilter is added. | 14:30 |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Microversion 2.64 - Use new format policy in server group https://review.openstack.org/578261 | 14:30 |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Add support for microversion 2.65 https://review.openstack.org/583632 | 14:30 |
| *** moshele has joined #openstack-nova | 14:32 | |
| mriedem | pooja_jadhav: which release are you on? | 14:32 |
| openstackgerrit | Merged openstack/nova-specs master: Fix nits in the handling down cell spec https://review.openstack.org/581243 | 14:32 |
| mriedem | master? | 14:32 |
| pooja_jadhav | pooja_jadhav: Current master | 14:32 |
| pooja_jadhav | mriedem: from cuurent master, fetched patch | 14:33 |
| pooja_jadhav | using latest code | 14:33 |
| mriedem | and you boot from volume and nova hypervisor-stats is showing local_gb_used=1? | 14:33 |
| pooja_jadhav | yes | 14:33 |
| *** ttsiouts has joined #openstack-nova | 14:33 | |
| mriedem | are you using devstack? | 14:33 |
| pooja_jadhav | yes | 14:33 |
| *** jangutter has quit IRC | 14:34 | |
| mriedem | and libvirt driver correct? | 14:34 |
| pooja_jadhav | yes | 14:34 |
| *** jangutter has joined #openstack-nova | 14:34 | |
| mriedem | does the flavor have ephemeral_gb? | 14:34 |
| *** alexchadin has quit IRC | 14:35 | |
| pooja_jadhav | using flavor 1 (ephemeral_gb=0, swap=0) | 14:35 |
| mriedem | what is reserved_host_disk_mb set to in nova-cpu.conf? | 14:35 |
| pooja_jadhav | not set any value | 14:36 |
| tssurya | jaypipes: thanks | 14:36 |
| mriedem | pooja_jadhav: efried: oh i see exactly why | 14:37 |
| efried | mriedem: Do tell. I've been staring at this and slowly gleaning understanding, but am still out of my depth. | 14:38 |
| mriedem | https://github.com/openstack/nova/blob/eb4f65a7951e921b1cd8d05713e144e72f2f254f/nova/compute/resource_tracker.py#L1455 | 14:38 |
| mriedem | the usage dict doesn't know about bfv | 14:38 |
| mriedem | pooja_jadhav: can you check the resource allocations for that instance to make sure DISK_GB is unset or 0? | 14:38 |
| mriedem | pooja_jadhav: https://docs.openstack.org/osc-placement/latest/cli/index.html#resource-provider-allocation-show | 14:39 |
| mriedem | openstack resource provider allocation show <instance uuid> | 14:39 |
| *** shaohe_feng has quit IRC | 14:39 | |
| tssurya | mriedem: so the what I have it now is change for qfd -> migration and setting qfd sit on this. The patch for API changes is just a POC/WIP for nova list which I am working on to include the nova show as well. | 14:39 |
| mriedem | DISK_GB should either not be there or 0 | 14:39 |
| tssurya | the way* | 14:39 |
| pooja_jadhav | allocation_candidates not returning DISK_GB | 14:40 |
| mriedem | tssurya: right i don't like how the series diverges | 14:40 |
| mriedem | tssurya: it would be nice to line them all up | 14:40 |
| mriedem | pooja_jadhav: you mean "openstack resource provider allocation show <instance uuid>" doesn't show DISK_GB? | 14:40 |
| tssurya | mriedem: ack,I will rebase them the way you said | 14:40 |
| mriedem | tssurya: thanks | 14:40 |
| *** shaohe_feng has joined #openstack-nova | 14:40 | |
| mriedem | pooja_jadhav: if that's the case, then it's working as designed in dansmith's change, and we just have a latent bug in the resource tracker for reporting disk uage for volume-backed instances | 14:41 |
| mriedem | which isn't surprising to me | 14:41 |
| pooja_jadhav | its showing {u'VCPU': 1, u'MEMORY_MB': 512} | 14:41 |
| *** alexchadin has joined #openstack-nova | 14:41 | |
| pooja_jadhav | only | 14:41 |
| mriedem | and could be fixed separate from dansmith's change | 14:41 |
| mriedem | efried: ^ agree? | 14:41 |
| mriedem | pooja_jadhav: you could report a nova bug for that RT issue | 14:41 |
| mriedem | pooja_jadhav: this specifically https://github.com/openstack/nova/blob/eb4f65a7951e921b1cd8d05713e144e72f2f254f/nova/compute/resource_tracker.py#L1455 | 14:41 |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Update queued-for-delete from the ComputeAPI during deletion/restoration https://review.openstack.org/566813 | 14:41 |
| efried | mriedem: Not sure about "separate from dansmith's change". Gimme a minute here. | 14:42 |
| mriedem | efried: dan's change is really about allocations in placement, | 14:42 |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: [POC] Graceful handling of nova-list when a cell is down https://review.openstack.org/567785 | 14:42 |
| mriedem | there are still known issues with reporting disk usage, like if you have the DiskFilter enabled it's going to do a disk claim in the RT | 14:42 |
| mriedem | we can't drop the disk claims until we drop caching scheduler | 14:42 |
| mriedem | since it doesn't use placement | 14:42 |
| mriedem | fixing https://github.com/openstack/nova/blob/eb4f65a7951e921b1cd8d05713e144e72f2f254f/nova/compute/resource_tracker.py#L1455 is going to be non-trivial i think | 14:43 |
| mriedem | well maybe it's not that bad, we have an instance uuid in scope when it's used and from the instance we can determine if it's volume-backed or not | 14:44 |
| *** tidwellr has joined #openstack-nova | 14:45 | |
| *** namnh has joined #openstack-nova | 14:45 | |
| efried | mriedem: How do I access the compute node object from the compute service thingy (self.compute1)? | 14:47 |
| *** lpetrut has quit IRC | 14:47 | |
| *** moshele has quit IRC | 14:48 | |
| pooja_jadhav | mriedem: you mean to say 'root_gb' is taking directly from flavor.root_gb. whether we create instance from volume or not? correct? | 14:49 |
| mriedem | efried: in a functional test? | 14:49 |
| mriedem | pooja_jadhav: correct | 14:49 |
| efried | mriedem: yeah | 14:49 |
| mriedem | efried: self.compute1 is the service record, | 14:49 |
| *** shaohe_feng has quit IRC | 14:49 | |
| *** shaohe_feng has joined #openstack-nova | 14:50 | |
| mriedem | from that you can get the manager and host i think, and from that you can get the compute node record by hostname and then get it's uuid | 14:50 |
| mriedem | self.compute1.manager.host | 14:50 |
| *** peereb has quit IRC | 14:51 | |
| mriedem | that's how ComputeManager._get_compute_nodes_in_db works anyway | 14:51 |
| efried | mriedem: That's just a string. | 14:51 |
| *** sgordon has quit IRC | 14:51 | |
| mriedem | right to lookup the compute node | 14:52 |
| pooja_jadhav | mriedem: in that case, we need to check, if the instance is volume_backed then make root_gb is to 0. | 14:52 |
| mriedem | i guess you could use self.compute1.manager._resource_tracker.compute_nodes | 14:52 |
| mriedem | assuming those are set by the time you access the thing | 14:52 |
| mriedem | pooja_jadhav: yes i understand the issue, | 14:52 |
| mriedem | pooja_jadhav: but i don't think it needs to be done in dansmith's change | 14:52 |
| mriedem | it's a separate reporting issue which can be fixed on top of dan's change | 14:52 |
| pooja_jadhav | mriedem: Ohk | 14:53 |
| efried | mriedem: it is, thanks. | 14:53 |
| mriedem | pooja_jadhav: but if you can report the nova bug that would be helpful | 14:53 |
| efried | pooja_jadhav: Let me know when you have that bug open. I'm working on the test case for it. | 14:53 |
| pooja_jadhav | efried: Ok sure. | 14:54 |
| mriedem | i'm going to rebase dan's change | 14:54 |
| sahid | jaypipes: did you had a chance to look at this os_acc project? | 14:54 |
| sahid | https://review.openstack.org/#/c/583097/ | 14:54 |
| efried | mriedem: ack, fwiw there were no conflicts when I did that locally, even though gerrit complained. | 14:54 |
| mriedem | i've been meaning to follow up on dan's change and fix the move operation stuff too | 14:54 |
| sahid | it seems that they want to build the guest configuration (for libvirt domain XML) | 14:55 |
| efried | mriedem: mebbe there's something I need to pull on master since last night. | 14:55 |
| efried | mriedem: If it does wind up being manual, you wanna fix those nits I noted? | 14:55 |
| *** cfriesen_ has joined #openstack-nova | 14:55 | |
| mriedem | efried: yeah i'll fix those | 14:55 |
| jaypipes | sahid: not really yet, sorry :( | 14:55 |
| sahid | jaypipes: no no worries, it's just that i don't want give to them bad direction | 14:56 |
| mriedem | yeah stephenfin's RequestSpec.network_metadata change landed | 14:56 |
| *** pcaruana has quit IRC | 14:56 | |
| kashyap | sahid: What is that project even? | 14:56 |
| sahid | it's not really clear for me, my thinking is Cyborg would be that generic device manager we are missing and os-acc the data model used to talk between cyborg and nova | 14:57 |
| jangutter | kashyap: My personal lossy summary of Cyborg: How to cloud-ify FPGA's. | 14:58 |
| jangutter | kashyap, sahid: the first use-case is for FPGA's, and hopefully it should be a general device manager. The idea is that os-acc should function as the object model and a plugin model for Nova. | 14:59 |
| kashyap | jangutter: "Someone" needs to write-up a damn good spec with a clear beginning, middle and an end. | 14:59 |
| *** med_ has joined #openstack-nova | 14:59 | |
| sahid | jangutter: ok thanks for i have well understood the thing :) | 15:00 |
| *** shaohe_feng has quit IRC | 15:00 | |
| efried | kashyap: There are specs. | 15:00 |
| kashyap | jangutter: But noted :-) | 15:00 |
| efried | kashyap: They're in the cyborg repo, not the cyborg-specs repository, confusingly. | 15:00 |
| *** mdnadeem has quit IRC | 15:00 | |
| kashyap | efried: Oh, I'll have to look it up | 15:00 |
| *** shaohe_feng has joined #openstack-nova | 15:00 | |
| jangutter | kashyap: I accidentally got this info via osmosis in Dublin. | 15:00 |
| kashyap | Also that patch should be broken down and made into series with great (yes, great) commit messages. | 15:00 |
| *** jmlowe has quit IRC | 15:00 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Add gabbi coverage for an inventory change https://review.openstack.org/583642 | 15:01 |
| jangutter | kashyap: hehe, first review comment should be a link to the spec, probably. | 15:01 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Add gabbi coverage for inv of missing rp https://review.openstack.org/583643 | 15:01 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] cover bad content-length header https://review.openstack.org/583644 | 15:01 |
| efried | kashyap: doc/source/specs in openstack/cyborg, and one open patch: https://review.openstack.org/#/c/577438/ | 15:02 |
| *** ttsiouts has quit IRC | 15:03 | |
| *** ttsiouts has joined #openstack-nova | 15:04 | |
| *** hongbin has joined #openstack-nova | 15:06 | |
| mriedem | efried: can i deal with the changed link in https://review.openstack.org/#/c/515423/29 in a follow up? also your concern about the test isn't something i'm worried about - the dicts in question have only a single key | 15:07 |
| *** elod has quit IRC | 15:07 | |
| *** artom_ has joined #openstack-nova | 15:08 | |
| *** elod has joined #openstack-nova | 15:09 | |
| *** shaohe_feng has quit IRC | 15:10 | |
| efried | mriedem: okay, +2. | 15:10 |
| mriedem | thanks; looks like there are other things in the series from which i can draw a fup cleanup patch | 15:10 |
| *** alexchadin has quit IRC | 15:10 | |
| *** artom has quit IRC | 15:10 | |
| *** shaohe_feng has joined #openstack-nova | 15:11 | |
| efried | mriedem: Are you in the middle of that rebase or can I push that test case on top and have you carry it along? | 15:12 |
| efried | pooja_jadhav: How's that bug report coming? | 15:12 |
| *** gongysh has quit IRC | 15:13 | |
| *** namnh has quit IRC | 15:13 | |
| mriedem | efried: pushing now | 15:13 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Avoid requesting DISK_GB allocation for root_gb on BFV instances https://review.openstack.org/580720 | 15:14 |
| *** ttsiouts has quit IRC | 15:14 | |
| *** lpetrut has joined #openstack-nova | 15:16 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Func test for improper local DISK_GB reporting https://review.openstack.org/583646 | 15:18 |
| efried | mriedem, pooja_jadhav: ^ | 15:18 |
| efried | Just need to fill in that bug number. | 15:18 |
| *** shaohe_feng has quit IRC | 15:20 | |
| mriedem | efried: funny it doesn't report swap disk usage | 15:20 |
| *** shaohe_feng has joined #openstack-nova | 15:23 | |
| mriedem | comments inline, but lgtm | 15:23 |
| *** jmlowe has joined #openstack-nova | 15:24 | |
| *** maciejjozefczyk has quit IRC | 15:26 | |
| *** wolverineav has quit IRC | 15:26 | |
| *** wolverineav has joined #openstack-nova | 15:27 | |
| *** ttsiouts has joined #openstack-nova | 15:27 | |
| *** ttsiouts has quit IRC | 15:28 | |
| mriedem | so it appears that going back to grizzly when the DiskFilter was added, we only accounted for local disk usage using root_gb and ephemeral_gb, not swap https://review.openstack.org/#/c/13182/ | 15:28 |
| *** chyka has joined #openstack-nova | 15:28 | |
| *** shaohe_feng has quit IRC | 15:30 | |
| mriedem | the DiskFilter was fixed in Icehouse but the RT wasn't https://review.openstack.org/#/c/51323/ | 15:30 |
| * mriedem opens bug for something that's been broken since grizzly... | 15:30 | |
| *** shaohe_feng has joined #openstack-nova | 15:32 | |
| efried | mriedem: That /os-hypervisors/statistics API isn't working, something missing in the fixture setup. Worth fixing that? | 15:32 |
| *** rmart04 has quit IRC | 15:32 | |
| mriedem | efried: not working how? | 15:33 |
| mriedem | are you using the admin_api client? | 15:33 |
| *** wwriverrat has joined #openstack-nova | 15:33 | |
| efried | mriedem: http://paste.openstack.org/show/726208/ | 15:33 |
| efried | Probably not :) | 15:33 |
| efried | mriedem: Same thing via the admin_api | 15:35 |
| efried | Looking at the func test that actually does this... | 15:35 |
| mriedem | neo huh | 15:35 |
| mriedem | /home/efried/Neo/nova/.tox/functional | 15:35 |
| mriedem | tsk tsk | 15:36 |
| efried | mriedem: swhat our project used to be called before it was called NovaLink | 15:36 |
| mriedem | oh i know | 15:36 |
| mriedem | i knew about neo before it couldn't be talked about openly | 15:37 |
| *** lpetrut has quit IRC | 15:37 | |
| mriedem | i'm a neo hipster really | 15:37 |
| efried | 'cept you can't grow a beard | 15:39 |
| mriedem | i don't see anything special in the os-hypervisors api samples functional tests for the statistics call | 15:39 |
| mriedem | hipster bears are on their way out | 15:39 |
| mriedem | *beards | 15:39 |
| efried | mriedem: Me neither. I thought it might be the json headers, but I tried adding those and it didn't help. | 15:39 |
| efried | So what you're saying is you were a beardless hipster before it was cool. | 15:39 |
| efried | ya know, it's possible I'm injecting unintended side effects by using remote_pdb. Lemme try unwinding that... | 15:40 |
| *** JulienCornuwel has quit IRC | 15:41 | |
| *** shaohe_feng has quit IRC | 15:41 | |
| mriedem | efried: you just blew my mind | 15:41 |
| *** sridharg has quit IRC | 15:41 | |
| *** shaohe_feng has joined #openstack-nova | 15:41 | |
| *** ispp has quit IRC | 15:41 | |
| *** vladikr has quit IRC | 15:42 | |
| melwitt | mriedem: we have a bug where the libvirt driver will unconditionally destroy disks even on shared storage during a rebuild failure, and we're looking at how to write the fix in such a way that it's backportable. what we really need to know in the libvirt driver spawn() is whether recreate=True or destroy_disks_on_failure should be True/False. would such a change to the spawn method signature be banned for a backport? | 15:43 |
| efried | mriedem: That was it, though I don't know why. | 15:43 |
| melwitt | mriedem: this is the WIP of the change, avoiding the method signature change, and it duplicates code from compute manager's default rebuild impl into the libvirt driver https://review.openstack.org/578846 | 15:44 |
| *** vladikr has joined #openstack-nova | 15:44 | |
| mriedem | melwitt: i had a change related to that same bug which did the logic in the spawn method based on the instance.task_state | 15:45 |
| melwitt | mdbooth ^ | 15:45 |
| mriedem | https://review.openstack.org/#/c/288109/1/nova/virt/libvirt/driver.py | 15:45 |
| mriedem | i told mdbooth about that 2 weeks ago | 15:45 |
| mriedem | and he was aware of my old attempt | 15:46 |
| *** ragiman has quit IRC | 15:47 | |
| mriedem | melwitt: i would hack something backportable into the driver spawn() based on task_state and/or active evac migration record (as cfriesen_ noted in my change), and then make the driver signature change on master only | 15:47 |
| mdbooth | So, the driver signature change would be something like adding destroy_disks_on_failure to spawn(). | 15:48 |
| mdbooth | Although as noted in my comment there's scope to make the libvirt driver cleverer still. | 15:49 |
| mdbooth | mriedem: So... task state for the backport only? | 15:49 |
| * mdbooth doesn't like overloading task state in general as it gets really complicated. | 15:50 | |
| mriedem | mdbooth: if the change could be confined to a tactical fix in the spawn() method itself then that's easier than making interface changes (even though that's allowed and unversioned) | 15:50 |
| *** med_ has quit IRC | 15:50 | |
| mriedem | but as i said, cfriesen_ made a point about needing to know if we're evacuating or not, which you can only know based on the task_state + active migration record | 15:50 |
| mriedem | because evac uses the same task_states as evacuate (i think) | 15:50 |
| *** wwriverrat has quit IRC | 15:51 | |
| mdbooth | mriedem: Yeah, task_state is a bit wild west. | 15:51 |
| *** shaohe_feng has quit IRC | 15:51 | |
| mriedem | well, evac/rebuild is a wild west | 15:51 |
| mdbooth | mriedem: Heh, also true. | 15:51 |
| mriedem | because it's munged into a single turducken | 15:51 |
| cdent | evac != evacuate ? | 15:51 |
| *** wwriverrat has joined #openstack-nova | 15:51 | |
| * cdent flips planet | 15:52 | |
| melwitt | I think he meant evacuate uses same task_states as rebuild | 15:52 |
| *** shaohe_feng has joined #openstack-nova | 15:52 | |
| mdbooth | turducken is an awesome word. | 15:52 |
| *** lpetrut has joined #openstack-nova | 15:55 | |
| openstackgerrit | sean mooney proposed openstack/os-vif master: add upper_constraints support https://review.openstack.org/583582 | 15:55 |
| *** links has quit IRC | 15:55 | |
| mriedem | efried: https://bugs.launchpad.net/nova/+bug/1782393 | 15:59 |
| openstack | Launchpad bug 1782393 in OpenStack Compute (nova) "hypervisor local_gb_used still reports usage even with volume-backed instances after fix for bug 1469179" [Medium,Triaged] | 15:59 |
| mriedem | pooja_jadhav: ^ | 15:59 |
| *** pcaruana has joined #openstack-nova | 16:00 | |
| *** shaohe_feng has quit IRC | 16:01 | |
| *** shaohe_feng has joined #openstack-nova | 16:02 | |
| *** lbragstad has quit IRC | 16:02 | |
| *** mgariepy has quit IRC | 16:02 | |
| mriedem | i'm now overwhelmed with the amount of stuff i could be or should be working on | 16:02 |
| *** chyka_ has joined #openstack-nova | 16:03 | |
| *** chyka has quit IRC | 16:06 | |
| mriedem | kashyap: now that you've reminded me, did you ever get a chance to look at the changes to https://review.openstack.org/#/c/560317/ with the extra cpu flags? | 16:07 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Func test for improper cn local DISK_GB reporting https://review.openstack.org/583646 | 16:08 |
| efried | mriedem: Done and done ^ | 16:08 |
| mriedem | ack; need to write something to the ML quick and then i'll get back on it | 16:09 |
| mriedem | and add the move fixes for dan's patch | 16:09 |
| *** med_ has joined #openstack-nova | 16:09 | |
| *** med_ has quit IRC | 16:09 | |
| *** med_ has joined #openstack-nova | 16:09 | |
| *** lei-zh has joined #openstack-nova | 16:09 | |
| *** shaohe_feng has quit IRC | 16:11 | |
| *** shaohe_feng has joined #openstack-nova | 16:13 | |
| *** efried is now known as efried_rollin | 16:13 | |
| *** udesale_ has quit IRC | 16:13 | |
| *** udesale has joined #openstack-nova | 16:13 | |
| *** lei-zh has quit IRC | 16:13 | |
| *** tbachman has quit IRC | 16:14 | |
| mriedem | melwitt: http://lists.openstack.org/pipermail/openstack-dev/2018-July/132329.html | 16:14 |
| *** artom__ has joined #openstack-nova | 16:17 | |
| *** lpetrut has quit IRC | 16:17 | |
| *** udesale has quit IRC | 16:18 | |
| *** artom_ has quit IRC | 16:20 | |
| melwitt | mriedem: ack thanks | 16:21 |
| kashyap | mriedem: /me clicks | 16:21 |
| kashyap | Oh, this one | 16:21 |
| *** Luzi has quit IRC | 16:21 | |
| *** shaohe_feng has quit IRC | 16:22 | |
| kashyap | Yes, I did add a few comments here: https://review.openstack.org/#/c/560317/19/nova/virt/libvirt/driver.py | 16:22 |
| kashyap | I need to take a fresher look at it; was buried in some urgent downstream work the past weeks :-( | 16:22 |
| * kashyap adds it for tomorrow's TODO queue; thanks for the reminder! | 16:23 | |
| *** shaohe_feng has joined #openstack-nova | 16:25 | |
| *** owalsh has quit IRC | 16:26 | |
| *** owalsh has joined #openstack-nova | 16:27 | |
| *** owalsh_ has joined #openstack-nova | 16:29 | |
| mriedem | melwitt: tssurya: are we having a cells meeting today? | 16:30 |
| melwitt | I don't have anything | 16:31 |
| mriedem | reviewing tssurya's series for down cell is obviously top priority, | 16:31 |
| mriedem | and i have https://review.openstack.org/#/c/582412/ for https://review.openstack.org/#/c/582342/ for some up-call bugs | 16:32 |
| mriedem | the latter fixes the bfv cross_az_attach up-call | 16:32 |
| *** shaohe_feng has quit IRC | 16:32 | |
| *** shaohe_feng has joined #openstack-nova | 16:32 | |
| *** tssurya has quit IRC | 16:33 | |
| melwitt | I have the instance group bug (still) which I think could be supported by gibi's tempest tests but also needs a func test for the multi cell case, IIRC? | 16:33 |
| *** owalsh has quit IRC | 16:33 | |
| mriedem | i would have to look in detail again, but i'd rather we have a functional multi-cell test case for that than unit tests | 16:34 |
| mriedem | given the complexity | 16:34 |
| mriedem | and logic issues i found in the unit test | 16:34 |
| melwitt | yeah, I'm going to ditch the unit tests | 16:35 |
| *** AlexeyAbashkin has quit IRC | 16:36 | |
| *** AlexeyAbashkin has joined #openstack-nova | 16:39 | |
| *** sahid has quit IRC | 16:40 | |
| *** shaohe_feng has quit IRC | 16:42 | |
| *** AlexeyAbashkin has quit IRC | 16:43 | |
| *** shaohe_feng has joined #openstack-nova | 16:43 | |
| *** tbachman has joined #openstack-nova | 16:45 | |
| melwitt | I was having difficulty with the func test, need to attack it again | 16:45 |
| sean-k-mooney | does anyone have a doc or pointers on running nova tests under a debugger beyond pdb | 16:48 |
| *** abhishekk has joined #openstack-nova | 16:49 | |
| *** lbragstad has joined #openstack-nova | 16:50 | |
| stephenfin | sean-k-mooney: What do you mean "beyond pdb"? | 16:50 |
| sean-k-mooney | i can get pdb to work but im trying to get visual studio code or pycharm to work | 16:51 |
| stephenfin | Oh, that I've no idea about | 16:52 |
| stephenfin | mdbooth _might_ be able to help you. I hear he uses those things | 16:52 |
| *** shaohe_feng has quit IRC | 16:52 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Add functional tests for numa-aware-vswitches https://review.openstack.org/583288 | 16:52 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: scheduler: Start utilizing RequestSpec.network_metadata https://review.openstack.org/564452 | 16:52 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Assorted cleanups from numa-aware-vswitches series https://review.openstack.org/582651 | 16:52 |
| sean-k-mooney | stephenfin: im trying to step through https://review.openstack.org/#/c/583339 to figure out what it does not trigger the bug | 16:53 |
| *** shaohe_feng has joined #openstack-nova | 16:53 | |
| stephenfin | mriedem, gibi: I know you're busy right now but that's my first attempt at functional tests above for when you do have time ^ | 16:53 |
| stephenfin | sean-k-mooney: I only use pdb or printf debugging so I'm no help, unfortunately | 16:54 |
| sean-k-mooney | stephenfin: no worries i used to have pycharm work for this but new dev setup so all that gone. | 16:55 |
| mriedem | sean-k-mooney: i thought efried_rollin wrote something about that in the dev docs | 16:55 |
| mriedem | https://docs.openstack.org/ironic/queens/contributor/dev-quickstart.html#debugging-unit-tests | 16:56 |
| sean-k-mooney | mriedem: that will partly work. it will give you a pdb console but i was hoping to get it working in an ide. | 16:58 |
| *** do3meli has quit IRC | 16:58 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use placement 1.28 in scheduler report client https://review.openstack.org/583667 | 16:58 |
| mriedem | https://www.youtube.com/watch?v=nksiGORLDZw ? | 16:58 |
| *** licanwei has quit IRC | 16:59 | |
| sean-k-mooney | mriedem: watching... its not a big deal but its nice to have. | 17:00 |
| *** derekh has quit IRC | 17:02 | |
| *** shaohe_feng has quit IRC | 17:03 | |
| *** shaohe_feng has joined #openstack-nova | 17:04 | |
| melwitt | thinking about the rebuild destroy_disks_on_failure thing again, I have the same question as cfriesen_ from the old review, it seems like we shouldn't destroy disks on any rebuild failure, shared or not. if rebuild didn't complete, shouldn't the disk remain? | 17:08 |
| *** munimeha1 has quit IRC | 17:11 | |
| *** tssurya has joined #openstack-nova | 17:12 | |
| *** yamahata has joined #openstack-nova | 17:12 | |
| *** shaohe_feng has quit IRC | 17:13 | |
| *** shaohe_feng has joined #openstack-nova | 17:15 | |
| mdbooth | sean-k-mooney: I tend to use pdb, tbh | 17:19 |
| sean-k-mooney | mdbooth: well apparent the process is now. install pycharm, turn on gevent suport then rightclick the test file you want and select debug | 17:20 |
| sean-k-mooney | not sure how to doe it with vscode but since pycharm is working im happy | 17:20 |
| mdbooth | sean-k-mooney: Interesting. Never even attempted to get a visual debugger working with Nova tests. | 17:21 |
| *** moshele has joined #openstack-nova | 17:21 | |
| sean-k-mooney | mdbooth: if you kickout 1 infinity loop in nova compute it works for the main agent too | 17:21 |
| *** shaohe_feng has quit IRC | 17:23 | |
| *** shaohe_feng has joined #openstack-nova | 17:23 | |
| *** artom__ is now known as artom | 17:23 | |
| * mdbooth -> back shortly | 17:23 | |
| *** dtantsur is now known as dtantsur|afk | 17:26 | |
| mriedem | melwitt: sure, but w/o changes to the spawn method, you need to check if there is an active evacuate migration record for the instance to know if you're rebuilding or evacuating | 17:33 |
| mriedem | since the task_state only tells you doing one of those | 17:33 |
| *** shaohe_feng has quit IRC | 17:33 | |
| mriedem | *you're | 17:33 |
| mriedem | efried_rollin: i've got the move cases handled for dansmith's is_bfv change, and functional test written to show it works, but of course there are a billion unit tests that blow up for unmocked thigns | 17:34 |
| *** mdbooth has quit IRC | 17:34 | |
| *** tesseract has quit IRC | 17:35 | |
| *** shaohe_feng has joined #openstack-nova | 17:36 | |
| melwitt | mriedem: but does it matter whether we're rebuilding vs evacuating? I was thinking destroy_disks_on_failure=False if rebuild or evac, both | 17:36 |
| *** mdbooth has joined #openstack-nova | 17:37 | |
| melwitt | checking the task_state makes sense, just not seeing why we need to differentiate between rebuild and evac (though I do understand your point that checking the migration record is needed if we do need to differentiate) | 17:39 |
| *** shaohe_feng has quit IRC | 17:44 | |
| *** r-daneel has quit IRC | 17:45 | |
| openstackgerrit | Merged openstack/nova master: API: add support to abort queued live migration in microversion 2.65 https://review.openstack.org/573136 | 17:45 |
| *** shaohe_feng has joined #openstack-nova | 17:45 | |
| sean-k-mooney | mriedem: for the antiafinity filter... should this be false based on the comment ? https://github.com/openstack/nova/blob/eb4f65a7951e921b1cd8d05713e144e72f2f254f/nova/scheduler/filters/affinity_filter.py#L99 | 17:45 |
| *** r-daneel has joined #openstack-nova | 17:47 | |
| sean-k-mooney | im steping through your regrssion test in pycharm but i think that is a latent bug. | 17:49 |
| *** abhishekk has quit IRC | 17:51 | |
| sean-k-mooney | also running the regression under the debugger it also very clear we only ever use the uuid of the first instance in the list. | 17:51 |
| *** r-daneel_ has joined #openstack-nova | 17:53 | |
| *** r-daneel has quit IRC | 17:53 | |
| *** r-daneel_ is now known as r-daneel | 17:53 | |
| *** shaohe_feng has quit IRC | 17:54 | |
| *** shaohe_feng has joined #openstack-nova | 17:54 | |
| *** jchhatbar has quit IRC | 17:55 | |
| *** dklyle has joined #openstack-nova | 17:55 | |
| *** med_ has quit IRC | 17:56 | |
| *** dklyle has quit IRC | 18:01 | |
| *** artom_ has joined #openstack-nova | 18:01 | |
| *** artom has quit IRC | 18:01 | |
| *** mdnadeem has joined #openstack-nova | 18:03 | |
| *** shaohe_feng has quit IRC | 18:04 | |
| openstackgerrit | Merged openstack/nova master: Address nit in 79dac41fee178dabb547f4d7bc10609630767131 https://review.openstack.org/583212 | 18:05 |
| *** shaohe_feng has joined #openstack-nova | 18:05 | |
| mriedem | melwitt: ok, idk, i'm really half assing trying to understand that problem atm | 18:07 |
| * melwitt nods | 18:08 | |
| mriedem | sean-k-mooney: it's a double negative | 18:08 |
| mriedem | "must not return the source as a non-possible destination" | 18:08 |
| *** pcaruana has quit IRC | 18:08 | |
| mriedem | meaning, must return the source as a possible destination | 18:08 |
| mriedem | for resize to same host | 18:08 |
| sean-k-mooney | right. so we want to allow resize to same host | 18:09 |
| mriedem | yes | 18:09 |
| sean-k-mooney | ? | 18:09 |
| mriedem | which is what that was added for | 18:09 |
| sean-k-mooney | ok | 18:09 |
| mriedem | and there is a related functional test somewhere for it | 18:09 |
| sean-k-mooney | well in the multi create case this cases the second instance to pass on the first host because we use the same request spec for all instances and there for the same uuid | 18:10 |
| *** flwang1 has joined #openstack-nova | 18:10 | |
| sean-k-mooney | so for both instance both hosts pass | 18:11 |
| flwang1 | mriedem: hello, i have a question about this nova/bin/privsep-helper | 18:11 |
| flwang1 | mriedem: could you please help me understand if it's still used? | 18:11 |
| flwang1 | thanks | 18:11 |
| sean-k-mooney | flwang1: as far as i am aware teh privsep-helper is till used to spawn the privesep deamon | 18:13 |
| flwang1 | can anybody help me understand where can I get the privsep-helper? we're using an old version of Nova, and recently after a re packaging, we can't find the file | 18:13 |
| *** itlinux has joined #openstack-nova | 18:13 | |
| flwang1 | sean-k-mooney: thanks, do you know how is it installed? by os-brick? | 18:14 |
| *** shaohe_feng has quit IRC | 18:14 | |
| mriedem | sean-k-mooney: yes that's the bug | 18:15 |
| *** shaohe_feng has joined #openstack-nova | 18:15 | |
| openstackgerrit | Hongbin Lu proposed openstack/nova master: [DNM] Test neutron change https://review.openstack.org/583687 | 18:16 |
| mriedem | sean-k-mooney: but i'm not sure why my functional test wouldn't recreate that issue if the first weighed host is going to be picked for both instances | 18:16 |
| *** moshele has quit IRC | 18:16 | |
| mriedem | flwang1: i'm not sure what that is, but it sounds like something you'd have to find from the source rpm in the old package | 18:17 |
| sean-k-mooney | mriedem: in the debugger selected_hosts==set('host2','host1') i need to step through the weighing to figure out why however | 18:17 |
| flwang1 | mriedem: yep, i know, i'm trying to figure out where does it come from so that I can include that debian pkg or build it with source code | 18:18 |
| mriedem | flwang1: http://git.openstack.org/cgit/openstack/oslo.privsep/tree/setup.cfg#n40 | 18:19 |
| mriedem | it's a binary in oslo.privsep | 18:19 |
| sean-k-mooney | flwang1: its created by installing oslo privsep https://github.com/openstack/oslo.privsep/blob/master/setup.cfg#L40 | 18:19 |
| mriedem | I WIN | 18:19 |
| sean-k-mooney | :) | 18:19 |
| flwang1 | sean-k-mooney: mriedem: hah, many thanks, folks | 18:19 |
| flwang1 | mriedem: hope you're doing well, man | 18:19 |
| flwang1 | time flys | 18:20 |
| *** sambetts is now known as sambetts|afk | 18:21 | |
| mriedem | huh, wtf, why does unshelve even call populate_retry - unshelve failures aren't rescheduled.... | 18:22 |
| mriedem | flwang1: just fixing bugs from grizzly | 18:22 |
| flwang1 | mriedem: yep | 18:22 |
| flwang1 | sean-k-mooney: mriedem: damn it, we're using a nova version at that moment, there is no oslo.privsep in the world :( | 18:23 |
| *** shaohe_feng has quit IRC | 18:25 | |
| *** cdent has quit IRC | 18:25 | |
| sean-k-mooney | flwang1: in that case it should still be using rootwrap | 18:25 |
| sean-k-mooney | flwang1: so you should not need privsep-helper | 18:25 |
| flwang1 | oslo rootwrap? | 18:26 |
| flwang1 | or the rootwrap of nova itself | 18:26 |
| *** mdnadeem has quit IRC | 18:26 | |
| sean-k-mooney | flwang1: also that means your running pre kilo? | 18:26 |
| sean-k-mooney | flwang1: good question | 18:26 |
| flwang1 | sean-k-mooney: a little bit better than kilo | 18:26 |
| *** dklyle has joined #openstack-nova | 18:27 | |
| *** shaohe_feng has joined #openstack-nova | 18:28 | |
| *** lpetrut has joined #openstack-nova | 18:28 | |
| sean-k-mooney | flwang1: anyway back to your question nova's rootwapper delegates to oslo | 18:30 |
| *** vivsoni__ has joined #openstack-nova | 18:30 | |
| flwang1 | but in oslo.rootwrap, i still can't find the privsep-helper command :( | 18:30 |
| flwang1 | sean-k-mooney: and you're right, i can see nova-rootwrap = oslo_rootwrap.cmd:main | 18:31 |
| *** itlinux has quit IRC | 18:31 | |
| sean-k-mooney | am what version of nova are you trying to deploy? | 18:31 |
| sean-k-mooney | flwang1: or what error are your actully hitting? you mention os-brick earlier is that throwing an error for you | 18:33 |
| *** itlinux has joined #openstack-nova | 18:34 | |
| flwang1 | sean-k-mooney: http://paste.openstack.org/show/726222/ | 18:35 |
| *** shaohe_feng has quit IRC | 18:35 | |
| flwang1 | in our debian install code, we have something like above | 18:35 |
| *** shaohe_feng has joined #openstack-nova | 18:36 | |
| flwang1 | and it failed because can't find the privsep-helper under /opt/cat/openstack/nova/bin/ | 18:36 |
| sean-k-mooney | right well for a start line 9 should never execute | 18:37 |
| flwang1 | sean-k-mooney: sorry, what does that mean? | 18:38 |
| sean-k-mooney | line 8 you check if /usr/bin/privsep-helper does not exisits then on line 9 if that is true you check is /usr/bin/privsep-helper a file but line 8 asserts it does not exisit | 18:39 |
| sean-k-mooney | flwang1: so rm /usr/bin/privsep-helper can never run | 18:41 |
| flwang1 | sean-k-mooney: yep, so hence why line 10 will try to copy it from /opt/cat/openstack/nova/bin/ to /usr/bin | 18:42 |
| flwang1 | and failed | 18:42 |
| sean-k-mooney | well as written it will always try to create the symlink. line 9 can be removed without changeing the behavior. | 18:43 |
| sean-k-mooney | but back to your issue | 18:43 |
| *** shaohe_feng has quit IRC | 18:45 | |
| *** med_ has joined #openstack-nova | 18:45 | |
| sean-k-mooney | nova does not provide privsep-helper ever. it depends on oslo.prevsep to do that. so the nova-compute deb package should depend on oslo.prevsep and the post install of oslo.privsep should copy it to /usr/bin/privsep-helper | 18:45 |
| flwang1 | but depend on the new oslo.privsep for a old nova version is not safe, right? | 18:47 |
| *** shaohe_feng has joined #openstack-nova | 18:47 | |
| flwang1 | because it may cause some version conflicts | 18:47 |
| sean-k-mooney | old nova did not use privsep it used rootwrapper instead. | 18:47 |
| flwang1 | ok, i can see your point now. | 18:48 |
| sean-k-mooney | so unless you have backported privsep support then your nova does not need the privsep-helper | 18:48 |
| flwang1 | so does that mean, if the nova version doesn't need oslo.privsep, then it probably mean we don't need this privsep-helper? | 18:48 |
| sean-k-mooney | yep unless it uses a lib like os-bricks or os-vif that needit but its not the nova package that should be installing it in that case | 18:49 |
| sean-k-mooney | so basically if the nova you are using dose not use privsep you can proably delete that postint scripts or at least that part of it | 18:50 |
| flwang1 | sean-k-mooney: thanks a lot, that makes my day | 18:51 |
| flwang1 | now it's 6:51AM here :) | 18:52 |
| sean-k-mooney | almost 8PM here. you should proably get some sleep either that or you are an early riser | 18:52 |
| *** _ix has joined #openstack-nova | 18:53 | |
| *** harlowja has joined #openstack-nova | 18:54 | |
| flwang1 | sean-k-mooney: many thanks, mate. I will probably say hi if I can see you at Berlin summit ;) | 18:54 |
| *** moshele has joined #openstack-nova | 18:55 | |
| *** shaohe_feng has quit IRC | 18:55 | |
| *** shaohe_feng has joined #openstack-nova | 18:57 | |
| *** priteau has quit IRC | 18:58 | |
| *** maciejjozefczyk has joined #openstack-nova | 18:59 | |
| *** rmart04 has joined #openstack-nova | 18:59 | |
| *** rmart04 has quit IRC | 19:00 | |
| *** owalsh_ is now known as owalsh | 19:03 | |
| *** shaohe_feng has quit IRC | 19:06 | |
| *** shaohe_feng has joined #openstack-nova | 19:06 | |
| *** eharney has quit IRC | 19:07 | |
| *** shaohe_feng has quit IRC | 19:16 | |
| *** awaugama has quit IRC | 19:16 | |
| *** shaohe_feng has joined #openstack-nova | 19:17 | |
| sean-k-mooney | mriedem: im still stepping throught the regression test by the way. we do actully allocate both instance to the first host after weighing | 19:19 |
| sean-k-mooney | mriedem: so before we call self._ensure_sufficent_hosts here https://github.com/openstack/nova/blob/c54428cfe1aab8ef9c83df2f9a2718cae864f862/nova/scheduler/filter_scheduler.py#L245 both instance are scheduled to host1 | 19:20 |
| *** vivsoni__ has quit IRC | 19:21 | |
| *** dklyle has quit IRC | 19:23 | |
| *** felipemonteiro_ has joined #openstack-nova | 19:24 | |
| *** felipemonteiro_ is now known as felipemonteiro | 19:24 | |
| *** jlk has joined #openstack-nova | 19:25 | |
| *** shaohe_feng has quit IRC | 19:26 | |
| *** shaohe_feng has joined #openstack-nova | 19:28 | |
| jlk | Hey all. I'd like some help confirming my investigation. I'm digging into how Nova does a hard reboot of an instance (libvirt, kvm), and from what I can tell, it looks like it relies on a libvirt API call to 'destroy()', which from my reading will by default ASK the OS to shutdown cleanly, with a "reasonable" timeout before SIGKILL. | 19:30 |
| *** felipemonteiro_ has joined #openstack-nova | 19:30 | |
| jlk | It doesn't look like there is a way w/ Nova <-> libvirt to skip the SIGTERM and just go directly to SIGKILL | 19:30 |
| jlk | nor can I tell what the "reasonable timeout" is. | 19:30 |
| *** felipemonteiro has quit IRC | 19:33 | |
| *** shaohe_feng has quit IRC | 19:36 | |
| *** maciejjozefczyk has quit IRC | 19:37 | |
| *** shaohe_feng has joined #openstack-nova | 19:37 | |
| *** mgariepy has joined #openstack-nova | 19:37 | |
| *** shaohe_feng has quit IRC | 19:47 | |
| *** dpawlik has quit IRC | 19:47 | |
| *** arvindn05 has quit IRC | 19:48 | |
| *** shaohe_feng has joined #openstack-nova | 19:48 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Avoid requesting DISK_GB allocation for root_gb on BFV instances https://review.openstack.org/580720 | 19:48 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Heal RequestSpec.is_bfv for legacy instances during moves https://review.openstack.org/583715 | 19:49 |
| *** efried_rollin is now known as efried | 19:49 | |
| efried | mriedem: https://review.openstack.org/#/c/580720/ needed a UT test - that 'or False' thing apparently matters in a weird way that other non-booleans don't. | 19:49 |
| *** felipemonteiro_ has quit IRC | 19:50 | |
| efried | UT fix* | 19:50 |
| *** felipemonteiro_ has joined #openstack-nova | 19:50 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Func test for improper cn local DISK_GB reporting https://review.openstack.org/583646 | 19:50 |
| *** yamahata has quit IRC | 19:50 | |
| *** arvindn05 has joined #openstack-nova | 19:52 | |
| mriedem | by lucifers beard | 19:52 |
| mriedem | oh yeah root can be None | 19:54 |
| mriedem | derp | 19:54 |
| *** flwang1 has quit IRC | 19:54 | |
| sean-k-mooney | mriedem: is there a way in the funtional tests to override what filters are enabled or turn off retries in the conductor | 19:56 |
| *** shaohe_feng has quit IRC | 19:57 | |
| sean-k-mooney | mriedem: actully i think i have found the filters at least | 19:57 |
| *** shaohe_feng has joined #openstack-nova | 19:58 | |
| mriedem | yes you can configure the filters prior to starting the scheduler service | 19:58 |
| sean-k-mooney | mriedem: im pretty sure you regression test is passing becasue a retry is happening and the second instance it being schduled to an alternate host | 19:58 |
| efried | mriedem: Yeah, but why (oh, why) is it okay to assign a BooleanField other non-boolean values but not None? | 19:59 |
| sean-k-mooney | efried: because python. or in this case proably because ovo | 19:59 |
| efried | Totally ovo. Python loves treating None as False. | 20:00 |
| mriedem | sean-k-mooney: hmm, yeah, could also just set CONF.scheduler.max_attempts=0 | 20:02 |
| mriedem | that would disable retries | 20:03 |
| mriedem | *reschedules | 20:03 |
| sean-k-mooney | ill try both | 20:03 |
| mriedem | also, even if we did reschedule, the affinity check in compute should kick that out b/c the other instance is already on that host | 20:03 |
| sean-k-mooney | thats the thing. the schedule is selecting host1 for both because of the weigher the the scond instance appears to be reschduled to host2 and you end up with anti affinity | 20:05 |
| *** pchavva has quit IRC | 20:05 | |
| mriedem | OH | 20:06 |
| mriedem | ok yeah then just set max_attempts=0 and that should kill that | 20:06 |
| mriedem | we don't get the reschedule in devstack b/c we disabled the up-calls to the api db for that late check in the compute | 20:06 |
| mriedem | that's something else you could do is disable the late-check up-call to mimic dsvm setup | 20:07 |
| mriedem | that might be better | 20:07 |
| mriedem | because honestly we shouldn't get alternatives for either of these instances in a 2 host setup | 20:07 |
| mriedem | since there would be no alternatives | 20:07 |
| *** shaohe_feng has quit IRC | 20:07 | |
| sean-k-mooney | setting the max seams quicker to test. if that cases it to fail i can try disableing the up call | 20:07 |
| *** shaohe_feng has joined #openstack-nova | 20:08 | |
| *** itlinux has quit IRC | 20:09 | |
| jlk | oh wow, hrm. Testing out domain.destroy() seems to NOT send a soft shutdown signal. INTERESTING. | 20:11 |
| sean-k-mooney | mriedem: yep adding self.flags(max_attempts=0, group='scheduler') makes the regression fail | 20:12 |
| *** artom_ has quit IRC | 20:13 | |
| mriedem | sean-k-mooney: sweet, can you try with disabling the up-calls? | 20:16 |
| mriedem | sean-k-mooney: https://docs.openstack.org/nova/latest/configuration/config.html#workarounds.disable_group_policy_check_upcall | 20:16 |
| sean-k-mooney | actully it failed because max attempts must be >= 1 | 20:16 |
| sean-k-mooney | testing again | 20:16 |
| mriedem | then if that introduces the regression you could fix up the test and push that up, | 20:16 |
| mriedem | then i could rebase the bug fix on top of it | 20:17 |
| *** shaohe_feng has quit IRC | 20:17 | |
| *** shaohe_feng has joined #openstack-nova | 20:18 | |
| *** munimeha1 has joined #openstack-nova | 20:18 | |
| sean-k-mooney | ok with it set to max_attempts=1 i get Build of instance cbf4b2dd-8301-4d62-97f5-d74b00b21d64 was re-scheduled: Anti-affinity instance group policy was violated. | 20:20 |
| sean-k-mooney | so ya ill try disableing that check now | 20:20 |
| mriedem | yup as expected | 20:20 |
| *** yamahata has joined #openstack-nova | 20:21 | |
| mriedem | cfriesen_: re: your reply in the ML thread about this bug, i think we can do both fixes actually for this same thing - because we ran into it this time in the antiaffinity filter, but the fact we pick the first request spec and send that to the scheduler with a list of other instance uuids and send that reqspec into each filter with what could be the wrong instance_uuid is still a thing worth fixing, | 20:21 |
| sean-k-mooney | what config group do i set disable_group_policy_check_upcall in? is workarounds its own group | 20:21 |
| mriedem | and num_instances being set per-scheduler request is something else we could fix | 20:22 |
| mriedem | sean-k-mooney: [workarounds] | 20:22 |
| mriedem | yes | 20:22 |
| mriedem | honestly num_instances shouldn't even be needed anymore, but it predates sending the list of instance uuids to select_destinations | 20:22 |
| mriedem | we kind of sorely need to drop scheduler rpcapi 4.x compatibility and move to 5.0 | 20:22 |
| mriedem | so we can drop a lot of this garbage compat code in the scheduler | 20:23 |
| sean-k-mooney | is that something we can do in stien. im guessing we need 4.x to live as long as cellsv1 or something else is still around? | 20:24 |
| mriedem | it's usually sometihng we do before we cut a release, | 20:24 |
| mriedem | we add a compat shim and then drop the shim once we are in n+1 master | 20:24 |
| mriedem | dansmith is the one that's always done it | 20:25 |
| sean-k-mooney | ah. ok so with disableing the upcall i new get to your assertion and it fails with 2!=1 | 20:25 |
| mriedem | sean-k-mooney: awesome | 20:25 |
| mriedem | want to fix the assertion and add a FIXME note in there about cleaning up the assertion once the bug is fixed? | 20:25 |
| mriedem | probably need to update the docstring in the test also | 20:26 |
| sean-k-mooney | sure will do. | 20:26 |
| sean-k-mooney | this would have been a nightmare to find without a debugger | 20:26 |
| mriedem | if i would have been thinking clearly about the devstack env, i would have eventually realized that we might be hitting a reschedule which devstack doesn't | 20:27 |
| mriedem | but, brain is mush | 20:27 |
| mriedem | i would realize it at 3am most likely | 20:27 |
| *** shaohe_feng has quit IRC | 20:28 | |
| *** shaohe_feng has joined #openstack-nova | 20:28 | |
| *** r-daneel has quit IRC | 20:31 | |
| *** jmlowe has quit IRC | 20:36 | |
| *** itlinux has joined #openstack-nova | 20:37 | |
| *** shaohe_feng has quit IRC | 20:38 | |
| *** shaohe_feng has joined #openstack-nova | 20:39 | |
| *** dklyle has joined #openstack-nova | 20:39 | |
| *** r-daneel has joined #openstack-nova | 20:40 | |
| *** itlinux has quit IRC | 20:42 | |
| openstackgerrit | sean mooney proposed openstack/nova master: Add regression test for bug 1781710 https://review.openstack.org/583339 | 20:44 |
| openstack | bug 1781710 in OpenStack Compute (nova) "ServersOnMultiNodesTest.test_create_server_with_scheduler_hint_group_anti_affinity failing with "Servers are on the same host"" [High,Fix released] https://launchpad.net/bugs/1781710 - Assigned to Matt Riedemann (mriedem) | 20:44 |
| sean-k-mooney | by the way have people hit this. | 20:47 |
| sean-k-mooney | RuntimeError: On Python 3, zvm sdk supports to Python 3.5 | 20:47 |
| *** felipemonteiro__ has joined #openstack-nova | 20:48 | |
| sean-k-mooney | im currently on centos 7.5 and i install python 3.6. we dont support 3.4 upstream right | 20:48 |
| *** shaohe_feng has quit IRC | 20:48 | |
| *** lpetrut has quit IRC | 20:48 | |
| sean-k-mooney | centos only ship 3.4 and 3.6 in the package manager | 20:49 |
| *** shaohe_feng has joined #openstack-nova | 20:49 | |
| *** rtjure has joined #openstack-nova | 20:50 | |
| sean-k-mooney | efried: do you know what are the changes of getting 3.6 support in the zvm sdk | 20:50 |
| efried | sean-k-mooney: You mean so you can tox with the effin thing? | 20:50 |
| sean-k-mooney | yes | 20:51 |
| sean-k-mooney | right now i cand run pep8 or any of the python3 based envs | 20:51 |
| sean-k-mooney | unless in install 3.5 that is or downgrade to 3.4 | 20:51 |
| efried | sean-k-mooney: Sorry, no idea. I got around it by sudo pip installing zVMCloudConnector on my system (not the venv) | 20:51 |
| efried | sean-k-mooney: Oh, maybe 3.6 support is a different issue. | 20:52 |
| *** felipemonteiro_ has quit IRC | 20:52 | |
| sean-k-mooney | there is a check for the python 3 version. i symlinked python3 to 3.6 inst normally 3.4 | 20:52 |
| sean-k-mooney | on centos anyway. | 20:52 |
| sean-k-mooney | i can just swap the symlink back to 3.4 for now | 20:53 |
| *** munimeha1 has quit IRC | 20:54 | |
| sean-k-mooney | are the gates still running ubuntu 16.04? | 20:55 |
| sean-k-mooney | ubuntu 18.04 also ships 3.6 by defualt i think now | 20:55 |
| *** dpawlik has joined #openstack-nova | 20:56 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Heal RequestSpec.is_bfv for legacy instances during moves https://review.openstack.org/583715 | 20:57 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix wonky reqspec handling in conductor.unshelve_instance https://review.openstack.org/583739 | 20:57 |
| mriedem | we're still testing against 16.04 in the gate for rocky so that's what i use | 20:57 |
| *** rtjure has quit IRC | 20:57 | |
| sean-k-mooney | well untill the zvm sdk is drop as a depency or upgraded to support 3.6 we wont be able to move nova gate jobs to 18.04 | 20:58 |
| *** flwang1 has joined #openstack-nova | 20:58 | |
| sean-k-mooney | at least given the python3 first goal | 20:58 |
| *** shaohe_feng has quit IRC | 20:58 | |
| mriedem | alert the TC! | 20:58 |
| mriedem | that's this thing right? https://github.com/mfcloud/python-zvm-sdk/blob/master/setup.py | 20:59 |
| efried | sean-k-mooney: I wish dansmith or mikal or someone had warned us about problems like this. http://lists.openstack.org/pipermail/openstack-dev/2018-April/129294.html | 20:59 |
| *** shaohe_feng has joined #openstack-nova | 20:59 | |
| mriedem | https://github.com/mfcloud/python-zvm-sdk/blob/master/setup.py#L39 | 20:59 |
| *** rtjure has joined #openstack-nova | 21:00 | |
| efried | http://lists.openstack.org/pipermail/openstack-dev/2018-April/129350.html | 21:00 |
| *** eharney has joined #openstack-nova | 21:00 | |
| mriedem | i guess this is where we'd report a bug that it doesn't work with 3.6 https://bugs.launchpad.net/python-zvm-sdk/+bugs | 21:00 |
| *** dpawlik has quit IRC | 21:00 | |
| mriedem | and then harass the shit out of jichenjc to get that fixed and released | 21:01 |
| efried | support 3.7 while we're at it, so we don't get behind immediately. | 21:01 |
| sean-k-mooney | mriedem: ya the weird thing is the error message seams to indicate it support 3.5 but only declares 2.7 support in that file | 21:01 |
| mriedem | yeah it definitely supports 3.5 https://github.com/mfcloud/python-zvm-sdk/blob/master/tox.ini#L27 | 21:01 |
| mriedem | what blows up with 3.6? | 21:02 |
| mriedem | it just doesn't install/ | 21:02 |
| mriedem | ? | 21:02 |
| sean-k-mooney | yep tox fails while pip installing it | 21:02 |
| mriedem | what's the error? | 21:02 |
| sean-k-mooney | one sec | 21:02 |
| *** tidwellr has quit IRC | 21:02 | |
| sean-k-mooney | http://paste.openstack.org/show/726237/ | 21:04 |
| mriedem | bingo https://github.com/mfcloud/python-zvm-sdk/blob/master/zvmsdk/version.py#L28 | 21:06 |
| *** yamahata has quit IRC | 21:06 | |
| sean-k-mooney | you know if you replace that 5 with a 6 it will proably work | 21:07 |
| *** felipemonteiro__ has quit IRC | 21:07 | |
| mriedem | try it | 21:07 |
| *** felipemonteiro__ has joined #openstack-nova | 21:07 | |
| mriedem | or just try doing that and add a tox36 target and run it's unit tests | 21:07 |
| mriedem | this could be your first (and only ever) zvmcloudconnector patch :) | 21:08 |
| *** shaohe_feng has quit IRC | 21:09 | |
| sean-k-mooney | some might say its one too many. | 21:09 |
| *** shaohe_feng has joined #openstack-nova | 21:09 | |
| sean-k-mooney | that said i just downloaded and ran its py27 unitests and one of them still failed | 21:09 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Heal RequestSpec.is_bfv for legacy instances during moves https://review.openstack.org/583715 | 21:09 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix wonky reqspec handling in conductor.unshelve_instance https://review.openstack.org/583739 | 21:09 |
| mriedem | let me state one more time, for the record, for this week, that i hate all of the request spec compat code in conductor | 21:10 |
| sean-k-mooney | mriedem: duely noted. | 21:11 |
| sean-k-mooney | i just found another bug in there test code... i am going to go down a rabbit whole with this i can tell | 21:11 |
| _ix | Afternoon folks. I realize this channel is less support oriented, but there's no response in #openstack. I'm having an issue in our pike deployment with resource allocations still existing on one compute, but the vm itself resides on another. It's not reflecting additional usage on the original VM, but the scheduler still won't allow me to instantiate there. | 21:12 |
| _ix | How can I clear the allocation from the original hypervisor? | 21:12 |
| *** moshele has quit IRC | 21:13 | |
| _ix | > There are allocations remaining against the source host that might need to be removed | 21:13 |
| sean-k-mooney | _ix: did you migrate or resize the vm at some point? if you have not completed the migrations then it will have both allocations if i rembere correctly | 21:14 |
| _ix | I did, but the migration failed. | 21:14 |
| _ix | I believe it was the case where a live migration was ongoing, and the targeted compute's services were unavailable or became unavailable. | 21:15 |
| _ix | From a user's perspective, there doesn't seem to be a way to back out of the migration. | 21:15 |
| sean-k-mooney | _ix so the vm migrated but the migration timeded out becase the node became unableale? | 21:15 |
| _ix | The vm stayed on the source but received allocation on the target. | 21:16 |
| sean-k-mooney | _ix: well migration is an admin action so "end users" should not be aware of them in general | 21:16 |
| _ix | That's a fair point. | 21:16 |
| sean-k-mooney | mriedem: jaypipes efried do you know how to clean up the stale allocation? | 21:17 |
| _ix | How can I force the issue? I've manually added allocations records to reflect reality in the nova_api.allocations table and marked the failed migration as deleted in nova.migrations. | 21:17 |
| efried | sean-k-mooney: I haven't been following the conversation. You want to manually remove an allocation from placement? | 21:17 |
| _ix | Yes please. | 21:18 |
| sean-k-mooney | efried: live migration failed but allocation on dest remain | 21:18 |
| efried | mriedem is the osc-placement expert. I can only tell you how to do it programmatically :) | 21:18 |
| *** shaohe_feng has quit IRC | 21:19 | |
| _ix | It's an open bug, but it's such an edge case, I totally understand why this might lead to something out-of-state: https://bugs.launchpad.net/nova/+bug/1753676 | 21:19 |
| openstack | Launchpad bug 1753676 in OpenStack Compute (nova) "Live migration not working as Expected when Restarting nova-compute service while migration from source node" [Undecided,Confirmed] | 21:19 |
| *** shaohe_feng has joined #openstack-nova | 21:20 | |
| *** moshele has joined #openstack-nova | 21:20 | |
| _ix | I'm pleased to report live migrations tend to work quite well and fast, I might add. | 21:20 |
| sean-k-mooney | _ix: ya migration in pike was kind of buggy. dansmith was working on fixing that i think | 21:20 |
| mriedem | yeah osc-placement | 21:20 |
| mriedem | https://docs.openstack.org/osc-placement/latest/index.html | 21:21 |
| mriedem | https://docs.openstack.org/osc-placement/latest/cli/index.html#resource-provider-allocation-set specifically | 21:21 |
| _ix | mriedem: Thanks! | 21:21 |
| mriedem | if the consumer (instance in this case) shows having allocations against multiple resource providers (compute nodes), then determine the one you want to keep and the one you want to remove, and update the allocations | 21:21 |
| mriedem | you can view the allocations with https://docs.openstack.org/osc-placement/latest/cli/index.html#resource-provider-aggregate-list | 21:22 |
| mriedem | oops | 21:22 |
| sean-k-mooney | mriedem: would the consumer not be the migration in this case | 21:22 |
| mriedem | https://docs.openstack.org/osc-placement/latest/cli/index.html#resource-provider-allocation-show | 21:22 |
| mriedem | sean-k-mooney: not in pike | 21:22 |
| sean-k-mooney | oh ok | 21:22 |
| mriedem | that change happened in queens | 21:22 |
| mriedem | https://specs.openstack.org/openstack/nova-specs/specs/queens/implemented/migration-allocations.html | 21:22 |
| mriedem | also, pike GA had a few bugs with cleaning up allocations during failures, | 21:22 |
| mriedem | which we fixed in stable point releases of pike | 21:23 |
| mriedem | _ix: which version of pike are you running? latest stable i hope | 21:23 |
| mriedem | https://docs.openstack.org/releasenotes/nova/pike.html | 21:23 |
| _ix | mriedem: Latest stable, I believe. | 21:24 |
| _ix | I must be missing a client. What enables the placement hierarchy? | 21:24 |
| mriedem | _ix: https://pypi.org/project/osc-placement/ | 21:25 |
| mriedem | it's an osc plugin | 21:25 |
| _ix | I wish I were more familiar. I'll do some reading. Thanks very much! | 21:25 |
| mriedem | np | 21:27 |
| mriedem | placement is a new big required thing to a lot of people in pike so it's ok to ask operational questions here | 21:28 |
| *** felipemonteiro__ has quit IRC | 21:28 | |
| *** shaohe_feng has quit IRC | 21:29 | |
| *** shaohe_feng has joined #openstack-nova | 21:30 | |
| *** tssurya has quit IRC | 21:34 | |
| *** yamahata has joined #openstack-nova | 21:35 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add regression test for bug 1781710 https://review.openstack.org/583339 | 21:36 |
| openstack | bug 1781710 in OpenStack Compute (nova) "ServersOnMultiNodesTest.test_create_server_with_scheduler_hint_group_anti_affinity failing with "Servers are on the same host"" [High,Fix released] https://launchpad.net/bugs/1781710 - Assigned to Matt Riedemann (mriedem) | 21:36 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Update RequestSpec.instance_uuid during scheduling https://review.openstack.org/583347 | 21:36 |
| mriedem | jaypipes: melwitt: sean-k-mooney: now rebased on top of the regression test which also helped me find some other places i needed to touch up | 21:37 |
| _ix | So... if a consumer is provided for by.... four providers, that probably isn't so good. | 21:38 |
| mriedem | _ix: nope | 21:39 |
| *** shaohe_feng has quit IRC | 21:39 | |
| mriedem | in case it's not clear, you can correlate the provider id to hypervisor ids | 21:40 |
| mriedem | using nova hypervisor-list | 21:40 |
| sean-k-mooney | efried: mriedem pull request sent https://github.com/mfcloud/python-zvm-sdk/pull/7 | 21:40 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Update RequestSpec.instance_uuid during scheduling https://review.openstack.org/583347 | 21:40 |
| mriedem | sean-k-mooney: update the setup.py too | 21:41 |
| *** antosh has quit IRC | 21:41 | |
| sean-k-mooney | mriedem: oh ya will do | 21:42 |
| *** dklyle has quit IRC | 21:42 | |
| *** shaohe_feng has joined #openstack-nova | 21:43 | |
| _ix | I've probably spent too much time looking the through the database entries trying to figure out how this was supposed to work. Perhaps I don't see the forest for the trees, but I would guess that the consumers aren't only instances. Is that the case? | 21:44 |
| *** dklyle has joined #openstack-nova | 21:45 | |
| sean-k-mooney | mriedem: i just did a force push to my forked copy and the pull request updated. im not sure if i like this or now but done | 21:48 |
| *** shaohe_feng has quit IRC | 21:50 | |
| jaypipes | mriedem: ack. ++ | 21:50 |
| *** shaohe_feng has joined #openstack-nova | 21:52 | |
| *** david-lyle has joined #openstack-nova | 21:53 | |
| melwitt | ack | 21:53 |
| *** dklyle has quit IRC | 21:54 | |
| *** edmondsw has quit IRC | 21:54 | |
| efried | _ix: In pike they are; in more recent releases we introduced a special consumer to represent migrations so we didn't have to double up instance allocs like you're seeing. | 21:55 |
| *** _ix has quit IRC | 21:58 | |
| *** shaohe_feng has quit IRC | 22:00 | |
| *** moshele has quit IRC | 22:00 | |
| *** shaohe_feng has joined #openstack-nova | 22:02 | |
| *** linkmark has joined #openstack-nova | 22:03 | |
| openstackgerrit | sean mooney proposed openstack/os-vif master: add upper_constraints support https://review.openstack.org/583582 | 22:04 |
| sean-k-mooney | efried: addressed you comments and changed to CONSTRAINTS_OPT | 22:09 |
| *** shaohe_feng has quit IRC | 22:10 | |
| *** shaohe_feng has joined #openstack-nova | 22:11 | |
| *** _ix has joined #openstack-nova | 22:11 | |
| *** MrWatson has quit IRC | 22:12 | |
| sean-k-mooney | anyway its 11 so i should proably log off | 22:12 |
| *** NostawRm has joined #openstack-nova | 22:12 | |
| sean-k-mooney | i have missed working upstream | 22:12 |
| sean-k-mooney | efried: stephenfin melwitt mriedem before i go. if we merge https://review.openstack.org/#/c/583582 today ill update https://review.openstack.org/#/c/583319 in the morning and os-vif will be updated for rocky. if their are any other change ye want ill adress them in the morning | 22:17 |
| *** mlavalle has quit IRC | 22:20 | |
| *** shaohe_feng has quit IRC | 22:20 | |
| *** shaohe_feng has joined #openstack-nova | 22:22 | |
| *** med_ has quit IRC | 22:24 | |
| *** jmlowe has joined #openstack-nova | 22:24 | |
| *** jmlowe has quit IRC | 22:25 | |
| *** gongysh has joined #openstack-nova | 22:25 | |
| *** jmlowe has joined #openstack-nova | 22:25 | |
| *** wolverineav has quit IRC | 22:28 | |
| *** lbragstad has quit IRC | 22:29 | |
| *** shaohe_feng has quit IRC | 22:31 | |
| *** david-lyle has quit IRC | 22:31 | |
| *** shaohe_feng has joined #openstack-nova | 22:31 | |
| *** hongbin has quit IRC | 22:32 | |
| *** rcernin has joined #openstack-nova | 22:32 | |
| *** shaohe_feng has quit IRC | 22:41 | |
| *** Guest61292 has quit IRC | 22:42 | |
| *** shaohe_feng has joined #openstack-nova | 22:44 | |
| *** imacdonn has quit IRC | 22:44 | |
| *** imacdonn has joined #openstack-nova | 22:45 | |
| *** brault has quit IRC | 22:46 | |
| melwitt | sean-k-mooney: ack | 22:50 |
| *** shaohe_feng has quit IRC | 22:51 | |
| *** shaohe_feng has joined #openstack-nova | 22:52 | |
| *** dpawlik has joined #openstack-nova | 22:57 | |
| *** jmlowe has quit IRC | 22:58 | |
| *** jmlowe has joined #openstack-nova | 22:58 | |
| *** wolverineav has joined #openstack-nova | 22:59 | |
| *** owalsh_ has joined #openstack-nova | 23:00 | |
| *** owalsh has quit IRC | 23:01 | |
| *** chyka_ has quit IRC | 23:01 | |
| *** dpawlik has quit IRC | 23:01 | |
| *** shaohe_feng has quit IRC | 23:01 | |
| *** shaohe_feng has joined #openstack-nova | 23:02 | |
| *** wolverineav has quit IRC | 23:03 | |
| *** itlinux has joined #openstack-nova | 23:04 | |
| *** owalsh_ is now known as owalsh | 23:04 | |
| *** jmlowe has quit IRC | 23:05 | |
| *** dklyle has joined #openstack-nova | 23:09 | |
| *** shaohe_feng has quit IRC | 23:12 | |
| *** shaohe_feng has joined #openstack-nova | 23:13 | |
| *** itlinux has quit IRC | 23:16 | |
| *** vladikr has quit IRC | 23:18 | |
| *** shaohe_feng has quit IRC | 23:22 | |
| *** shaohe_feng has joined #openstack-nova | 23:23 | |
| *** artom has joined #openstack-nova | 23:25 | |
| *** itlinux has joined #openstack-nova | 23:28 | |
| *** vladikr has joined #openstack-nova | 23:31 | |
| *** shaohe_feng has quit IRC | 23:32 | |
| *** shaohe_feng has joined #openstack-nova | 23:32 | |
| openstackgerrit | Merged openstack/nova master: [placement] Add gabbi coverage for an inventory change https://review.openstack.org/583642 | 23:33 |
| openstackgerrit | Merged openstack/nova master: [placement] Add gabbi coverage for inv of missing rp https://review.openstack.org/583643 | 23:33 |
| openstackgerrit | Merged openstack/nova master: [placement] cover bad content-length header https://review.openstack.org/583644 | 23:33 |
| *** _ix has quit IRC | 23:34 | |
| *** jmlowe has joined #openstack-nova | 23:37 | |
| *** shaohe_feng has quit IRC | 23:42 | |
| *** shaohe_feng has joined #openstack-nova | 23:44 | |
| *** threestrands has joined #openstack-nova | 23:50 | |
| *** shaohe_feng has quit IRC | 23:53 | |
| *** med_ has joined #openstack-nova | 23:53 | |
| *** shaohe_feng has joined #openstack-nova | 23:54 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!