| openstackgerrit | Matt Riedemann proposed openstack/nova master: Scheduler multiple workers support https://review.openstack.org/159382 | 00:11 |
|---|---|---|
| *** mriedem has joined #openstack-nova | 00:11 | |
| *** acormier has quit IRC | 00:17 | |
| *** lbragstad has joined #openstack-nova | 00:20 | |
| *** acormier has joined #openstack-nova | 00:22 | |
| *** hiro-kobayashi has joined #openstack-nova | 00:25 | |
| *** lbragstad has quit IRC | 00:27 | |
| *** amodi has quit IRC | 00:36 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform servergroup.addmember notification https://review.openstack.org/541101 | 00:41 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform aggregate.update_metadata notification https://review.openstack.org/460625 | 00:41 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add functional tests for traits API https://review.openstack.org/524094 | 00:42 |
| *** hshiina has joined #openstack-nova | 00:46 | |
| *** claudiub has quit IRC | 00:55 | |
| *** slaweq has joined #openstack-nova | 01:15 | |
| *** slaweq has quit IRC | 01:20 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Support shared/dedicated vCPUs in one instance https://review.openstack.org/545734 | 01:29 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Support shared and dedicated VMs in one host https://review.openstack.org/543805 | 01:34 |
| *** hongbin has joined #openstack-nova | 01:36 | |
| *** amodi has joined #openstack-nova | 02:02 | |
| *** itlinux has quit IRC | 02:02 | |
| *** yamahata has joined #openstack-nova | 02:03 | |
| *** links has joined #openstack-nova | 02:11 | |
| *** lbragstad has joined #openstack-nova | 02:15 | |
| *** r-daneel has joined #openstack-nova | 02:24 | |
| *** amodi has quit IRC | 02:29 | |
| *** yangyapeng has quit IRC | 02:30 | |
| *** r-daneel has quit IRC | 02:32 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717 | 02:35 |
| *** amodi has joined #openstack-nova | 02:39 | |
| *** Dinesh_Bhor has joined #openstack-nova | 02:42 | |
| *** salv-orl_ has joined #openstack-nova | 02:45 | |
| *** salv-orlando has quit IRC | 02:48 | |
| *** yamamoto_ has joined #openstack-nova | 03:08 | |
| *** yamamoto has quit IRC | 03:12 | |
| *** pooja_jadhav has joined #openstack-nova | 03:18 | |
| *** lbragstad has quit IRC | 03:29 | |
| *** sree has joined #openstack-nova | 03:32 | |
| *** acormier has quit IRC | 03:34 | |
| *** r-daneel has joined #openstack-nova | 03:40 | |
| *** yangyapeng has joined #openstack-nova | 03:44 | |
| *** udesale has joined #openstack-nova | 03:59 | |
| *** rmcall has quit IRC | 04:02 | |
| *** rmcall has joined #openstack-nova | 04:12 | |
| *** moshele has joined #openstack-nova | 04:18 | |
| *** moshele has quit IRC | 04:23 | |
| *** sridharg has joined #openstack-nova | 04:28 | |
| *** abhishekk has joined #openstack-nova | 04:28 | |
| *** takashin_ has joined #openstack-nova | 04:31 | |
| *** takashin has quit IRC | 04:33 | |
| *** bhujay has joined #openstack-nova | 04:40 | |
| *** psachin has joined #openstack-nova | 04:47 | |
| *** trinaths has joined #openstack-nova | 04:47 | |
| *** dave-mccowan has quit IRC | 04:53 | |
| *** stakeda has quit IRC | 05:04 | |
| *** stakeda has joined #openstack-nova | 05:15 | |
| *** takashin_ has left #openstack-nova | 05:17 | |
| *** slaweq has joined #openstack-nova | 05:17 | |
| *** takashin has joined #openstack-nova | 05:17 | |
| *** slaweq has quit IRC | 05:22 | |
| *** claudiub has joined #openstack-nova | 05:27 | |
| *** lajoskatona has joined #openstack-nova | 05:29 | |
| *** amodi has quit IRC | 05:29 | |
| *** jafeha__ has joined #openstack-nova | 05:30 | |
| *** moshele has joined #openstack-nova | 05:33 | |
| *** jafeha has quit IRC | 05:33 | |
| *** yamamoto has joined #openstack-nova | 05:34 | |
| *** janki has joined #openstack-nova | 05:38 | |
| *** yamamoto_ has quit IRC | 05:38 | |
| *** mdnadeem has joined #openstack-nova | 05:47 | |
| *** amodi has joined #openstack-nova | 05:48 | |
| *** hongbin has quit IRC | 05:53 | |
| *** threestrands has quit IRC | 06:10 | |
| *** threestrands has joined #openstack-nova | 06:23 | |
| *** kholkina has joined #openstack-nova | 06:32 | |
| *** acormier has joined #openstack-nova | 06:35 | |
| *** acormier has quit IRC | 06:40 | |
| *** alexchadin has joined #openstack-nova | 06:42 | |
| *** amodi has quit IRC | 06:51 | |
| *** jafeha__ is now known as jafeha | 06:58 | |
| *** threestrands has quit IRC | 07:00 | |
| *** rcernin has quit IRC | 07:11 | |
| *** maciejjozefczyk has quit IRC | 07:23 | |
| hrw | morning | 07:26 |
| *** andreas_s has joined #openstack-nova | 07:27 | |
| *** alexchadin has quit IRC | 07:27 | |
| *** hoonetorg has quit IRC | 07:30 | |
| *** yamamoto has quit IRC | 07:33 | |
| *** alexchadin has joined #openstack-nova | 07:39 | |
| *** yamamoto has joined #openstack-nova | 07:41 | |
| *** slaweq has joined #openstack-nova | 07:43 | |
| *** slaweq_ has joined #openstack-nova | 07:45 | |
| *** yamamoto has quit IRC | 07:46 | |
| *** hoonetorg has joined #openstack-nova | 07:46 | |
| *** pcaruana has joined #openstack-nova | 07:47 | |
| *** slaweq has quit IRC | 07:48 | |
| *** links has quit IRC | 07:55 | |
| *** AlexeyAbashkin has joined #openstack-nova | 07:56 | |
| *** slaweq has joined #openstack-nova | 08:03 | |
| *** links has joined #openstack-nova | 08:07 | |
| *** damien_r has joined #openstack-nova | 08:09 | |
| *** ccamacho has joined #openstack-nova | 08:09 | |
| *** slaweq has quit IRC | 08:10 | |
| *** ttsiouts has joined #openstack-nova | 08:10 | |
| *** tesseract has joined #openstack-nova | 08:15 | |
| *** ragiman has joined #openstack-nova | 08:22 | |
| *** sree_ has joined #openstack-nova | 08:29 | |
| *** sree_ is now known as Guest79222 | 08:29 | |
| *** ralonsoh has joined #openstack-nova | 08:30 | |
| *** rmcall has quit IRC | 08:31 | |
| *** sree has quit IRC | 08:32 | |
| *** Eran_Kuris has joined #openstack-nova | 08:38 | |
| *** krtaylor has quit IRC | 08:38 | |
| *** yamamoto has joined #openstack-nova | 08:39 | |
| *** Eran_Kuris has quit IRC | 08:40 | |
| *** amoralej|off is now known as amoralej | 08:41 | |
| *** jpena|off is now known as jpena | 08:43 | |
| *** bauwser is now known as bauazs | 08:53 | |
| *** bauazs is now known as bauzas | 08:53 | |
| bauzas | good morning Nova | 08:54 |
| *** Dinesh_Bhor has quit IRC | 08:56 | |
| hrw | bauzas: morning | 08:56 |
| hrw | my patch to control pcie ports is ready for reviews: https://review.openstack.org/545034 - there are tests for aarch64 and for x86-64 | 08:57 |
| hrw | release note present, config entry commented | 08:57 |
| *** sean-k-mooney has quit IRC | 08:59 | |
| *** Dinesh_Bhor has joined #openstack-nova | 09:01 | |
| *** tssurya has joined #openstack-nova | 09:04 | |
| *** lpetrut has joined #openstack-nova | 09:06 | |
| *** tetsuro has joined #openstack-nova | 09:07 | |
| *** elmaciej has joined #openstack-nova | 09:08 | |
| *** Dinesh_Bhor has quit IRC | 09:11 | |
| *** Dinesh_Bhor has joined #openstack-nova | 09:12 | |
| *** ameeda has joined #openstack-nova | 09:16 | |
| bauzas | hrw: I beg you for my ignorance about PCIe ports, but when you say "libvirt adds pcie ports as it needs" in https://review.openstack.org/#/c/545034/8//COMMIT_MSG, WDYM ? | 09:17 |
| *** cdent has joined #openstack-nova | 09:17 | |
| bauzas | hrw: when you say "libvirt", it means libvirtd, right? not the nova libvirt driver | 09:18 |
| ameeda | Hello when I remove pycrypto as this bug needed https://bugs.launchpad.net/openstack-requirements/+bug/1749574, I got this error https://pasteboard.co/H8l6IC4.png | 09:19 |
| openstack | Launchpad bug 1749574 in OpenStack DBaaS (Trove) "[tracking] removal and migration of pycrypto" [Undecided,In progress] - Assigned to Ameed Ashour (ameeda) | 09:19 |
| hrw | bauzas: when I say libvirt I mean upstream libvirt so yes, libvirtd | 09:19 |
| hrw | ameeda: migrating pycrypto -> cryphography? | 09:20 |
| *** derekh has joined #openstack-nova | 09:21 | |
| ameeda | hrw: requirements.txt file already has cryphography | 09:22 |
| *** mgoddard_ has joined #openstack-nova | 09:23 | |
| ameeda | https://github.com/openstack/barbican/blob/master/requirements.txt#L7 | 09:23 |
| hrw | sure | 09:24 |
| *** lajoskatona has quit IRC | 09:25 | |
| *** r-daneel has quit IRC | 09:28 | |
| ameeda | hrw: as I saw, the error caused by test_dogtag, what do you think ? | 09:28 |
| bauzas | hrw: had some comments | 09:28 |
| bauzas | hrw: the main one is that I think you should create a blueprint for your feature | 09:28 |
| bauzas | hrw: and engage with the community whether you need a spec | 09:29 |
| bauzas | it's better for tracking your feature | 09:29 |
| *** salladi has joined #openstack-nova | 09:29 | |
| bauzas | you're also adding a new config option, that properly needs to be released | 09:29 |
| bauzas | hrw: see https://docs.openstack.org/nova/latest/contributor/blueprints.html for the process-ish thingy | 09:30 |
| bauzas | hrw: I'm personnally on the side that a blueprint is enough, we don't really need a spec | 09:30 |
| bauzas | but devil could be in details, and somehow some people could argue about your implementation design that would mean a spec is needed | 09:31 |
| *** takashin has left #openstack-nova | 09:32 | |
| *** links has quit IRC | 09:32 | |
| *** Dinesh_Bhor has quit IRC | 09:33 | |
| hrw | bauzas: thanks. will create bp | 09:33 |
| *** salladi has quit IRC | 09:34 | |
| bauzas | hrw: add your blueprint in the next nova meeting agenda then | 09:35 |
| hrw | ok | 09:35 |
| bauzas | and be present at the nova meeting | 09:35 |
| hrw | sure | 09:35 |
| bauzas | so we could validate your blueprint or ask you for a spec | 09:35 |
| bauzas | you're not in a rush, we're just beginning to review specs :) | 09:36 |
| *** alexchadin has quit IRC | 09:36 | |
| hrw | bauzas: would love to backport it to queens one day | 09:36 |
| bauzas | in case you're not familiar with the release cadence, we generally stop approving specs and blueprints past the first milestone | 09:36 |
| bauzas | hrw: well, that's a feature, and we don't feature backport upstream | 09:37 |
| hrw | please rephrase last sentence | 09:37 |
| bauzas | hrw: feel free to do whatever you want in your downstream product, but there are no chances that a change including a features relnote could be backported | 09:37 |
| hrw | ok. | 09:38 |
| bauzas | speaking of the differences between the upstream nova project and any other distribution | 09:38 |
| bauzas | hrw: I guess you're not familiar with the stable branches procedure, lemme find you some doc | 09:39 |
| bauzas | hrw: https://docs.openstack.org/project-team-guide/stable-branches.html | 09:39 |
| hrw | bauzas: so far backported just simple changes | 09:39 |
| *** masahisa has joined #openstack-nova | 09:40 | |
| bauzas | hrw: np, not all of us are familiar with all the procedures, I get that :p | 09:42 |
| hrw | yep | 09:42 |
| *** links has joined #openstack-nova | 09:45 | |
| *** r-daneel has joined #openstack-nova | 09:46 | |
| *** Eran_Kuris has joined #openstack-nova | 09:54 | |
| *** Eran_Kuris has quit IRC | 09:55 | |
| *** Eran_Kuris has joined #openstack-nova | 09:55 | |
| hrw | bauzas: replied | 09:58 |
| hrw | bauzas: and thanks for detailed review | 09:58 |
| *** salv-orl_ has quit IRC | 10:02 | |
| *** salv-orlando has joined #openstack-nova | 10:03 | |
| hrw | https://blueprints.launchpad.net/nova/+spec/configure-amount-of-pcie-ports | 10:05 |
| *** r-daneel has quit IRC | 10:06 | |
| *** salv-orlando has quit IRC | 10:07 | |
| *** Guest79222 has quit IRC | 10:09 | |
| *** mdnadeem has quit IRC | 10:11 | |
| *** sree_ has joined #openstack-nova | 10:12 | |
| *** sree_ is now known as Guest85075 | 10:12 | |
| *** zames2 has joined #openstack-nova | 10:14 | |
| *** kmalloc has joined #openstack-nova | 10:14 | |
| *** Guest85075 has quit IRC | 10:16 | |
| *** r-daneel has joined #openstack-nova | 10:22 | |
| *** mdnadeem has joined #openstack-nova | 10:27 | |
| *** mdnadeem_ has joined #openstack-nova | 10:30 | |
| *** mdnadeem has quit IRC | 10:32 | |
| *** lajoskatona has joined #openstack-nova | 10:33 | |
| *** moshele has quit IRC | 10:33 | |
| *** rmart04 has joined #openstack-nova | 10:34 | |
| *** cdent has quit IRC | 10:36 | |
| *** Eran_Kuris has quit IRC | 10:38 | |
| *** zames2 has quit IRC | 10:39 | |
| *** stakeda has quit IRC | 10:40 | |
| *** lajoskatona has quit IRC | 10:40 | |
| hrw | bauzas: where in meeting agenda would you suggest to add spec link? | 10:41 |
| bauzas | hrw: in the nova meeting agenda, lemme find it | 10:41 |
| hrw | https://wiki.openstack.org/wiki/Meetings/Nova | 10:41 |
| hrw | bauzas: I mean "which part of meeting" | 10:41 |
| *** abhishekk has quit IRC | 10:48 | |
| *** zames2 has joined #openstack-nova | 10:48 | |
| *** hshiina has quit IRC | 10:50 | |
| bauzas | hrw: sorry got distracted | 10:51 |
| hrw | bauzas: no problem. work is work | 10:51 |
| bauzas | hrw: so, in https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting (seems a bit stale, you should comment that it's for the 22th), in Open Discussion | 10:52 |
| hrw | thx | 10:52 |
| bauzas | saying "specless BP approval" | 10:52 |
| hrw | bauzas: added | 10:54 |
| openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: Allow to configure amount of PCIe ports https://review.openstack.org/545034 | 10:56 |
| *** alexchadin has joined #openstack-nova | 10:58 | |
| hrw | commit message partially rewritten + BP link. some suggested code changes | 10:58 |
| *** cdent has joined #openstack-nova | 11:01 | |
| *** AlexeyAbashkin has quit IRC | 11:07 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Fix PatternPropertiesTestCase for py 3.6 https://review.openstack.org/545798 | 11:09 |
| *** ragiman has quit IRC | 11:13 | |
| *** ragiman has joined #openstack-nova | 11:18 | |
| *** lucas-afk is now known as lucasagomes | 11:18 | |
| *** priteau has joined #openstack-nova | 11:19 | |
| *** krtaylor has joined #openstack-nova | 11:22 | |
| *** dtantsur|afk is now known as dtantsur | 11:23 | |
| *** salv-orlando has joined #openstack-nova | 11:25 | |
| *** ccamacho has quit IRC | 11:26 | |
| *** moshele has joined #openstack-nova | 11:26 | |
| *** sree has joined #openstack-nova | 11:27 | |
| *** ccamacho has joined #openstack-nova | 11:28 | |
| *** sree has quit IRC | 11:32 | |
| *** AlexeyAbashkin has joined #openstack-nova | 11:32 | |
| *** alex_ has joined #openstack-nova | 11:34 | |
| *** udesale_ has joined #openstack-nova | 11:35 | |
| *** udesale has quit IRC | 11:37 | |
| *** udesale_ has quit IRC | 11:39 | |
| *** alex_ is now known as fusmu | 11:40 | |
| *** bnemec is now known as bnemec-pto | 11:41 | |
| *** ccamacho is now known as ccamacho|lunch | 11:46 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Support shared/dedicated vCPUs in one instance https://review.openstack.org/545734 | 11:47 |
| *** cdent has quit IRC | 11:56 | |
| *** masber has joined #openstack-nova | 11:58 | |
| *** alexchadin has quit IRC | 11:59 | |
| *** sree has joined #openstack-nova | 12:03 | |
| *** slaweq has joined #openstack-nova | 12:07 | |
| *** zames2 has quit IRC | 12:08 | |
| *** slaweq has quit IRC | 12:12 | |
| *** dave-mccowan has joined #openstack-nova | 12:16 | |
| *** cdent has joined #openstack-nova | 12:16 | |
| *** belmoreira has joined #openstack-nova | 12:16 | |
| *** trinaths has quit IRC | 12:19 | |
| *** slagle has joined #openstack-nova | 12:20 | |
| *** yamamoto has quit IRC | 12:21 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Support shared and dedicated VMs in one host https://review.openstack.org/543805 | 12:23 |
| *** tetsuro has left #openstack-nova | 12:24 | |
| *** yamamoto has joined #openstack-nova | 12:25 | |
| *** sdague has joined #openstack-nova | 12:25 | |
| *** takashin has joined #openstack-nova | 12:27 | |
| openstackgerrit | Tetiana Lashchova proposed openstack/python-novaclient master: Fix the docstring for the update method https://review.openstack.org/545819 | 12:32 |
| *** acormier has joined #openstack-nova | 12:35 | |
| *** yasemin has quit IRC | 12:36 | |
| *** links has quit IRC | 12:37 | |
| *** jpena is now known as jpena|lunch | 12:39 | |
| *** acormier has quit IRC | 12:40 | |
| *** hiro-kobayashi has quit IRC | 12:41 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Extending delete_cell --force to delete instance_mappings https://review.openstack.org/540073 | 12:42 |
| *** sean-k-mooney has joined #openstack-nova | 12:46 | |
| sean-k-mooney | stephenfin: enhancing your specs with images http://logs.openstack.org/90/541290/4/check/build-openstack-sphinx-docs/da8d825/html/specs/rocky/approved/numa-aware-vswitches.html thats almost cheating :P | 12:49 |
| cdent | I thought the same thing | 12:51 |
| *** psachin has quit IRC | 12:51 | |
| *** artom__ has quit IRC | 12:52 | |
| sean-k-mooney | cdent: you can tell being on the docs team is rubing off on him :) that said the fact that he uses images to help explain numa rather then 10 paragraps of text no one will follow might be a reason he is a good person to be on the docs team. | 12:55 |
| cdent | indeed | 12:56 |
| *** masahisa has quit IRC | 12:56 | |
| *** tssurya has quit IRC | 13:03 | |
| *** edmondsw has joined #openstack-nova | 13:12 | |
| *** sree_ has joined #openstack-nova | 13:14 | |
| *** sree_ is now known as Guest87972 | 13:15 | |
| *** sree has quit IRC | 13:17 | |
| *** mchlumsky has joined #openstack-nova | 13:17 | |
| *** tssurya has joined #openstack-nova | 13:18 | |
| *** ccamacho|lunch has quit IRC | 13:27 | |
| *** ccamacho has joined #openstack-nova | 13:27 | |
| *** jpena|lunch is now known as jpena | 13:31 | |
| *** mchlumsky has quit IRC | 13:35 | |
| *** mchlumsky has joined #openstack-nova | 13:38 | |
| *** mgoddard_ has quit IRC | 13:39 | |
| *** acormier has joined #openstack-nova | 13:40 | |
| openstackgerrit | Jay Pipes proposed openstack/nova-specs master: Account for host agg allocation ratio in placement https://review.openstack.org/544683 | 13:41 |
| *** pchavva has joined #openstack-nova | 13:42 | |
| *** pchavva1 has joined #openstack-nova | 13:43 | |
| *** acormier has quit IRC | 13:43 | |
| *** acormier has joined #openstack-nova | 13:44 | |
| *** belmorei_ has joined #openstack-nova | 13:46 | |
| *** belmoreira has quit IRC | 13:48 | |
| *** vladikr has joined #openstack-nova | 13:48 | |
| cdent | sean-k-mooney: live migration is for virtualisers not cloud native people! true or false? | 13:48 |
| *** liverpooler has joined #openstack-nova | 13:50 | |
| *** acormier has quit IRC | 13:51 | |
| kashyap | False | 13:51 |
| kashyap | cdent: Because: | 13:51 |
| cdent | kashyap for context https://review.openstack.org/#/c/541290/4/specs/rocky/approved/numa-aware-vswitches.rst@241 (in other words I was mostly making a joke, but still want to know the because) | 13:52 |
| kashyap | People say Cloud is all about 'cattle' (sorry, can't think of a less trite analogy), but people _do_ care about live migratability in Cloud | 13:52 |
| kashyap | cdent: I realized you're joking with a straight face (also with the exclamation) :P | 13:53 |
| * kashyap clicks on the review | 13:53 | |
| cdent | much of that spec has trouble aligning with cattle-ness, but people do care about it | 13:54 |
| *** masuberu has joined #openstack-nova | 13:54 | |
| edleafe | Scheduler subteam meeting in 5 minutes in #openstack-meeting-alt | 13:55 |
| kashyap | cdent: Saw asomething else on the spec, will comment there too | 13:55 |
| *** masber has quit IRC | 13:58 | |
| *** yasemin has joined #openstack-nova | 13:59 | |
| sean-k-mooney | cdent: cloud native people should be able to have there application live migrated too. | 13:59 |
| cdent | sean-k-mooney: for sake of getting your point of view, why? Shouldn't it just be restarted somewhere else? | 14:00 |
| sean-k-mooney | cdent: live migration is really to allow operators to do maintenace when they done really know what the tenant is running | 14:00 |
| edmondsw | restarting somewhere else means possible downtime | 14:01 |
| sean-k-mooney | cdent: live migration in openstack is admin only so its tenant initiated its operator driven | 14:01 |
| *** ttsiouts_ has joined #openstack-nova | 14:01 | |
| *** artom has joined #openstack-nova | 14:01 | |
| *** acormier has joined #openstack-nova | 14:01 | |
| *** acormier has quit IRC | 14:01 | |
| sean-k-mooney | edmondsw: yes that said livemigration could still violate your sla with your tenant | 14:01 |
| cdent | surely every service should have at least two instances of itself all the time :) | 14:02 |
| *** leakypipes is now known as jaypipes | 14:02 | |
| *** acormier has joined #openstack-nova | 14:02 | |
| edmondsw | cdent ha! if only | 14:02 |
| sean-k-mooney | cdent: the best example counter example to that is a provider edge router | 14:02 |
| edmondsw | sean-k-mooney that all depends | 14:02 |
| edmondsw | hence your "could", I'm sure :) | 14:02 |
| *** jackie-truong has joined #openstack-nova | 14:02 | |
| cdent | shadow provider edge router | 14:03 |
| sean-k-mooney | cdent: you cant have ha by design as it is respocible for ratelimiting your broadband connection so all traffic from one endpoint most be handeled by 1 PE router | 14:03 |
| *** zames2 has joined #openstack-nova | 14:03 | |
| *** zames2 has quit IRC | 14:03 | |
| *** zames2 has joined #openstack-nova | 14:04 | |
| sean-k-mooney | cdent: you dont want to reboot in this case because it can thke 10+ minutes to rebuild your peering relationships with other PE routers | 14:04 |
| *** takedakn has joined #openstack-nova | 14:05 | |
| *** acormier has quit IRC | 14:05 | |
| kashyap | cdent: High availability, is it... | 14:06 |
| kashyap | Yeah, sean-k-mooney explained it clearer -- about migratability | 14:06 |
| *** amoralej is now known as amoralej|lunch | 14:06 | |
| cdent | I hear and ack what you're saying. | 14:06 |
| cdent | But will remain frustrated :) | 14:06 |
| kashyap | Just take a look at the number of config knobs we have for live migration | 14:07 |
| kashyap | A common scenario for it in OpenStack context is upgrades | 14:07 |
| edmondsw | also think about dev on the cloud... I'm still developing this service and haven't setup HA yet, but I don't want you restarting it while I'm working | 14:07 |
| kashyap | Where you update Compute-B, migrate all instances from Compute-A to B | 14:07 |
| kashyap | And so forth | 14:07 |
| sean-k-mooney | cdent: well live migration cannot always be assumed to be available in a cloud and even if it is it my not be viable due to your perfromce constratints | 14:07 |
| kashyap | Yeah, random restarts that catch tenant users off-guard are completely undesirable | 14:08 |
| *** moshele has quit IRC | 14:09 | |
| kashyap | sean-k-mooney: Yeah, that's also why, as you pointed out, tenant users don't get to migrate instances. It is admins that drive it. | 14:09 |
| cdent | Sounds like there could be a problem with "random restarts" that is orthogonal to everything else | 14:09 |
| sean-k-mooney | i would generally say livemigration is more of an enterprise feature then telco as most nfv application cant tollerate the dataplane impact of a livemigration (latency,bandwith and packet loss) | 14:09 |
| kashyap | True | 14:10 |
| edmondsw | kashyap tenant users don't get to migrate instances because migration assumes knowledge of the infrastructure. If I don't have that knowledge, why would I try to move it :) | 14:11 |
| sean-k-mooney | cdent: to your original question a cloud native application should not require livemigration to be avalible at all as it should be fault tolerant enough to cover a cold migration but most applications are not cloud native. | 14:11 |
| kashyap | edmondsw: Yeah, indeed. | 14:12 |
| cdent | I should probably have put some ™ in my original statement to make it clear that I was talking about over-idealized and over-hyped pictures. | 14:12 |
| edmondsw | is there such a thing as over-idealized? ;) | 14:13 |
| sean-k-mooney | edmondsw: that and allowing tennants to live migrate stuff is a great way to ddos you management network as that is what all the openstack installers configure to use by default. | 14:13 |
| edmondsw | ++ | 14:14 |
| edleafe | edmondsw: that's like being "more unique" | 14:14 |
| edmondsw | but if there's no use case to allow tenants to live migrate (because they lack the necessary infra knowledge) all other concerns become secondary | 14:14 |
| * edmondsw nods at edleafe | 14:15 | |
| sean-k-mooney | edmondsw: i would say the existence of apple product in the world would be evedence of over-idealized design above fuction. | 14:15 |
| edmondsw | sean-k-mooney touche! | 14:15 |
| jaypipes | mriedem, dansmith, melwitt, bauzas, stephenfin: we've got pretty broad agreement on the approach in https://review.openstack.org/#/c/540111/. would be great to approve that and unblock the patch series implementing it. | 14:15 |
| *** arvindn05 has joined #openstack-nova | 14:16 | |
| bauzas | jaypipes: stephenfin is on PTO this week | 14:16 |
| bauzas | but I can review it for sure | 14:16 |
| sean-k-mooney | oh by the way is master open for rocky and i assume not is a good time to repopose the feature based schduling spec form last cycle? | 14:16 |
| edmondsw | sean-k-mooney master is open | 14:17 |
| sean-k-mooney | edmondsw: cool i have some patches to rebase that missed code freeze | 14:17 |
| edmondsw | join the club! | 14:17 |
| jaypipes | sean-k-mooney: feature-based scheduling? | 14:18 |
| jaypipes | sean-k-mooney: oh, the nic feature one? | 14:18 |
| sean-k-mooney | jaypipes: ya rodolfos work | 14:18 |
| jaypipes | gotcha | 14:18 |
| sean-k-mooney | he is full time on yardstick in opnfv now so im pickup all his patches at least to the end of Q1 /early Q2 | 14:19 |
| *** esberglu has joined #openstack-nova | 14:20 | |
| *** belmorei_ has quit IRC | 14:21 | |
| *** jackie-truong has quit IRC | 14:21 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Provide a hint when performing a server action can't find the method https://review.openstack.org/545382 | 14:21 |
| *** belmoreira has joined #openstack-nova | 14:21 | |
| mriedem | the bug fix series starting here needs review https://review.openstack.org/#/c/539758/ - +2 on all changes; fixes a problem going back to at least newton, if not since bfv was added forever ago | 14:23 |
| *** lbragstad has joined #openstack-nova | 14:23 | |
| *** pchavva1 has quit IRC | 14:25 | |
| *** mgoddard_ has joined #openstack-nova | 14:26 | |
| *** jafeha__ has joined #openstack-nova | 14:26 | |
| mriedem | jaypipes: i'll go through that spec later this morning if someone else hasn't gotten to it first | 14:28 |
| mriedem | i know dan has been through it before | 14:29 |
| jaypipes | mriedem: I'll go through that patch series for the bug. | 14:29 |
| *** jafeha has quit IRC | 14:29 | |
| *** pchavva has quit IRC | 14:29 | |
| efried | kumbaya | 14:30 |
| sean-k-mooney | jaypipes: mriedem by the way the intel nfv ci should be operational again. can ye see any logs here http://http//52.27.155.124//portland/2018-02-19/540073/4/check/tempest-dsvm-intel-nfv-xenial/b045a8e | 14:30 |
| *** takedakn has quit IRC | 14:30 | |
| sean-k-mooney | oh i see whats wrong they are not generating the url correctly.. ill let them know | 14:31 |
| *** jistr is now known as jistr|mtg | 14:31 | |
| *** yasemin has quit IRC | 14:32 | |
| *** udesale has joined #openstack-nova | 14:38 | |
| *** rmart04 has quit IRC | 14:40 | |
| *** udesale has quit IRC | 14:40 | |
| *** udesale has joined #openstack-nova | 14:40 | |
| *** links has joined #openstack-nova | 14:41 | |
| *** sree has joined #openstack-nova | 14:41 | |
| *** eharney has joined #openstack-nova | 14:42 | |
| *** rmart04 has joined #openstack-nova | 14:43 | |
| *** pchavva has joined #openstack-nova | 14:44 | |
| *** Guest87972 has quit IRC | 14:45 | |
| *** jroll has quit IRC | 14:47 | |
| *** Placeed has joined #openstack-nova | 14:47 | |
| *** jroll has joined #openstack-nova | 14:48 | |
| *** tssurya has quit IRC | 14:49 | |
| *** links has quit IRC | 14:49 | |
| Placeed | Hi all, I have a problem with ressource_tracker.py on my compute instance. This script return a disk usage of 90% to nova so I can't run new instances. The problem is on the calculation ... Actually I use Cinder with NFS backend on /var/lib/nova/mnt with few tera avaiable. I don't run ephemeral vm's. It sounds like the script sum all disk usage of vm running and check only the local disk space of the compute server. | 14:50 |
| Placeed | Someone can help me ? | 14:50 |
| *** archit has joined #openstack-nova | 14:50 | |
| *** archit is now known as amodi | 14:51 | |
| *** arvindn05 has left #openstack-nova | 14:52 | |
| Placeed | As you can see there : https://paste.ofcode.org/SGqTUpeuWMErjfUwmHwQN2 The node think he have only 270Go available but in reality he have few tera through NFS | 14:53 |
| *** lbragstad has quit IRC | 14:53 | |
| efried | Placeed: Is this a boot-from-volume setup? | 14:55 |
| *** lbragstad has joined #openstack-nova | 14:55 | |
| Placeed | efried : Yes, In user point of view, I create a new instance based on image and it will create a cinder volume for the root disk of the instance | 14:56 |
| *** amoralej|lunch is now known as amoralej | 14:56 | |
| Placeed | But it failed to find a host acceptable because of this "bug" | 14:57 |
| *** sree has quit IRC | 14:57 | |
| *** awaugama has joined #openstack-nova | 14:58 | |
| *** kholkina has quit IRC | 14:58 | |
| kashyap | bauzas: I'm not sure a spec is required in this case https://review.openstack.org/#/c/545034/ ("Allow to configure amount of PCIe ports") | 14:58 |
| *** sree has joined #openstack-nova | 14:58 | |
| *** yamahata has quit IRC | 14:58 | |
| *** yamahata has joined #openstack-nova | 14:58 | |
| bauzas | kashyap: sure, I discussed with hrw about the process | 14:58 |
| bauzas | he created a BP, and asked for a specless approval for the next nova meeting | 14:59 |
| kashyap | Ah-ha | 14:59 |
| kashyap | hrw: The above change should also mention something about its usefulness in context of Nova coping with Q35 machine types | 14:59 |
| kashyap | (Recall the discussion from #virt, OFTC) | 14:59 |
| kashyap | At some point distributions will default to the Q35, and Nova should be ready to gracefully handle it. | 14:59 |
| kashyap | So your change helps with that; so it's worth mentioning. | 15:00 |
| *** tidwellr has joined #openstack-nova | 15:00 | |
| kashyap | I'll add a note in the review | 15:00 |
| *** dtantsur is now known as dtantsur|brb | 15:01 | |
| *** links has joined #openstack-nova | 15:01 | |
| *** ttsiouts_ has quit IRC | 15:02 | |
| efried | Placeed: I know mriedem has been looking at bfv stuff lately, though I don't know the details. I think he has recently put up a series of fixes - but again, no idea if the bug is related to this. Looking... | 15:02 |
| *** jistr|mtg is now known as jistr | 15:02 | |
| efried | Placeed: Here's the code series: https://review.openstack.org/#/c/539758/ | 15:02 |
| openstackgerrit | Hamdy Khader proposed openstack/nova master: Adding NVMEoF for libvirt driver https://review.openstack.org/482640 | 15:02 |
| *** acormier has joined #openstack-nova | 15:03 | |
| efried | Placeed: Hmm, the bug doesn't sound like your issue: https://bugs.launchpad.net/nova/+bug/1404867 | 15:03 |
| openstack | Launchpad bug 1404867 in OpenStack Compute (nova) "Volume remains in-use status, if instance booted from volume is deleted in error state" [Medium,In progress] - Assigned to melanie witt (melwitt) | 15:03 |
| *** yasemin has joined #openstack-nova | 15:03 | |
| *** sree has quit IRC | 15:03 | |
| mriedem | efried: it's not, | 15:05 |
| mriedem | it's likely https://bugs.launchpad.net/nova/+bug/1469179 | 15:05 |
| openstack | Launchpad bug 1469179 in OpenStack Compute (nova) "instance.root_gb should be 0 for volume-backed instances" [Medium,In progress] - Assigned to melanie witt (melwitt) | 15:05 |
| mriedem | https://review.openstack.org/#/q/topic:fix-bfv-boot-resources+(status:open+OR+status:merged) | 15:05 |
| *** fusmu has quit IRC | 15:05 | |
| *** zames2 has quit IRC | 15:05 | |
| *** tssurya has joined #openstack-nova | 15:05 | |
| mriedem | which has been continually deferred until we have sharing disk_gb provider support, which continually gets deferred | 15:05 |
| efried | mriedem: "sharing disk_gb provider support" as in placement? | 15:06 |
| *** bhujay has quit IRC | 15:06 | |
| *** cdent_ has joined #openstack-nova | 15:06 | |
| *** cdent has quit IRC | 15:06 | |
| *** acormier has quit IRC | 15:06 | |
| mriedem | yeah | 15:06 |
| efried | okay. | 15:06 |
| mriedem | having the computes / RT be aware of shared storage pools | 15:06 |
| Placeed | mriedem : it really sounds like https://bugs.launchpad.net/nova/+bug/1469179 | 15:06 |
| openstack | Launchpad bug 1469179 in OpenStack Compute (nova) "instance.root_gb should be 0 for volume-backed instances" [Medium,In progress] - Assigned to melanie witt (melwitt) | 15:06 |
| mriedem | Placeed: yes it's an old old bug | 15:07 |
| *** acormier has joined #openstack-nova | 15:07 | |
| *** cdent has joined #openstack-nova | 15:07 | |
| *** bhujay has joined #openstack-nova | 15:07 | |
| sean-k-mooney | kashyap: i responded to your question regarding numa aware vswitches | 15:07 |
| Placeed | mriedem : It mean there is a fix actually ? I'm running SUSE Official Openstack, maybe they run an old version ? | 15:07 |
| efried | Placeed: There's not a fix yet. | 15:08 |
| *** takashin has left #openstack-nova | 15:08 | |
| *** rmart04_ has joined #openstack-nova | 15:08 | |
| mriedem | Placeed: not unless suse patched it in their distro | 15:08 |
| kashyap | sean-k-mooney: Thanks; will read | 15:08 |
| sean-k-mooney | kashyap: the patch you linked is not quite what we need but i have added the details to the spec for what we would need to allow numa aware pcie virtualisation within the guest | 15:08 |
| efried | Placeed: The shared provider support mriedem is talking about is feature work that's been in progress for several releases now. | 15:08 |
| *** rmart04 has quit IRC | 15:08 | |
| *** rmart04_ is now known as rmart04 | 15:08 | |
| Placeed | mriedem : Could you help me to identify which fix it is ? I can ask them | 15:08 |
| mriedem | Placeed: there is no fix upstream yet | 15:08 |
| kashyap | sean-k-mooney: Ah; right. For PCIe, we'd also mention the need for Q35 machine type — as that's mandatory for PCIe | 15:09 |
| kashyap | (I'd guess you already know that.) | 15:09 |
| mriedem | Placeed: there is https://review.openstack.org/#/q/topic:fix-bfv-boot-resources+(status:open+OR+status:merged) but there is not agreement on merging those changes | 15:09 |
| mriedem | as they would introduce some technical debt | 15:09 |
| sean-k-mooney | kashyap: well i dont think it acutlly is | 15:09 |
| mriedem | Placeed: you could take those, rebase them, and patch them into your env as a workaround | 15:09 |
| sean-k-mooney | kashyap: can you show me where that is stated? | 15:09 |
| kashyap | sean-k-mooney: Let me double-check on my comment w/ the Virt folks | 15:09 |
| kashyap | Because, pretty sure I was told so by a QEMU dev | 15:09 |
| efried | Placeed: For the "real fix", a lot of the groundwork has been laid at this point, and it's just possible we'll put it on the slate for Rocky. We'll be talking about that at the PTG next week. | 15:09 |
| mriedem | Placeed: but if those patches never land, you're left with the fork | 15:09 |
| Placeed | mriedem : But it mean actually everybody who is using NFS / CInder have the same issue on openstack right ? | 15:10 |
| Placeed | Or it's only me on my suse distribution | 15:10 |
| mnaser | good morning everyone, just going to drop https://review.openstack.org/#/q/status:open+topic:bug/1404867 here if someone feels like going through these patches, they're ready for final review (afaik) | 15:10 |
| sean-k-mooney | kashyap: based on https://github.com/qemu/qemu/blob/master/docs/pcie.txt it does seam to be related to Q35 but not sure its a hard dependecy | 15:10 |
| *** cdent_ has quit IRC | 15:10 | |
| mriedem | Placeed: it's not just you | 15:11 |
| hrw | kashyap: thx | 15:11 |
| sean-k-mooney | kashyap: using the Q35 chipset is likely a good idea anyway | 15:11 |
| Placeed | mriedem : So another workaround would be to have a local disk as big as all my vm's root disk running on that server right ? | 15:11 |
| hrw | kashyap: BP mentions q35 but right, commit can describe it more too | 15:12 |
| hrw | kashyap: release note describes x86/q35 and aarch64/virt | 15:13 |
| kashyap | sean-k-mooney: Okay, I double-confirmed, and indeed Q35 is a hard dep for PCIe. | 15:14 |
| kashyap | sean-k-mooney: That doc you linked to should be updated; I'll update it | 15:14 |
| mriedem | Placeed: i'm not sure about that | 15:14 |
| kashyap | sean-k-mooney: But, look here (the URL from the doc you mentioned), on slide-13: https://wiki.qemu.org/images/4/4e/Q35.pdf | 15:15 |
| kashyap | sean-k-mooney: It says: "Q35-only features: PCIe goodies [...]" | 15:15 |
| hrw | sean-k-mooney: q35 is hard dep for pcie. | 15:15 |
| -openstackstatus- NOTICE: Zuul has been restarted to pick up latest memory fixes. Queues were saved however patches uploaded after 14:40UTC may have been missed. Please recheck if needed. | 15:15 | |
| Placeed | mriedem : What could be a workaround while i wait the official patch ? | 15:15 |
| sean-k-mooney | kashyap: yes but the feature i was memtioning is not listed in that slide and qemu had limited pcie support before the uese of q35 | 15:16 |
| hrw | sean-k-mooney: qemu has 3 x86 platforms: isa-pc (no one uses), i440fx (pci based), q35 (pcie based) | 15:16 |
| *** sree has joined #openstack-nova | 15:16 | |
| mriedem | Placeed: i'm not intimately familiar with the bug nor the fix - melwitt has carried the patch for a long time, so she might be able to help once she's around | 15:16 |
| sean-k-mooney | hrw: really i taught i440fx supported pcie | 15:16 |
| hrw | sean-k-mooney: i440fx is older than pcie ;D | 15:16 |
| mriedem | Placeed: you could also ask about that bug in the #openstack-operators channel and ask if anyone has patched it out of tree and if so, how? | 15:17 |
| kashyap | sean-k-mooney: Yeah, first I'll read your full comment before talking further. And yes, Q35 does have the limitation you note. | 15:17 |
| mriedem | Placeed: i know SAP is probably doing something for it | 15:17 |
| mriedem | they use NFS for everything, and are i think suse customers | 15:17 |
| Placeed | mriedem : Ok thank you very very much for your help. I will also check with SUSE Guys. | 15:18 |
| kashyap | sean-k-mooney: My second sentence in my previous message was misphrased. Correct: Q35 _solves_ that limitation of older QEMUs PCIe support. | 15:18 |
| hrw | would be nice to see some future qemu dropping <q35 ;D | 15:19 |
| hrw | but that's not gonna happen | 15:19 |
| bauzas | stupid question but we get network information from the compute right ? https://bugs.launchpad.net/nova/+bug/1740241 | 15:19 |
| openstack | Launchpad bug 1740241 in OpenStack Compute (nova) "Network info not always displayed for a created instance" [Undecided,New] | 15:19 |
| bauzas | at least until we do that from the conductor | 15:20 |
| *** mlavalle has joined #openstack-nova | 15:20 | |
| *** mdnadeem_ has quit IRC | 15:20 | |
| *** rmart04 has quit IRC | 15:20 | |
| *** rmart04 has joined #openstack-nova | 15:21 | |
| *** sree has quit IRC | 15:21 | |
| mriedem | efried: questoin here https://review.openstack.org/#/c/540111/5/specs/rocky/approved/update-provider-tree.rst@73 | 15:21 |
| bauzas | nevermind, got the line | 15:21 |
| *** elmaciej has quit IRC | 15:22 | |
| mriedem | lyarwood: are you around this week? i want to flush some ocata patches since ocata eol is supposedly next week | 15:23 |
| kashyap | hrw: Yes, that won't happen. Majority still use the 'i440fx', as you can guess. | 15:24 |
| hrw | yep | 15:25 |
| kashyap | mriedem: FWIW, I at least saw him on IRC briefly in the (CET) morning | 15:25 |
| *** slaweq has joined #openstack-nova | 15:26 | |
| *** sree has joined #openstack-nova | 15:27 | |
| bauzas | mriedem: I can help | 15:28 |
| bauzas | mriedem: lemme know which ones you'd like to see reviewed | 15:28 |
| bauzas | for the moment, looking at LP | 15:28 |
| *** cdent has quit IRC | 15:28 | |
| mriedem | bauzas: https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/ocata+label:Code-Review=2 | 15:30 |
| mriedem | some of those might have things on pike yet | 15:31 |
| mriedem | like https://review.openstack.org/#/c/539008/ | 15:31 |
| bauzas | mriedem: ack, will look at them ASAIC | 15:31 |
| mriedem | thanks | 15:31 |
| *** slaweq has quit IRC | 15:31 | |
| *** sridharg has quit IRC | 15:31 | |
| bauzas | 24 open bugs :p | 15:31 |
| bauzas | woooot | 15:31 |
| *** acormier_ has joined #openstack-nova | 15:32 | |
| *** burt has joined #openstack-nova | 15:34 | |
| *** lucasagomes is now known as lucas-hungry | 15:34 | |
| *** acormier has quit IRC | 15:36 | |
| *** sree has quit IRC | 15:37 | |
| sean-k-mooney | hrw: kashyap is there a spec to enable Q35 proposed. i assume it will be via an image metadata item or perhaps a config option in the libvirt section of the nova.conf | 15:45 |
| kashyap | sean-k-mooney: I updated the upstream QEMU doc to clearly spell out machine type requirements for PCIe features. | 15:46 |
| hrw | sean-k-mooney: you set hw_machine_type = q35 in image and done | 15:46 |
| kashyap | sean-k-mooney: Hmm, not that I know of; I have a TODO item to propose one, unless someone beats me to it | 15:46 |
| *** awaugama has quit IRC | 15:46 | |
| kashyap | But yeah, currently what hrw tells above ^ | 15:46 |
| hrw | sean-k-mooney: and then you review https://review.openstack.org/#/c/545034/ to have slots for hotplug ;d | 15:46 |
| kashyap | You can either set it via the machine type Nova image metadat property | 15:46 |
| kashyap | Or via the global configuration option | 15:46 |
| hrw | sean-k-mooney: as AArch64 porter I would love to see more and more people moving to Q35 | 15:47 |
| kashyap | sean-k-mooney: FWIW, I sent that (hopefully) clarifying patch -- https://lists.nongnu.org/archive/html/qemu-devel/2018-02/msg04905.html | 15:47 |
| hrw | sean-k-mooney: as then they will share similar issues as we have on aarch64 ;D | 15:50 |
| *** sree_ has joined #openstack-nova | 15:50 | |
| *** sree_ is now known as Guest18191 | 15:51 | |
| *** slaweq_ has quit IRC | 15:53 | |
| *** dtantsur|brb is now known as dtantsur | 15:54 | |
| *** slaweq has joined #openstack-nova | 15:54 | |
| *** Placeed has quit IRC | 15:55 | |
| * hrw off | 15:56 | |
| *** ccamacho has quit IRC | 15:56 | |
| kashyap | sean-k-mooney: So these are the two possibilities with Nova: | 15:57 |
| kashyap | (1) Image metadata property: | 15:57 |
| kashyap | $ openstack image set --property hw_machine_type=x86_64=pc-q35-2.10 Fedora27_Template | 15:57 |
| kashyap | (2) Nova conf: | 15:57 |
| kashyap | [libvirt] | 15:57 |
| kashyap | ... | 15:57 |
| kashyap | hw_machine_type=x86_64=pc-q35-2.10 | 15:57 |
| *** ccamacho has joined #openstack-nova | 15:58 | |
| *** slaweq has quit IRC | 15:58 | |
| *** chyka has joined #openstack-nova | 15:58 | |
| kashyap | sean-k-mooney: The remaining item in Nova is graceful handling of "distributions shipping 'q35' by default". | 15:58 |
| *** salv-orlando has quit IRC | 15:59 | |
| *** Guest18191 has quit IRC | 16:00 | |
| openstackgerrit | Jay Pipes proposed openstack/os-vif master: zuul: Enable functional tests in gate https://review.openstack.org/530961 | 16:03 |
| *** pcaruana has quit IRC | 16:07 | |
| sean-k-mooney | kashyap: yep that all makes sense to me. operator can set in in nova.conf if they want to force it by default or tenant can request it in image metadata if there image requires q35 features to work e.g. vIOMMU support | 16:07 |
| sean-k-mooney | kashyap: hrw it seams pretty strait forward to add Q35 support so a specless blueprint might be sufficent. will ye be at the ptg next week? | 16:09 |
| *** lpetrut has quit IRC | 16:09 | |
| *** cdent has joined #openstack-nova | 16:09 | |
| kashyap | sean-k-mooney: Yes, at least I'll be there | 16:10 |
| kashyap | (And nod to your previous comment, too) | 16:11 |
| *** _pewp_ has quit IRC | 16:11 | |
| *** mdnadeem_ has joined #openstack-nova | 16:12 | |
| * kashyap goes to file a spec-less BP | 16:13 | |
| *** _pewp_ has joined #openstack-nova | 16:14 | |
| *** yamamoto has quit IRC | 16:16 | |
| mriedem | dansmith: left a comment in https://review.openstack.org/#/c/544698/2/nova/tests/unit/db/test_sqlalchemy_migration.py - if you're meh on that, then i'll +2 anyway | 16:16 |
| dansmith | mriedem: I'm pretty meh | 16:19 |
| *** masuberu has quit IRC | 16:21 | |
| *** udesale has quit IRC | 16:22 | |
| *** mdnadeem_ has quit IRC | 16:22 | |
| *** slaweq has joined #openstack-nova | 16:23 | |
| efried | jaypipes, cdent: edleafe: mriedem found a discrepancy in the update_provider_tree spec that I'd like to discuss and clear up. | 16:23 |
| cdent | was just reading that | 16:23 |
| efried | Here's the scenario: my compute node is aggregated with a sharing provider. The sharing provider happens to be part of its own tree. We want the sharing provider itself to be included in the ProviderTree that u_p_t gets - that's for sure. | 16:23 |
| *** r-daneel has quit IRC | 16:23 | |
| efried | But | 16:23 |
| efried | Do we want to include the sharing provider's whole tree? Or just the provider itself, as a "root" even if it isn't really a root in its own tree? | 16:24 |
| efried | jaypipes: I want to say you and I discussed this briefly when I was writing the code and decided on the latter, but thinking through it again, I'm not convinced that's The Right Thing. | 16:25 |
| edleafe | efried: sounds like you have a huge recursion problem if you include trees of trees of trees of ... | 16:25 |
| mriedem | i assume the compute driver got an n-1 view of things it doesn't control | 16:25 |
| mriedem | edleafe: yes agree | 16:25 |
| jaypipes | mriedem: I don't understand what you mean by an "n-1 view" | 16:26 |
| efried | edleafe: Well, we have explicit boundaries. We definitely don't go and spider out to the sharing provider's aggregate-associated providers. | 16:26 |
| mriedem | 1 removed | 16:26 |
| efried | ^ that | 16:26 |
| mriedem | so the virt driver gets the compute node + the sharing provider aggregate, but none of that things children | 16:26 |
| edleafe | efried: another way to look at it: would the compute node ever need to know about /interact with any of the shared provider's tree other than the SP itself? | 16:26 |
| efried | Arguably the unrelated bits of the sharing provider's tree are n-2 - but I'm not sure that's a horrible thing. | 16:26 |
| *** acormier_ has quit IRC | 16:26 | |
| *** rmart04 has quit IRC | 16:26 | |
| cdent | I think edleafe's question is the crux. | 16:27 |
| efried | edleafe: No. But. I don't feel great about the caveat that it would show up as a root even if it's not a root. Rather have more information (that's not used) in the tree than have some of it be inaccurate? | 16:27 |
| mriedem | if the virt driver shouldn't mess with stuff outside of the compute node's tree, it should just be able to see that there is a relationship, right? but not the whole other thing's tree, lest the virt driver gets the idea it has access to update that other thing (which it shouldn't do) | 16:27 |
| *** slaweq has quit IRC | 16:27 | |
| efried | Yeah, I agree with that. | 16:27 |
| *** slaweq has joined #openstack-nova | 16:28 | |
| mriedem | i can't tell if it being a root or not matters | 16:28 |
| efried | That's the thing. | 16:28 |
| efried | We *do* want the virt to be able to mess with the sharing provider. | 16:28 |
| mriedem | if my virt driver shouldn't mess with the sharing provider, why do i care if it's a root or not? | 16:28 |
| mriedem | why? | 16:28 |
| jaypipes | mriedem: this goes back to the powervm virt driver structure. | 16:28 |
| cdent | efried: yes we do, but for those sharing providers that we want to mess with, do we want them to existing in a nested hieararchy? I would think/hope not. | 16:28 |
| jaypipes | mriedem: they want the ability to manage storage pools in their virt driver. | 16:28 |
| efried | As one example, yes. I think VCenter also had an example. | 16:28 |
| cdent | vcenter's example is basically the same | 16:29 |
| *** slaweq has quit IRC | 16:29 | |
| edleafe | efried: if the shared provider is the one with the inventory, it would be the one we allocate against, no? Why do we need its root provider? | 16:29 |
| * cdent points at edleafe | 16:29 | |
| mriedem | the spec says, "Note, | 16:29 |
| mriedem | however, that it may contain providers not directly owned/controlled by the | 16:29 |
| mriedem | compute host. Care must be taken not to remove or modify such providers | 16:29 |
| mriedem | inadvertently." | 16:29 |
| jaypipes | edleafe: the virt driver doesn't allocate. | 16:29 |
| dansmith | mriedem: rc2 is tagged, yeah? are we good to land the compute rpc thing now? | 16:29 |
| efried | mriedem: Yes, there's that too. | 16:29 |
| sean-k-mooney | jaypipes: just regarding those 2 os-vif changes you +2'd i am happy to merge them too but i think there may be an alternitive way to adress https://review.openstack.org/#/c/531358/ that would also work on centos but it might be tricky to do in the gate so ill run it by ye in dublin. | 16:29 |
| mriedem | dansmith: sure | 16:29 |
| mriedem | dansmith: i haven't looked at it yet | 16:29 |
| edleafe | jaypipes: but I thought we were talking placement | 16:30 |
| *** lucas-hungry is now known as lucasagomes | 16:30 | |
| jaypipes | edleafe: we're talking virt driver's use of the ProviderTree struct | 16:30 |
| dansmith | mriedem: okay, it's fairly straightforward and would like to land that soon before we break it of course | 16:30 |
| efried | That's for e.g. sean-k-mooney's cases when there's like a bandwidth provider that's owned by neutron, but is part of the compute node's tree structure. | 16:30 |
| mriedem | dansmith: i am now in update_provider_tree land | 16:30 |
| jaypipes | sean-k-mooney: feel free to approve them then and we'll discuss in Dublin | 16:30 |
| edleafe | jaypipes: sure, but we're also talking about shared providers linked to a compute node | 16:31 |
| dansmith | mriedem: good luck with that | 16:31 |
| *** slunkad_ has quit IRC | 16:31 | |
| sean-k-mooney | jaypipes: cool. i need to talk to stephen about how to add custom jobs via zullv3. i think i understand but i would like to get a linux bridge job there also | 16:31 |
| cdent | can we, at least temporarily, make guideance that shared providers don't nest? Does that get us further down the road without having to predict everything? | 16:32 |
| mriedem | efried: jaypipes: it doesn't seem like a good idea for the nova virt driver to be modifying things that aren't directly under nova's control, like compute resources - because if nova says it can mess with storage and network sharing providers, then cinder/neutron can also be mucking with those right? and we have a fun split brain problem. | 16:32 |
| efried | cdent: It would, yes. | 16:32 |
| kashyap | sean-k-mooney: I need to run to my Dutch class now; will add the blueprint tomm or later when I get back. | 16:32 |
| *** yamamoto has joined #openstack-nova | 16:32 | |
| cdent | or rather, if they do nest, we don't care about it | 16:32 |
| edleafe | ^^ that | 16:32 |
| cdent | because all we really need to know (for now) is that they are there | 16:32 |
| efried | mriedem: Up to this point, virt.PowerVM has full control over the Shared Storage Pool. We want to keep it that way in Placement-land. | 16:33 |
| jaypipes | mriedem: I'm not saying that it's my *preferred* architecture for a virt driver. but it is what it is.. | 16:33 |
| sean-k-mooney | efried: at least for the bandwidth case the provider would not be a shared provider but it was going to be associated to the neutron agent by an aggregate | 16:33 |
| efried | sean-k-mooney: Okay, maybe it was neutron owning the NICs/PFs? | 16:34 |
| mriedem | efried: are those SSP's restricted to a single nova-compute service? | 16:34 |
| sean-k-mooney | efried: in that case also not shared | 16:34 |
| mriedem | or can multiple virt drivers be monkeying with the same SSP? | 16:34 |
| efried | mriedem: The latter. We recognize and accept the caveats of co-management. | 16:34 |
| sean-k-mooney | neutron would be creating RPs in the compute node tree in both cases however | 16:34 |
| efried | ^ | 16:34 |
| sean-k-mooney | mriedem: SSP? | 16:35 |
| mriedem | shared storage pool | 16:35 |
| efried | sean-k-mooney: Shared Storage Pool, PowerVM style in this case. | 16:35 |
| sean-k-mooney | oh ya ok | 16:35 |
| mriedem | ok so the spec says the virt driver can see these other things but shouldn't modify them, but then you're saying you totally do want to modify them and that's the intent | 16:36 |
| efried | mriedem: No, those are different things. | 16:36 |
| mriedem | in other words, update_provider_tree is a blank check to the virt driver | 16:36 |
| mriedem | as we've seen with the RT managing allocations, | 16:37 |
| mriedem | this is probably goign to end badly | 16:37 |
| *** yamamoto has quit IRC | 16:37 | |
| efried | The networky examples in sean-k-mooney's camp are going to be providers in the tree that's rooted at the compute node RP. Those are the ones virt should avoid mucking with. | 16:37 |
| sean-k-mooney | efried: oh speaking of PowerVM as an aside it support PCI devices corret but does not expose pci address am i rembering that correctly. i mentioned it on stephenfin's numa aware vswitch spec but was not sure if i remembered correctly | 16:37 |
| *** acormier has joined #openstack-nova | 16:37 | |
| efried | sean-k-mooney: I haven't made it all the way through that spec yet, but will look out for that, thanks. | 16:38 |
| mriedem | so non-compute child resource providers - don't touch; non-compute shared aggregate providers, go nuts | 16:38 |
| *** awaugama has joined #openstack-nova | 16:38 | |
| *** ttsiouts_ has joined #openstack-nova | 16:38 | |
| efried | mriedem: Blank check, sort of. It's going to be important for the virt driver to understand what it's doing with its providers. | 16:38 |
| mriedem | do you have an example of what a virt driver would need to do with a shared provider aggregate? | 16:39 |
| efried | mriedem: The caveats are just caveats. If a virt driver figures out some way it's going to co-manage the providers neutron creates... that sounds weird but doable. | 16:39 |
| sean-k-mooney | mriedem: well it depends for the neutron created resouce providers i had envisioned the nova conductor makeing the claims against them | 16:39 |
| sean-k-mooney | mriedem: that said nova could delegate that to neutron as part of the port bind also | 16:40 |
| mriedem | sean-k-mooney: are you talking about the bw based scheduling spec? | 16:40 |
| mriedem | we want any kind of 'claim' happening before we hit a compute, which is where port binding happens | 16:40 |
| sean-k-mooney | mriedem: ya. that was one of the open questions should nova claim the bandwith or tell neutron to do it as part of port bining which we want to move to condutor | 16:41 |
| mriedem | port binding was never moving to conductor | 16:41 |
| efried | mriedem: Sure: we're polling the SSP to get available capacity for inventory reporting. If that changes (disk is added, disk goes offline, cluster node tanks, whatever), we want to modify the SSP's inventory accordingly. In this example, whichever virt gets there first, wins. | 16:41 |
| mriedem | port creation was moving to conductor | 16:41 |
| sean-k-mooney | mriedem: it was as part of the multi port binding spec | 16:41 |
| jaypipes | sean-k-mooney: port binding != resource allocation. I'd rather have the conductor do the resource allocation steps than the nova-compute during port-binding | 16:41 |
| *** moshele has joined #openstack-nova | 16:42 | |
| mriedem | jaypipes: agree, otherwise we have a late claim on the wrong host and we fail | 16:42 |
| sean-k-mooney | jaypipes: today yes port-binding happens on compute but intent is for it to move to conductor before we call the compute node | 16:42 |
| jaypipes | mriedem: and we're back to square one again. | 16:42 |
| *** jpena is now known as jpena|brb | 16:42 | |
| mriedem | sean-k-mooney: i don't remember ever talking about moving port binding to conductor | 16:42 |
| sean-k-mooney | let me see if i can get the relevent part of the spec | 16:42 |
| *** slaweq has joined #openstack-nova | 16:42 | |
| mriedem | in the long ago, johnthetubaguy was working on moving port *creation* to conductor | 16:42 |
| efried | To cdent's point, would y'all be on board with imposing a restriction in the code that MISC_SHARES_VIA_AGGREGATE and tree-ness are mutually exclusive? | 16:43 |
| jaypipes | mriedem: and that is resource allocation, not port binding... | 16:43 |
| * johnthetubaguy nodes longingly | 16:43 | |
| johnthetubaguy | s/nodes/nods/ | 16:43 |
| efried | Freudian slip ^ :) | 16:43 |
| johnthetubaguy | :) | 16:43 |
| jaypipes | efried: I don't really see the need, frankly... | 16:43 |
| cdent | efried: I think it's probably okay to not enforce it. just don't got parent or child hunting in u_p_t | 16:43 |
| mriedem | jaypipes: port creation is resource allocation? in what way? | 16:44 |
| mriedem | besides quota | 16:44 |
| efried | jaypipes: It's not a *need* per se; would just allow us to kick this can down the road. | 16:44 |
| jaypipes | efried: just don't ever search deeper than a single node level for any sharing providers | 16:44 |
| johnthetubaguy | it was nice for case where you pick the correct IP segment for routed networks | 16:44 |
| *** ttsiouts_ has quit IRC | 16:44 | |
| johnthetubaguy | where placement deals with the IP resources, that may be limited in the case of public ones | 16:44 |
| jaypipes | mriedem: what johnthetubaguy said. | 16:44 |
| jaypipes | mriedem: but it's not important right now. | 16:45 |
| jaypipes | mriedem: I would be fine just mandating that sharing provider information gathering never exceeds the single level of the sharing provider. | 16:45 |
| sean-k-mooney | mriedem: jaypipes https://git.openstack.org/cgit/openstack/nova-specs/tree/specs/queens/approved/neutron-new-port-binding-api.rst#n139 | 16:45 |
| efried | jaypipes: Okay, but do we include the sharing provider's whole tree or not? | 16:45 |
| mriedem | efried: i'd say no | 16:45 |
| cdent | no | 16:46 |
| efried | I'm worried about the future impact of spoofing a child as a root | 16:46 |
| mriedem | efried: i personally don't think the virt driver should be managing all of this stuff | 16:46 |
| *** bhujay has quit IRC | 16:46 | |
| jaypipes | sean-k-mooney: oh, that's for live migration... yeah. | 16:46 |
| cdent | efried: just because you don't include the info, doesn't mean it isn't out there | 16:46 |
| mriedem | i mean, we've talked about not wanting nova to orchestrate everything in the cloud, and this is a giant step (it seems to me at least) in that direction of orchestratoin | 16:46 |
| jaypipes | cdent: the truth is out there, Scully. | 16:46 |
| sean-k-mooney | jaypipes: ya but the intent was to do the port binding always in the conductor form that point on. both binding and creation actully | 16:46 |
| cdent | it's all lies jaypipes | 16:46 |
| efried | cdent: Well, that's kind of the issue, because right now, we have no way in ProviderTree of creating a root with a parent_uuid | 16:46 |
| mriedem | sean-k-mooney: https://git.openstack.org/cgit/openstack/nova-specs/tree/specs/queens/approved/neutron-new-port-binding-api.rst#n139 is not port creation | 16:47 |
| jaypipes | efried: no, we should not. the "whole tree" is clearly more than "the single level of the node". | 16:47 |
| sean-k-mooney | mriedem: right it happens after port creation | 16:47 |
| mriedem | sean-k-mooney: that spec has no reliance on johnthetubaguy's work to move port creation to the super-conductor | 16:47 |
| mriedem | as far as i remember anyway | 16:47 |
| mriedem | it's for live migration, where we already have the port created on a live vm | 16:48 |
| cdent | efried: all we want to do is represent a provider, in tree. you're saying that if a provider has a parent we can't _not_ include the parent in the provider tree? | 16:48 |
| *** slaweq has quit IRC | 16:48 | |
| mriedem | john's thing was for picking the correct host for routed networks | 16:48 |
| efried | cdent: No, I'm saying if we do include it, we have to lie about its parentage. | 16:48 |
| cdent | no we don't. we just don't have it | 16:48 |
| sean-k-mooney | mriedem: that spec was written with live migration but for bandwitdh based sceduling we needed to move port/creation to the conductor too so we can read the bandwith request form the neutron port before hitting placement | 16:48 |
| *** damien_r has quit IRC | 16:48 | |
| efried | I suppose that's one way of looking at it. | 16:49 |
| mriedem | sean-k-mooney: sure, the bw based scheduling spec is a whole other clusterfuck | 16:49 |
| mriedem | sean-k-mooney: the live migration one for port binding is pretty clear | 16:49 |
| cdent | The ProviderTree is a representation of what we care about, now. It is not the whole truth? | 16:49 |
| cdent | I would hope that it is just whatever is useful. | 16:49 |
| sean-k-mooney | mriedem: maybe we did not explictly say it in either i guess i was jsut under the impressions we had discussed makeing that change for all code paths | 16:49 |
| mriedem | sean-k-mooney: the bw based scheduling spec has at least like 3 major dependencies | 16:50 |
| efried | cdent: But please let's not make the argument that we shouldn't include the non-owned descendants of the compute host. | 16:50 |
| mriedem | and at this rate will get done in maybe the next openstack A release | 16:50 |
| *** jackie-truong has joined #openstack-nova | 16:50 | |
| *** AlexeyAbashkin has quit IRC | 16:50 | |
| mriedem | sean-k-mooney: the dep tree within just nova is already in LP https://blueprints.launchpad.net/nova/+spec/bandwidth-resource-provider | 16:51 |
| efried | cdent: We have literally no way of knowing which ones those would be. | 16:51 |
| mriedem | that doesn't include changes needed in neutron | 16:51 |
| sean-k-mooney | mriedem: well we could strip it donw a little but ya i know its currently state it need many things like nova neutron negociation via os-vif object which is very nice but strictly required | 16:51 |
| mriedem | you mean but *not* strictly required? | 16:51 |
| sean-k-mooney | sorry yes | 16:52 |
| cdent | efried: my feeling is that the core meaning of "ProviderTree" is "this compute node and its children". That it includes auxilliary things like shared providers is bonus. | 16:52 |
| mriedem | dansmith: are the CI failures in https://review.openstack.org/#/c/543580/ all unrelated? | 16:53 |
| sean-k-mooney | the os-vif object depency was there to not have yet another unversioned dictionary of stings going back and fort, it could be done without that chage however | 16:53 |
| mriedem | dansmith: all the grenade jobs failed there? | 16:53 |
| efried | cdent: Acknowledging that the virt driver at least needs to be able to *see* (if not control) the sharing providers, but isn't allowed to talk to placement, I'm still asserting it's a requirement, not a bonus. | 16:53 |
| mriedem | i think seeing that they are there is OK | 16:54 |
| mriedem | like, if i have a compute node tree and see that it's getting it's disk_gb from a shared provider, that's good | 16:54 |
| mriedem | that's something the RT doesn't know about today, and is a problem | 16:54 |
| cdent | efried: it's a requirement for the interface which u_p_t wants to provide to be able to do that. That it is being done within the confines of the ProviderTree object is a matter of convenience (and a reasonably good choice). | 16:54 |
| efried | ptaytah ptahtah, cool. | 16:54 |
| efried | mriedem: For those other two comments, do you want me to call out hypervisor_hostname and ironic explicitly? | 16:55 |
| cdent | efried: you can understand why "something that is not in this tree" feel like it doesn't fit in the definition of ProviderTree (strictly from a naming standpoint)? | 16:55 |
| mriedem | efried: no, just verifying my understanding | 16:55 |
| efried | mriedem: And for the sharing tree issue, do you still want some ascii art? | 16:55 |
| dansmith | mriedem: I had only looked at the first one and it's been a while so figured a recheck, let me look at the others | 16:55 |
| efried | cdent: Yeah. But I ain't proposing a patch to rename it to ProviderCopse | 16:56 |
| mriedem | dansmith: http://logs.openstack.org/80/543580/1/check/neutron-grenade/f675ec2/logs/screen-n-api.txt.gz?level=TRACE#_Feb_12_22_21_47_795180 | 16:56 |
| mriedem | that's real | 16:56 |
| cdent | ooooh. nice name! Then we can have free roaming agents doing copicing. | 16:56 |
| dansmith | ah yup | 16:57 |
| mriedem | efried: well the prose is confusing | 16:57 |
| efried | mriedem: I'm gonna reprose it. | 16:57 |
| efried | Guess I'll do that and see if you think it's enough. | 16:57 |
| mriedem | efried: if we say that the provider tree just going to see the sharing provider and not if it's a root or none of its children (if it can even have children, but i don't see why it couldn't), then it's probably important | 16:57 |
| efried | I didn't quiite parse that, but I think I get what you're saying. | 16:58 |
| * efried edits... | 16:58 | |
| cdent | bbl | 16:58 |
| *** cdent has quit IRC | 16:58 | |
| *** slaweq has joined #openstack-nova | 17:00 | |
| efried | Will future ironic model each ironic node as a separate tree in its own right? Or will they always be children/subtrees of the compute host? | 17:01 |
| mriedem | https://awwapp.com/b/u8ghqr9rr/ | 17:01 |
| mriedem | efried: can you see ^? | 17:01 |
| *** trinaths has joined #openstack-nova | 17:01 | |
| efried | mriedem: yes | 17:01 |
| *** slaweq_ has joined #openstack-nova | 17:01 | |
| mriedem | efried: in that diagram i'm asserting that the provider tree that the virt driver sees is in blue | 17:01 |
| efried | mriedem: And yes, that's the idea. | 17:01 |
| efried | Yup. | 17:01 |
| *** yamamoto has joined #openstack-nova | 17:02 | |
| efried | or even... | 17:02 |
| mriedem | cdent: jaypipes: edleafe: agree? ^ | 17:02 |
| sean-k-mooney | efried: i would assume future ironic might model each chassie as a resouce provierd of inventoies of custome_baremetal_node object but i done no about a RP per bermtal node | 17:02 |
| *** chyka has quit IRC | 17:02 | |
| mriedem | efried: there is no resource provider for the compute host, just the nodes | 17:03 |
| dansmith | mriedem: oh, I bet this is because we don't have grenade mappings for queens yet, | 17:03 |
| mriedem | so they'd be separate | 17:03 |
| dansmith | mriedem: so we're actually upgrading from pike to rocky here | 17:03 |
| efried | mriedem: Okay. We don't have a way to handle that yet FYI. | 17:03 |
| dansmith | sdague: can you remind me what I need to do for that? | 17:03 |
| mriedem | dansmith: i see a stable/queens branch for grenade now, | 17:03 |
| mriedem | so maybe it was just done after that CI run and your recheck will be happy | 17:04 |
| dansmith | mriedem: yeah, but logs show we have pike computes in there | 17:04 |
| sean-k-mooney | efried: each ironic node bing its own provider tree? | 17:04 |
| efried | mriedem: But once we do, it'll be a true statement that the ProviderTree can have multiple full actual trees. | 17:04 |
| dansmith | mriedem: well, maybe, but I thought we had to do something | 17:04 |
| efried | sean-k-mooney: Yeah | 17:04 |
| *** belmoreira has quit IRC | 17:04 | |
| sean-k-mooney | efried: that should just be a simple virt driver change no? | 17:04 |
| efried | no | 17:04 |
| dansmith | mriedem: zuul backlog is 1249 items long, so it hasn't even started running yet :/ | 17:04 |
| efried | sean-k-mooney: rt will have no way to map the ironic node back to the compute host for purposes of scheduling. | 17:05 |
| *** ragiman has quit IRC | 17:05 | |
| sean-k-mooney | efried: you could use an aggregate for that | 17:05 |
| efried | s/rt/conductor/ I guess | 17:05 |
| efried | sean-k-mooney: Could. But like I say, we don't have that yet. | 17:05 |
| efried | And that would be an interesting case for aggregated provider trees that aren't sharing providers. | 17:06 |
| *** slaweq_ has quit IRC | 17:06 | |
| *** yamamoto has quit IRC | 17:06 | |
| efried | TBH, I don't know how ironic plans to model in u_p_t. mgoddard around? | 17:07 |
| jroll | efried: we haven't made any, that I know of | 17:07 |
| sean-k-mooney | efried: ironic may also want to use either nested resouce providers or aggregates to model node to chassis relationships for blade systems too | 17:07 |
| * jroll hasn't been engaged at all, and not really watching the u_p_t stuff much | 17:07 | |
| jroll | s/at all/on this topic/ | 17:08 |
| *** andreas_s has quit IRC | 17:08 | |
| *** moshele has quit IRC | 17:08 | |
| jroll | jaypipes may also have some thoughts on it, though | 17:08 |
| *** andreas_s has joined #openstack-nova | 17:09 | |
| *** links has quit IRC | 17:09 | |
| *** chyka has joined #openstack-nova | 17:09 | |
| sean-k-mooney | efried: i genally thing of aggregate just as arbitry groupings of RPs and shareing RPs as a special subset of that. in genneral i think aggregates are usfull for far mor thing that are not sharing related the sharing related uscases | 17:10 |
| efried | sean-k-mooney: I agree that there are plenty of non-sharing use cases for aggregates. | 17:11 |
| efried | sean-k-mooney: But as currently conceived (per recent refinement) we don't pull in non-sharing aggregated providers at all. | 17:11 |
| mordred | mriedem: is it possible to query nova as an end-user to find out what the default AZ is? | 17:11 |
| jroll | efried: so, in an ideal world, we just... don't care about mapping an ironic node to a compute host for scheduling purposes. because, any compute host can talk to ironic to do the build | 17:12 |
| mriedem | mordred: as in https://developer.openstack.org/api-ref/compute/#get-availability-zone-information doesn't tell you which is the default in config? | 17:12 |
| efried | jroll: Coolcool. But n-cond still needs to schedule a deploy to *some* n-cpu somewhere. | 17:13 |
| *** jackie-truong has quit IRC | 17:14 | |
| efried | jroll: So if the ironic nodes are independent providers (independent of their compute host) how does it know where? | 17:14 |
| *** andreas_s has quit IRC | 17:14 | |
| jroll | efried: agree, and I'm not sure we want to special case it. but if we did: host = random(get_ironic_compute_hosts()) | 17:14 |
| mriedem | mordred: doesn't look like it http://paste.openstack.org/show/677614/ | 17:14 |
| jroll | efried: I realize now I don't know how that works with placement. in the old world, both the host and node are in the compute_nodes record | 17:15 |
| mriedem | mordred: well, i guess if there is only 1 'available' zone then you know what the default is :) | 17:15 |
| efried | jroll: Oh, so all ironic nodes under an n-cond can be managed by any n-cpu under that same n-cond? | 17:15 |
| jroll | efried: in theory, yes, we don't do that in reality today | 17:15 |
| jroll | efried: but we do currently shuffle them between ironic n-cpu hosts that are up, as the hosts go up and down | 17:16 |
| *** felipemonteiro has joined #openstack-nova | 17:18 | |
| efried | jroll: But don't ops also always deploy specific ironic nodes? | 17:18 |
| jroll | efried: as in "nova boot this-specific-machine"? | 17:19 |
| efried | Yeah; maybe I'm misremembering that. | 17:19 |
| jroll | nope, not at all | 17:19 |
| efried | k | 17:19 |
| jroll | some do - and they use things like scheduler hints | 17:19 |
| efried | Rightright, that's what I was thinking - that it's a requirement to be able to do so. | 17:20 |
| jroll | other installations are just another cloud, with the hypervisors missing | 17:20 |
| jroll | efried: the "requirement" part of that depends who you ask :) | 17:20 |
| jroll | I'm not sure it works upstream today | 17:20 |
| efried | So yeah, I have absolutely no idea how we're planning to model ironic in Placement-land. But I suspect there will have to be some additional affordance outside of what we're doing in u_p_t. | 17:20 |
| efried | Because right now, if you tried to model each ironic node as its own root/tree, it wouldn't work. | 17:21 |
| efried | (by "right now" I mean "with NRP and u_p_t implemented as specced") | 17:21 |
| *** itlinux has joined #openstack-nova | 17:22 | |
| *** jpena|brb is now known as jpena | 17:22 | |
| jroll | yeah, I'm not up to speed on the nested things | 17:22 |
| *** tssurya has quit IRC | 17:22 | |
| jroll | I suspect jay has ideas here, dunno | 17:23 |
| *** slaweq_ has joined #openstack-nova | 17:25 | |
| *** acormier has quit IRC | 17:25 | |
| *** acormier has joined #openstack-nova | 17:25 | |
| openstackgerrit | Eric Fried proposed openstack/nova-specs master: Update Provider Tree https://review.openstack.org/540111 | 17:27 |
| efried | mriedem: See how that grabs ya ^ | 17:27 |
| efried | jaypipes, edleafe, cdent ^ | 17:27 |
| *** cfriesen has joined #openstack-nova | 17:28 | |
| *** slaweq_ has quit IRC | 17:29 | |
| edleafe | efried: stop grabbing me! | 17:32 |
| efried | mriedem: Updated that diagram a little bit too https://awwapp.com/b/u8ghqr9rr/ | 17:32 |
| efried | edleafe: Dangit, there goes my career 40 years from now. | 17:33 |
| mriedem | efried: then i think you definitely need a diagram in the spec to give an example | 17:34 |
| jroll | efried: I see nodename everywhere in this spec, that signals to me that one tree per ironic node is expected | 17:34 |
| mriedem | of what's in vs what's out when the driver gets the provider tree object | 17:34 |
| *** rmart04 has joined #openstack-nova | 17:34 | |
| efried | jroll: That's probably just a lack of precision on my part more than anything. | 17:34 |
| efried | jroll: I still don't understand the difference between nodename, hypervisor_hostname, and host.name. | 17:35 |
| *** lpetrut has joined #openstack-nova | 17:35 | |
| efried | jroll: But I wrote the code first, and 'nodename' was the var that made sense to pass around in there, so that's probably how it ended up in the spec. | 17:35 |
| jroll | efried: nodename ~= hypervisor_hostname, I'm not sure what host.name refers to | 17:36 |
| *** lpetrut has quit IRC | 17:36 | |
| *** lpetrut has joined #openstack-nova | 17:36 | |
| efried | nodename == hypervisor_hostname, except for ironic where they're never equal; but hypervisor_hostname == host.name always ??? Or something ??? | 17:36 |
| jroll | in the ironic world, nodename == hypervisor_hostname, for sure. compute_node.host is the different one (and the one that actually looks like a hostname) | 17:37 |
| efried | jroll: When you say ~= is that "regular expression match" à la Perl, or "not equal" à la other things? | 17:37 |
| efried | oh, okay. | 17:37 |
| efried | So that makes zero sense to me, that ironic node name should be the same as the hypervisor host's name. So the latter is actually a total misnomer. | 17:38 |
| jroll | efried: equal, but I'm not 100% sure on weird drivers, so I'm going with approximately equal :) | 17:38 |
| efried | gotcha | 17:38 |
| jroll | sure, it's a poorly named thing. originally it was for things like xen, where compute.host (how you reach n-cpu) might be separate from the actual hypervisor (how you reach xenapi) | 17:39 |
| jroll | s/hypervisor/hypervisor_hostname/ | 17:39 |
| mordred | mriedem: :) ... and yah - the lack of info about default in config in the az list is, I think, the thing | 17:40 |
| mordred | mriedem: came up in some discussions around nodepool and azs - thanks for confirming | 17:41 |
| mriedem | mordred: from what i remember, there are actually >1 nova config options related to a 'default' az | 17:41 |
| mordred | mriedem: awesome. and of course there are :) | 17:41 |
| openstackgerrit | sean mooney proposed openstack/nova master: Change 'InstancePCIRequest' spec field https://review.openstack.org/449257 | 17:41 |
| openstackgerrit | sean mooney proposed openstack/nova master: Add Neutron port capabilities to devspec in request https://review.openstack.org/451777 | 17:41 |
| openstackgerrit | sean mooney proposed openstack/nova master: Format NIC features using os-traits definitions https://review.openstack.org/466051 | 17:41 |
| openstackgerrit | sean mooney proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/507481 | 17:41 |
| mriedem | default_availability_zone and default_schedule_zone | 17:42 |
| *** tidwellr has quit IRC | 17:42 | |
| jroll | efried: as far as the one vs many trees for ironic... _update_available_resource is called once per ironic node, not once per nova-compute host. (and thus the same for get_inventory and such). in old-nova terms, it's called once per compute_node record, of which many may exist per compute service. does that make sense? | 17:42 |
| mordred | mriedem: at root is that currently nodepool gets a list of azs and the balances across them by default - which has led to unexpected behavior before (citycloud had an az specifically for a type of flavor and were confused why were launching nodes there) | 17:42 |
| jroll | efried: I'll do a full review of the spec and drop some comments there, if that works better for you | 17:42 |
| mordred | mriedem: but on clouds with more than one az where they're all the same, if we DON'T balance across them we wind up with usage only in the default zone | 17:43 |
| jroll | efried: I guess I'm mostly curious if the "one tree" constraint you're thinking of is "one root per compute service" or "one root per resource provider"? | 17:43 |
| efried | jroll: That would be great, though I don't know that we're going to get much satisfaction out of that. | 17:43 |
| mriedem | mordred: ok, and in the citycloud case, as a user, you don't have access to see the flavor linked to the AZ via host aggregate | 17:44 |
| mriedem | because users can't see host aggregates | 17:44 |
| efried | jroll: I explicitly state in the spec that update_provider_tree is being called once per ironic node, BUT that doesn't really affect how the result is used. | 17:44 |
| mriedem | mordred: in other words, if using this flavor, don't specify AZ | 17:44 |
| mordred | mriedem: yup | 17:45 |
| mriedem | in queens they can at least now put a description on the flavor to say that, but your tooling likely doesn't care about that until you've already tried debugging what's going wrong | 17:45 |
| mordred | mriedem: so there's a few potentially missing pieces of metadata .. however, clarkb has been advocating that we stop trying to balance across azs by default and instead require someone to configure az use explicitly | 17:46 |
| mriedem | and then you have to put in special logic for that flavor on that cloud | 17:46 |
| *** slaweq_ has joined #openstack-nova | 17:46 | |
| mordred | mriedem: exactly. when if there was some metadata, I could even validate the flavor/az combo in shade/openstacksdk | 17:46 |
| mordred | before we even bother making an API call | 17:46 |
| mordred | "dude, this combo won't work" | 17:46 |
| jroll | efried: hm, having trouble finding that note, but also seeing now that this code is mostly done, so maybe I need to back up and look at the bigger picture | 17:46 |
| mriedem | mordred: so do you end up getting a novalidhost in the citycloud case with that flavor and some other AZ? | 17:47 |
| efried | jroll: I was not trying to (actually "trying not to") predict how ironic is going to model. What I'm trying to say is, if ironic wants to model with a separate tree/root per ironic node, this spec is insufficient to handle it. | 17:47 |
| *** yamamoto has joined #openstack-nova | 17:47 | |
| mordred | mriedem: I don't think so - no - I think what happened was we got scheduled on hardware that was intended for something else (they weren't expecting people to request anything in that az unless someone told them to) so we got nodes that had messed up networking or something else | 17:48 |
| jroll | efried: sure, I'm trying to figure out what about this spec precludes doing such a thing | 17:48 |
| efried | jroll: https://review.openstack.org/#/c/540111/5/specs/rocky/approved/update-provider-tree.rst L57-61 | 17:48 |
| efried | jroll: It's not anything about the spec. It's the way the rest of Nova is shaped currently. | 17:48 |
| jroll | efried: ah, that block. I guess that reads to me as "one root per compute node", nothing that compute service != compute node. I guess I need to read the code. | 17:50 |
| efried | jroll: Actually, the freshly-updated text in the note on L69-76 *does* preclude the multiple-trees-that-aren't-sharing thing. | 17:50 |
| jroll | hmm | 17:50 |
| *** slaweq_ has quit IRC | 17:51 | |
| jroll | ok, I will dig, I need to learn more about NRP before I can say much else. thanks, efried | 17:51 |
| efried | jroll: Let me know if you need pointers. Much of the code for this is not yet merged. | 17:52 |
| efried | jroll: Pending code is in series starting at https://review.openstack.org/#/c/537648/ | 17:52 |
| jroll | efried: yep, I'm there | 17:52 |
| *** yamamoto has quit IRC | 17:52 | |
| openstackgerrit | Merged openstack/nova stable/ocata: Add 'delete_host' command in 'nova-manage cell_v2' https://review.openstack.org/513721 | 17:53 |
| openstackgerrit | Merged openstack/nova stable/ocata: Fix test_instance_get_all_by_host https://review.openstack.org/516486 | 17:53 |
| *** felipemonteiro has quit IRC | 17:54 | |
| lyarwood | mriedem: back online, yeah I'm around this week, I'll take a look this evening if there's anything left. | 17:54 |
| *** felipemonteiro has joined #openstack-nova | 17:54 | |
| efried | jaypipes: Do you have a couple minutes to help me understand the RT flow for ironic? | 17:59 |
| *** felipemonteiro has quit IRC | 17:59 | |
| *** derekh has quit IRC | 18:00 | |
| jroll | efried: looking through some of this, I'm too far behind on how some of this works to fully discuss why this is or isn't fundamentally broken for ironic today, but I hope to be able to later this week, or worst case in person next week | 18:00 |
| jroll | I can try to help you understand the current flow, if you have questions in mind | 18:00 |
| efried | jroll: Yeah, if you don't mind. | 18:01 |
| efried | I'm trying to understand how the current get_inventory is called. | 18:01 |
| *** mgoddard_ has quit IRC | 18:02 | |
| efried | In the ironic case, there's actually multiple ComputeNode s ? | 18:02 |
| *** ralonsoh has quit IRC | 18:02 | |
| jroll | correct - there is a ComputeNode per ironic node | 18:02 |
| efried | And there's a loop over those, and RT does the update_compute_node stuff for each? | 18:02 |
| jroll | actually, there's an instance of the RT class created for each ComputeNode, IIRC | 18:03 |
| efried | ah, that would explain why I wasn't finding said loop. | 18:03 |
| efried | But ultimately what it means is that we are indeed creating a provider in placement for each ironic node, separately. | 18:03 |
| jroll | that may have changed somewhere | 18:03 |
| jroll | correct | 18:03 |
| efried | Okay, that helps a lot. I need to go through this again and be more precise about using "host" vs "node", and probably reword the stuff in the aforementioned block about nodename. | 18:04 |
| openstackgerrit | Merged openstack/os-vif master: Configure privsep binary https://review.openstack.org/531358 | 18:05 |
| efried | Though TBH, I feel like we've reached the Pareto point with this spec... | 18:05 |
| jroll | efried: ah, it's still a singleton RT. but we call update_available_resource() for each node. this method is called per node: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L7241 | 18:05 |
| efried | jroll: Got it - here's the loop https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L7282 -- thanks. | 18:06 |
| jroll | efried: well, if each *node* may be a root, rather than each *host*, I think we'd be good to go (it would match what we're doing before nested RPs) | 18:06 |
| jroll | yep | 18:06 |
| *** acormier_ has joined #openstack-nova | 18:06 | |
| efried | jroll: Yes, I didn't realize we were already handling separate providers per node. So what I was saying earlier about things we don't handle was a lie (sean-k-mooney mriedem). | 18:07 |
| *** slaweq_ has joined #openstack-nova | 18:07 | |
| *** hongbin has joined #openstack-nova | 18:07 | |
| jroll | gotcha, cool. this seems workable then :) | 18:07 |
| efried | And in fact without further hacking, ironic *gets* a separate root/tree per node - doesn't actually have a choice in the matter :) | 18:08 |
| *** salv-orlando has joined #openstack-nova | 18:08 | |
| jroll | yep! | 18:08 |
| *** yamamoto has joined #openstack-nova | 18:08 | |
| *** acormier_ has quit IRC | 18:08 | |
| efried | Bootstrap-wise, update_provider_tree will get just the root for the node on the initial call; it'll be up to the virt's impl of update_provider_tree if it wants to make child providers under that root or whatever. | 18:08 |
| *** yamamoto has quit IRC | 18:08 | |
| jroll | I think the code is good to go anyway - line 1405 here concerns me a bit https://review.openstack.org/#/c/533821/22/nova/scheduler/client/report.py | 18:09 |
| *** acormier_ has joined #openstack-nova | 18:09 | |
| efried | YES | 18:10 |
| *** acormier_ has joined #openstack-nova | 18:10 | |
| *** acormier has quit IRC | 18:10 | |
| efried | Because I *think* that guy is actually going to contain *all* the trees for *all* the nodes. | 18:10 |
| jroll | yeah, either that or just the last tree operated on | 18:10 |
| jroll | old_tree = self._provider_trees[nodename] :) | 18:10 |
| *** slaweq_ has quit IRC | 18:12 | |
| jroll | because, we don't have an RP representing the compute host to be the root for all of those node RPs, they're all independent. if that makes esense. | 18:12 |
| jroll | s/esense/sense/ | 18:12 |
| efried | jroll: yeah. I think that needs to be fixed in two or three places. | 18:13 |
| jroll | possibly | 18:14 |
| efried | I think it's a bigger problem | 18:14 |
| *** acormier_ has quit IRC | 18:14 | |
| efried | or maybe, as you say, I do that filtering only here. | 18:14 |
| efried | I'm going to have to look at this with fresh eyes, now that I have a better understanding of the flow. | 18:15 |
| jroll | awesome, thanks efried, this was helpful for my understanding too :) | 18:15 |
| *** AlexeyAbashkin has joined #openstack-nova | 18:15 | |
| *** rmart04 has quit IRC | 18:16 | |
| efried | jroll: Your help is much appreciated too. Let's have a Guinness next week. | 18:16 |
| jroll | ++ | 18:16 |
| *** openstackgerrit has quit IRC | 18:18 | |
| *** jpena is now known as jpena|away | 18:19 | |
| *** AlexeyAbashkin has quit IRC | 18:20 | |
| *** openstackgerrit has joined #openstack-nova | 18:24 | |
| openstackgerrit | sean mooney proposed openstack/nova master: Add Neutron port capabilities to devspec in request https://review.openstack.org/451777 | 18:24 |
| openstackgerrit | sean mooney proposed openstack/nova master: Format NIC features using os-traits definitions https://review.openstack.org/466051 | 18:24 |
| openstackgerrit | sean mooney proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/507481 | 18:24 |
| mriedem | this pretty simple bug fix has been hanging out with a +2 for a few months now https://review.openstack.org/#/c/519464/ - simply unquiesce an instance if we quiesced it but creating volume snapshots fails | 18:27 |
| openstackgerrit | sean mooney proposed openstack/nova-specs master: Reintroduced nic feature based scheduling for rocky https://review.openstack.org/545951 | 18:30 |
| *** tesseract has quit IRC | 18:30 | |
| *** r-daneel has joined #openstack-nova | 18:39 | |
| dansmith | mriedem: got that and the two above it | 18:39 |
| dansmith | which seem like obvious should-be-doing-that items | 18:40 |
| *** sdague has quit IRC | 18:40 | |
| openstackgerrit | Merged openstack/nova master: Imported Translations from Zanata https://review.openstack.org/541561 | 18:41 |
| *** mgoddard_ has joined #openstack-nova | 18:44 | |
| *** moshele has joined #openstack-nova | 18:45 | |
| *** mvk_ has quit IRC | 18:48 | |
| *** cfriesen has quit IRC | 18:50 | |
| *** edmondsw has quit IRC | 18:55 | |
| *** edmondsw has joined #openstack-nova | 18:56 | |
| *** lucasagomes is now known as lucas-brb | 18:57 | |
| *** edmondsw has quit IRC | 19:00 | |
| *** felipemonteiro has joined #openstack-nova | 19:02 | |
| *** harlowja has joined #openstack-nova | 19:03 | |
| *** yamamoto has joined #openstack-nova | 19:09 | |
| *** dtantsur is now known as dtantsur|afk | 19:10 | |
| *** jafeha has joined #openstack-nova | 19:10 | |
| *** esberglu has quit IRC | 19:11 | |
| *** jafeha__ has quit IRC | 19:11 | |
| *** jpena|away is now known as jpena|off | 19:14 | |
| openstackgerrit | Eric Fried proposed openstack/nova-specs master: Update Provider Tree https://review.openstack.org/540111 | 19:14 |
| efried | mriedem, jaypipes, edleafe, cdent: I'm no stephenfin, but ^ | 19:15 |
| efried | jroll: ^ | 19:15 |
| * jroll looks | 19:15 | |
| *** slaweq_ has joined #openstack-nova | 19:16 | |
| edleafe | efried: if only you included an ASCII diagram of CN2... | 19:18 |
| efried | edleafe: Eh? Did I not? | 19:19 |
| mriedem | dansmith: thanks | 19:19 |
| edleafe | efried: I meant for the PT when UPT is invoked for CN2 | 19:20 |
| efried | oh. It's symmetrical. Are you feing bunny? | 19:20 |
| edleafe | efried: just busting your chops, of course :) | 19:20 |
| efried | phew | 19:20 |
| efried | consider my chops busted | 19:20 |
| *** slaweq_ has quit IRC | 19:21 | |
| *** damien_r has joined #openstack-nova | 19:21 | |
| openstackgerrit | Merged openstack/nova stable/ocata: libvirt: bandwidth param should be set in guest migrate https://review.openstack.org/519635 | 19:24 |
| *** alexchadin has joined #openstack-nova | 19:24 | |
| openstackgerrit | Merged openstack/python-novaclient master: Fix the docstring for the update method https://review.openstack.org/545819 | 19:24 |
| *** mgoddard_ has quit IRC | 19:25 | |
| *** yamamoto has quit IRC | 19:26 | |
| *** alexchadin has quit IRC | 19:28 | |
| *** tssurya has joined #openstack-nova | 19:28 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Combine error handling blocks in _do_build_and_run_instance https://review.openstack.org/545960 | 19:29 |
| *** esberglu has joined #openstack-nova | 19:29 | |
| *** damien_r has quit IRC | 19:31 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: unquiesce instance on volume snapshot failure https://review.openstack.org/545961 | 19:31 |
| *** alexchadin has joined #openstack-nova | 19:33 | |
| *** mriedem has quit IRC | 19:34 | |
| *** mriedem has joined #openstack-nova | 19:35 | |
| *** alexchadin has quit IRC | 19:36 | |
| *** moshele has quit IRC | 19:36 | |
| *** moshele has joined #openstack-nova | 19:38 | |
| *** moshele has quit IRC | 19:42 | |
| *** slaweq_ has joined #openstack-nova | 19:52 | |
| *** amodi has quit IRC | 19:52 | |
| *** mgoddard_ has joined #openstack-nova | 19:56 | |
| *** slaweq_ has quit IRC | 19:57 | |
| *** lucas-brb is now known as lucasagomes | 19:58 | |
| *** mvk_ has joined #openstack-nova | 19:59 | |
| *** hongbin_ has joined #openstack-nova | 20:01 | |
| *** hongbin has quit IRC | 20:03 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: unquiesce instance on volume snapshot failure https://review.openstack.org/545966 | 20:03 |
| *** alexchadin has joined #openstack-nova | 20:04 | |
| *** alexchadin has quit IRC | 20:04 | |
| *** amodi has joined #openstack-nova | 20:05 | |
| openstackgerrit | Merged openstack/nova stable/ocata: Use proper user and tenant in the owner section of libvirt.xml. https://review.openstack.org/525997 | 20:06 |
| mriedem | dansmith: want to get this simple fixture cleanup patch? https://review.openstack.org/#/c/539758/ | 20:08 |
| mriedem | that will unblock a few other approved changes | 20:08 |
| mriedem | this is the series where bfv failing during scheduling always orphans your volumes | 20:08 |
| mriedem | which sucks | 20:08 |
| mriedem | FYI in case any cores want to shed 1300+ LOC https://review.openstack.org/#/c/544698/ | 20:10 |
| *** acormier has joined #openstack-nova | 20:11 | |
| *** dave-mccowan has quit IRC | 20:14 | |
| *** lucasagomes is now known as lucas-afk | 20:14 | |
| dansmith | mriedem: ack | 20:16 |
| *** moshele has joined #openstack-nova | 20:22 | |
| openstackgerrit | Merged openstack/os-vif master: zuul: Enable functional tests in gate https://review.openstack.org/530961 | 20:22 |
| openstackgerrit | Merged openstack/nova master: unquiesce instance on volume snapshot failure https://review.openstack.org/519464 | 20:22 |
| *** amoralej is now known as amoralej|off | 20:23 | |
| *** janki has quit IRC | 20:28 | |
| *** edmondsw has joined #openstack-nova | 20:30 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: unquiesce instance on volume snapshot failure https://review.openstack.org/545973 | 20:32 |
| *** felipemonteiro_ has joined #openstack-nova | 20:32 | |
| *** awaugama has quit IRC | 20:33 | |
| *** felipemonteiro has quit IRC | 20:35 | |
| *** mgoddard has quit IRC | 20:35 | |
| *** itlinux has quit IRC | 20:36 | |
| *** moshele has quit IRC | 20:40 | |
| *** mgoddard has joined #openstack-nova | 20:42 | |
| *** tssurya has quit IRC | 20:43 | |
| *** tssurya has joined #openstack-nova | 20:47 | |
| *** tidwellr has joined #openstack-nova | 20:48 | |
| *** eharney has quit IRC | 20:49 | |
| *** acormier has quit IRC | 20:50 | |
| *** acormier has joined #openstack-nova | 20:51 | |
| *** mgoddard_ has quit IRC | 20:53 | |
| melwitt | dansmith, mriedem: on https://review.openstack.org/#/c/544698, "We dropped support for aggregates in newton" just means dropped support for the old "main db" aggregates, not dropped support for aggregates altogether, right? | 20:59 |
| mriedem | melwitt: we just deleted the API | 20:59 |
| mriedem | it was an admin-only extension, | 20:59 |
| mriedem | we said, fuck it | 20:59 |
| dansmith | melwitt: heh, right, sorry. I meant we dropped support for this stuff I'm removing :D | 21:00 |
| dansmith | melwitt: if you think it's important I can rev it | 21:00 |
| melwitt | when I first read it I was like O.o | 21:00 |
| melwitt | I'll just put a note to self on there | 21:01 |
| mriedem | dropped the migration compat code | 21:01 |
| mriedem | like for flavors | 21:01 |
| *** mvk_ has quit IRC | 21:01 | |
| melwitt | yeah. I see it in the patch, was just trying to connect the commit message in case there was something really major I was under a rock about | 21:01 |
| *** slaweq_ has joined #openstack-nova | 21:07 | |
| *** ccamacho has quit IRC | 21:12 | |
| *** slaweq_ has quit IRC | 21:12 | |
| *** pchavva has quit IRC | 21:15 | |
| *** moshele has joined #openstack-nova | 21:18 | |
| *** awaugama has joined #openstack-nova | 21:21 | |
| mriedem | dansmith: after reading the bug for https://review.openstack.org/#/c/543970/ can you make sure my comment is correct before I +W? | 21:23 |
| *** andreas_s has joined #openstack-nova | 21:26 | |
| *** AlexeyAbashkin has joined #openstack-nova | 21:26 | |
| *** clayton has quit IRC | 21:28 | |
| *** hongbin has joined #openstack-nova | 21:29 | |
| *** moshele has quit IRC | 21:30 | |
| *** hongbin_ has quit IRC | 21:30 | |
| dansmith | mriedem: replied for posterity, but in short: yep. | 21:30 |
| *** AlexeyAbashkin has quit IRC | 21:31 | |
| *** andreas_s has quit IRC | 21:31 | |
| hrw | sean-k-mooney: will be | 21:31 |
| mriedem | cool; i'll start the backport party | 21:31 |
| *** tssurya has quit IRC | 21:34 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Lazy-load instance attributes with read_deleted=yes https://review.openstack.org/545987 | 21:35 |
| *** rcernin has joined #openstack-nova | 21:36 | |
| *** threestrands has joined #openstack-nova | 21:37 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Lazy-load instance attributes with read_deleted=yes https://review.openstack.org/545988 | 21:37 |
| *** vladikr has quit IRC | 21:37 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: Lazy-load instance attributes with read_deleted=yes https://review.openstack.org/545989 | 21:38 |
| *** vladikr has joined #openstack-nova | 21:38 | |
| *** salv-orl_ has joined #openstack-nova | 21:42 | |
| hrw | sean-k-mooney: added myself to nova etherpad | 21:43 |
| *** salv-orlando has quit IRC | 21:44 | |
| *** felipemonteiro_ has quit IRC | 21:45 | |
| *** felipemonteiro_ has joined #openstack-nova | 21:45 | |
| mriedem | not sure why we'd even be lazy loading instance.system_metadata in that evacuate cleanup path, the db api should manually join it https://github.com/openstack/nova/blob/stable/pike/nova/db/sqlalchemy/api.py#L2141-L2143 | 21:45 |
| *** tidwellr has quit IRC | 21:46 | |
| *** takashin has joined #openstack-nova | 21:48 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: libvirt: Don't VIR_MIGRATE_NON_SHARED_INC without migrate_disks https://review.openstack.org/519636 | 21:50 |
| openstackgerrit | Merged openstack/nova master: Drop extra loop which modifies Cinder volume status https://review.openstack.org/539758 | 21:51 |
| openstackgerrit | Merged openstack/nova master: Store block device mappings in cell0 https://review.openstack.org/544748 | 21:51 |
| openstackgerrit | Merged openstack/nova master: Add functional tests to ensure BDM removal on delete https://review.openstack.org/544747 | 21:52 |
| dansmith | mriedem: the compute drop passed grenade this time | 21:53 |
| dansmith | failed something else, which I'm looking at but.. | 21:53 |
| *** felipemonteiro__ has joined #openstack-nova | 21:53 | |
| *** priteau has quit IRC | 21:53 | |
| dansmith | hmm, rabbit crash maybe? | 21:54 |
| dansmith | http://logs.openstack.org/80/543580/1/check/legacy-tempest-dsvm-cells/e4a862c/logs/screen-n-cpu.txt.gz?level=TRACE#_Feb_19_20_06_46_946074 | 21:54 |
| *** priteau has joined #openstack-nova | 21:54 | |
| mriedem | there are crash reports in http://logs.openstack.org/80/543580/1/check/legacy-tempest-dsvm-cells/e4a862c/logs/rabbitmq/rabbit@ubuntu-xenial-inap-mtl01-0002617609-sasl.txt.gz | 21:56 |
| dansmith | ah sweet | 21:56 |
| *** felipemonteiro_ has quit IRC | 21:57 | |
| *** priteau has quit IRC | 21:59 | |
| mriedem | mmedvede: can we get the pkvm CI to not vote at all on stable/ocata changes? it always fails immediately https://review.openstack.org/#/c/545973/ | 22:05 |
| *** lpetrut has quit IRC | 22:05 | |
| efried | edleafe, melwitt, mriedem, jaypipes, cdent: Specless bp to add ?required=<trait list> to GET /resource_providers ? | 22:06 |
| *** hamzy has quit IRC | 22:07 | |
| * efried forms full sentence... | 22:07 | |
| mriedem | mmedvede: looks like the tests randomly timeout because it takes forever to spawn the actual guest | 22:07 |
| mriedem | 2018-02-19 21:14:56.466 31787 INFO nova.compute.manager [req-21095306-b40d-4762-a083-8510184d386d tempest-ListServerFiltersTestJSON-391319940 tempest-ListServerFiltersTestJSON-391319940] [instance: 5b94a837-eb53-4275-b893-8e51b8d10518] Took 243.80 seconds to spawn the instance on the hypervisor. | 22:07 |
| efried | Would "adding ?required=<trait list> to GET /resource_providers" be considered for a specless blueprint? | 22:07 |
| edleafe | efried: yeah, I think specless is ok. Just clarify in the BP that it will modeled on GET /allocation_candidates | 22:08 |
| mriedem | efried: no | 22:08 |
| mriedem | it's an api change | 22:08 |
| melwitt | efried: I'd have expected an api change to be a spec, albeit a small one in that case | 22:08 |
| efried | ight | 22:08 |
| edleafe | oh geez, yeah, an API change. What they said. | 22:08 |
| *** eharney has joined #openstack-nova | 22:09 | |
| *** slaweq_ has joined #openstack-nova | 22:13 | |
| jaypipes | efried: I thought all API changes required a spec? | 22:14 |
| efried | jaypipes: Apparently that's true; see above. | 22:14 |
| * efried specs | 22:14 | |
| *** hamzy has joined #openstack-nova | 22:16 | |
| *** slaweq_ has quit IRC | 22:18 | |
| *** openstackgerrit has quit IRC | 22:18 | |
| *** priteau has joined #openstack-nova | 22:22 | |
| *** mvk_ has joined #openstack-nova | 22:22 | |
| *** masber has joined #openstack-nova | 22:23 | |
| *** elmaciej has joined #openstack-nova | 22:34 | |
| *** slaweq has quit IRC | 22:36 | |
| *** dave-mccowan has joined #openstack-nova | 22:37 | |
| *** acormier has quit IRC | 22:38 | |
| *** priteau has quit IRC | 22:38 | |
| *** acormier has joined #openstack-nova | 22:39 | |
| *** acormier_ has joined #openstack-nova | 22:41 | |
| *** openstackgerrit has joined #openstack-nova | 22:43 | |
| openstackgerrit | Merged openstack/nova master: Remove deprecated aggregate DB compatibility https://review.openstack.org/544698 | 22:43 |
| *** acormier has quit IRC | 22:45 | |
| *** slaweq has joined #openstack-nova | 22:45 | |
| *** slaweq has quit IRC | 22:50 | |
| openstackgerrit | Merged openstack/nova master: Lazy-load instance attributes with read_deleted=yes https://review.openstack.org/543970 | 22:54 |
| *** jafeha__ has joined #openstack-nova | 22:56 | |
| *** jafeha has quit IRC | 22:59 | |
| mriedem | mdbooth: you should take a look at this https://review.openstack.org/#/c/542646/ | 23:03 |
| *** dave-mccowan has quit IRC | 23:05 | |
| *** AlexeyAbashkin has joined #openstack-nova | 23:06 | |
| *** liverpooler has quit IRC | 23:10 | |
| *** AlexeyAbashkin has quit IRC | 23:10 | |
| *** tetsuro has joined #openstack-nova | 23:13 | |
| tetsuro | is avolkov around? | 23:19 |
| tetsuro | I’d like you to answer the question in https://review.openstack.org/#/c/539865/. … or anyone who can help me. | 23:20 |
| *** burt has quit IRC | 23:20 | |
| *** mlavalle has quit IRC | 23:24 | |
| *** felipemonteiro__ has quit IRC | 23:25 | |
| *** hongbin has quit IRC | 23:27 | |
| *** trinaths has quit IRC | 23:32 | |
| *** masber has quit IRC | 23:34 | |
| *** r-daneel has quit IRC | 23:36 | |
| *** masber has joined #openstack-nova | 23:37 | |
| *** masahisa has joined #openstack-nova | 23:39 | |
| mnaser | is there any way i can do something to get output from nova-compute about memory usage | 23:47 |
| mnaser | i have a nova-compute process sitting on 13gb of ram. | 23:47 |
| mnaser | (before i restart it, just to get helpful information out) | 23:48 |
| melwitt | mriedem: these comments aren't related to the patch, are they? just a suggestion on some better cleanup we could do in a separate patch? https://review.openstack.org/#/c/520158/3/nova/compute/api.py@2826 | 23:49 |
| mnaser | wow... actually almost every nova-compute process is using 13g of memory.. | 23:50 |
| *** amodi has quit IRC | 23:52 | |
| *** slaweq has joined #openstack-nova | 23:53 | |
| *** masber has quit IRC | 23:57 | |
| *** slaweq has quit IRC | 23:58 | |
| *** chyka has quit IRC | 23:58 | |
| *** chyka has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
