Wednesday, 2021-05-12

*** avolkov has quit IRC00:01
*** hamalq has quit IRC00:07
*** bbowen has joined #openstack-nova00:28
*** bbowen_ has quit IRC00:29
*** martinkennelly has quit IRC00:31
*** dave-mccowan has joined #openstack-nova00:32
*** bbowen has quit IRC00:39
*** bbowen has joined #openstack-nova00:39
*** bbowen_ has joined #openstack-nova00:44
*** bbowen has quit IRC00:45
*** bbowen__ has joined #openstack-nova00:47
*** bbowen_ has quit IRC00:49
*** bbowen__ has quit IRC00:51
openstackgerritTetsuro Nakamura proposed openstack/osc-placement master: Support auto-negotiated microversion  https://review.opendev.org/c/openstack/osc-placement/+/69602801:01
openstackgerritTetsuro Nakamura proposed openstack/osc-placement master: Switch default to use latest microversion  https://review.opendev.org/c/openstack/osc-placement/+/69604001:01
openstackgerritlikui proposed openstack/nova master: Replace getargspec with getfullargspec  https://review.opendev.org/c/openstack/nova/+/79040501:07
*** LinPeiWen has joined #openstack-nova01:09
*** whoami-rajat has quit IRC01:23
openstackgerritTetsuro Nakamura proposed openstack/osc-placement master: Support auto-negotiated microversion  https://review.opendev.org/c/openstack/osc-placement/+/69602801:54
openstackgerritTetsuro Nakamura proposed openstack/osc-placement master: Switch default to use latest microversion  https://review.opendev.org/c/openstack/osc-placement/+/69604001:54
*** gyee has quit IRC02:20
*** rcernin has quit IRC02:23
*** tbachman has quit IRC02:38
*** tbachman has joined #openstack-nova02:40
openstackgerritlikui proposed openstack/nova master: Replace getargspec with getfullargspec  https://review.opendev.org/c/openstack/nova/+/79040502:51
*** tbachman_ has joined #openstack-nova03:00
*** tbachman has quit IRC03:02
*** tbachman_ is now known as tbachman03:02
*** rcernin has joined #openstack-nova03:09
*** rcernin has quit IRC03:16
*** rcernin has joined #openstack-nova03:25
*** rcernin has quit IRC03:26
*** rcernin has joined #openstack-nova03:27
*** sapd1 has joined #openstack-nova03:28
*** tkajinam has quit IRC03:34
*** tkajinam has joined #openstack-nova03:34
*** links has joined #openstack-nova03:57
*** ratailor has joined #openstack-nova04:34
*** ignaziocassano has joined #openstack-nova04:35
*** ignaziocassano has quit IRC04:36
*** whoami-rajat_ has joined #openstack-nova04:44
*** hemanth_n has joined #openstack-nova04:55
*** ratailor_ has joined #openstack-nova05:13
*** ratailor has quit IRC05:15
*** lpetrut has joined #openstack-nova05:27
*** sapd1 has quit IRC05:35
*** macz_ has joined #openstack-nova05:37
*** macz_ has quit IRC05:43
*** mkrai has joined #openstack-nova05:52
*** ratailor__ has joined #openstack-nova05:56
*** avolkov has joined #openstack-nova05:57
*** ratailor_ has quit IRC05:58
*** happyhemant has joined #openstack-nova05:58
*** ralonsoh has joined #openstack-nova06:07
*** slaweq has joined #openstack-nova06:16
*** lpetrut has quit IRC06:21
*** mkrai has quit IRC06:42
*** ralonsoh has quit IRC06:48
*** ralonsoh has joined #openstack-nova06:49
*** tinwood has quit IRC06:54
*** swp20 has quit IRC06:55
*** tinwood has joined #openstack-nova06:57
*** jhesketh has quit IRC07:08
*** tosky has joined #openstack-nova07:09
*** dklyle has quit IRC07:15
*** macz_ has joined #openstack-nova07:17
*** andrewbonney has joined #openstack-nova07:17
*** macz_ has quit IRC07:21
*** rpittau|afk is now known as rpittau07:26
*** sapd1 has joined #openstack-nova07:26
*** efried has quit IRC07:30
*** efried1 has joined #openstack-nova07:31
*** efried1 is now known as efried07:33
*** lucasagomes has joined #openstack-nova08:02
*** sapd1 has quit IRC08:06
*** macz_ has joined #openstack-nova08:09
*** derekh has joined #openstack-nova08:09
*** sapd1 has joined #openstack-nova08:11
*** macz_ has quit IRC08:13
*** mgariepy has quit IRC08:15
*** ociuhandu has joined #openstack-nova08:17
*** mgariepy has joined #openstack-nova08:18
*** ociuhandu has quit IRC08:18
*** ociuhandu has joined #openstack-nova08:18
*** jhesketh has joined #openstack-nova08:23
*** dave-mccowan has quit IRC08:23
*** tbachman_ has joined #openstack-nova08:25
*** tbachman has quit IRC08:25
*** tbachman_ is now known as tbachman08:25
*** masayukig has quit IRC08:27
*** dave-mccowan has joined #openstack-nova08:28
*** martinkennelly has joined #openstack-nova08:28
*** masayukig has joined #openstack-nova08:29
*** swp20 has joined #openstack-nova08:40
*** dtantsur|afk is now known as dtantsur09:03
openstackgerritMerged openstack/osc-placement master: Support auto-negotiated microversion  https://review.opendev.org/c/openstack/osc-placement/+/69602809:03
openstackgerritMerged openstack/osc-placement master: Switch default to use latest microversion  https://review.opendev.org/c/openstack/osc-placement/+/69604009:03
*** rpittau is now known as rpittau|bbl09:07
*** ociuhandu has quit IRC09:08
gibi\o/ that is a real usability improvement ^^09:12
kashyapgibi: or anyone: Do you know if any other non-OpenStack project using 'reno' for release notes management?09:15
stephenfinI use it for my own stuff09:15
gibikashyap: I'm not aware of any but I never searched for it09:15
kashyapOr is it closely tied to how OpenStack produces docs?09:15
kashyapstephenfin: I see, good to know.  It's not tied to OpenStack's docs tooling, right?09:15
stephenfinIt's not closely tied to how we produce docs, but it is closely tied to our branching model09:16
kashyapstephenfin: The reason I ask is, QEMU upstream uses MediaWiki for changelog.  E.g. https://wiki.qemu.org/ChangeLog/6.009:16
kashyapstephenfin: I'm wondering if I can bring them to start using 'reno'09:16
kashyapOr any other generic Git-based rel note management.09:16
stephenfinFor example, I tried to integrate it into Sphinx, but they have a merge-back model whereby you propose things to stable branches and then periodically merge those back into master. That breaks reno, which expects branches stable branches to diverge09:17
stephenfinkashyap: towncrier09:17
kashyapstephenfin: Oh, I see.  So, if a project has non-converging stable -> master approach, then 'reno' is good?09:18
stephenfinkashyap: it's not as clever, but that's possibly a good thing. No git parsing logic. You simply create news fragment files and then once, before a release, you run a manual step to combine those into once NEWS/CHANGELOG/<whatever> file09:18
kashyapstephenfin: Didn't know of 'towncrier'; /me looks up.09:18
stephenfinkashyap: Yes, I think that's a fair summary09:19
kashyapstephenfin: Okay; thanks for the suggestion.  I'll check w/ the QEMU upstream folks to guage the appetite09:19
kashyapSince I got used to 'reno', I abhor MediaWiki :D09:19
stephenfinthough you also occasionally need to go in and create a static summary changelog (or whatever you want to call them), whereby you instruct reno not to scan past commit N and simply generate a static listing09:20
stephenfinif you don't do this, the build time will continue to grow with your commit history09:20
kashyapstephenfin: Ah, that's a very useful thing to know; thanks.09:21
stephenfincompare e.g. releasenotes/source/newton.rst with releasenotes/source/ocata.rst09:21
stephenfinthe git log for the former should explain some of the rationale09:21
* kashyap goes to compare09:21
stephenfinI should probably get this written down in the reno docs...09:21
stephenfinhope that helps09:21
kashyapstephenfin: Yes, very much.  Please document the above useful bits you just said.  It helps non-OpenStack projects to evaluate it clearly09:22
kashyapEspecially the non-converging branches expectation is a good one09:23
*** ociuhandu has joined #openstack-nova09:27
kashyapI worry about the "occasionally don't forget to generate static summary" from dead branches -- it's easy to forget.  /me thinks if there's a way to automate it (even if it's just an email reminder)09:29
kashyapstephenfin: Is Sphinx's merge-back model documented somewhere?09:33
stephenfinI don't think so09:33
stephenfinhttps://www.sphinx-doc.org/en/master/internals/contributing.html#bug-reports-and-feature-requests09:34
stephenfinIt's described in a round-about manner in the Getting started guide here09:35
kashyapAh; thanks :)09:35
*** ociuhandu has quit IRC09:35
*** sapd1_x has joined #openstack-nova09:43
*** ociuhandu has joined #openstack-nova09:47
*** sapd1_x has quit IRC09:59
openstackgerritBalazs Gibizer proposed openstack/nova-specs master: Allow provider re-parenting in placement  https://review.opendev.org/c/openstack/nova-specs/+/78824310:05
sean-k-mooneywait what10:06
*** tbachman_ has joined #openstack-nova10:07
sean-k-mooneywhy makeing osc using max is a huge breaking change10:07
gibisean-k-mooney: it is the CLI10:07
sean-k-mooneyi hope that is going to be a major version bump10:07
gibiuse by humand10:07
sean-k-mooneygibi: yes but it was intended to be stable for scripting10:07
gibiif somebody scripts against the CLI instead of against the SDK then I'm a sad panda10:08
sean-k-mooneyit predatae the sdk and not eveytone used python so saying use the sdk is not valid either10:08
*** tbachman has quit IRC10:08
*** tbachman_ is now known as tbachman10:08
sean-k-mooneygibi: it was one of osc design goal to not do this10:08
sean-k-mooneygibi: i would be fine with osc defaulting to the miniums micorverion required for the parmateres you passed10:09
*** macz_ has joined #openstack-nova10:09
sean-k-mooneysince that is still stable but follow the project client and going to latest is a regression in my view10:09
gibiOK, I will push a revert of the last patch that switch the default behaviopr10:11
gibistephenfin: ^^10:11
gibiand we can discuss10:11
gibiin the review10:11
gibias we might need tetsuro10:11
gibiI can imagine a solution where a flag or env variable can be used to enable auto negotiation instead10:13
gibiI presonally would like to use auto negotiation10:13
gibias I always forget that openstack resource provider trait list needs 1.6 at least10:13
sean-k-mooneygibi: i would be fine with auto negociation but negociating the latest i think is not correct. also yes a flag for enabling it i think would be fine10:14
*** macz_ has quit IRC10:14
gibiI understand your point10:14
stephenfinsean-k-mooney: osc is slowly switching to using the latest as part of the migration to sdk10:14
sean-k-mooneyif we want to do it by default we coudl but that in my view would need a major version bump10:14
gibiI migth disagree that we need to support people scripting against the CLI10:14
sean-k-mooneyit was one of the desigin reason why it had its current behavior10:15
stephenfinscripting users should use the sdk now. They can keep using older versions of osc also10:15
* gibi wonders why the IRC log is not updated in the web :/10:15
sean-k-mooneystephenfin: that assume they are using python10:15
stephenfinyes10:15
sean-k-mooneygibi: it lags there is a chorn script i think that runs evey few minutes10:15
sean-k-mooneystephenfin: that is not a valid assumtion10:15
sean-k-mooneyosc was create to be a stabel cli that is scriptable10:16
stephenfinmaybe not, but we've got limited maintainers and the ones that are there want auto-negotiation10:16
stephenfinosc was also created when we had many more maintainers10:16
sean-k-mooneywhich is why we can channge that if we want to but it need to be a major version bump to singnale we are breakign backward compatiabiltiy10:16
stephenfinyes, the last release was a major version bump iirc10:17
sean-k-mooneythe release contiang the default change patch would have to be a major version bump10:18
sean-k-mooneyhaving the ablity to auto  negociate is not a breaking change turning it on by default is10:18
*** rpittau|bbl is now known as rpittau10:19
*** ociuhandu has quit IRC10:20
sean-k-mooneygibi: stephenfin  if ye want to not revert and you make sure the next version of osc is a major version then i could proably live with that but i still think min required would be better that said i would like thinks like flavor validation to be on by default10:21
stephenfinthis only affects osc-placement, not osc btw10:22
* gibi lunches 10:22
sean-k-mooneyoh it does?10:22
sean-k-mooneyi tought it was all of osc10:22
stephenfinno. That's happening but separately. The change above was just for osc-placement10:23
sean-k-mooneyi still woudl prefer auto negociating the min reqruied not latest but maybe i have to just live with it being unstable10:23
sean-k-mooneyi rarely have need to script against osc so maybe it not worth avocating for those that do10:24
stephenfintbf, those users could simply request an explicit base version manually10:24
stephenfin'--os-compute-api-version 2.1' or whatever10:24
sean-k-mooneyi just dont want verizon complaining that they cant create flavor when the insitent on setting hw:numa_mem_policy that does not exist10:24
sean-k-mooneystephenfin: ya i know but many dont know what microverions are or care they just want the command they always used to work10:26
sean-k-mooneystephenfin: gibi if the desisin to move all of osc to latest has aleready been made then dont revert the placment patch10:27
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Enable 'vmcoreinfo' feature by default  https://review.opendev.org/c/openstack/nova/+/75291210:29
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Make vmcoreinfo dependent on feature flag  https://review.opendev.org/c/openstack/nova/+/77934210:29
openstackgerritStephen Finucane proposed openstack/nova master: tests: Remove 'create_fake_libvirt_mock'  https://review.opendev.org/c/openstack/nova/+/79094510:29
openstackgerritStephen Finucane proposed openstack/nova master: WIP: libvirt: Make vmcoreinfo checks guest arch specific  https://review.opendev.org/c/openstack/nova/+/79094610:29
*** macz_ has joined #openstack-nova10:30
gibisean-k-mooney: OK, then I won't revert.10:33
gibiI honestly don't know what will be the overall osc policy10:34
*** macz_ has quit IRC10:34
sean-k-mooneygibi: right now i really dont like that its different per command10:35
sean-k-mooneyas it make it much harder to use10:35
sean-k-mooneythat said i know ironic broke the consitency years ago so its not a new thing10:36
gibiI think it is the same for each call . max(server,client)10:36
sean-k-mooneygibi: not i mean nova and cinder command wont auto negociate10:36
sean-k-mooneybut ironic and plamcent will10:36
gibiI see10:36
gibibut if the overall osc change is coming the it will be in sync again10:37
sean-k-mooneyyes if but there was never a cross project agreement to do that as far as i knew10:37
sean-k-mooneybut if its happeing then i just hope it gets done this cycle10:37
sean-k-mooneydraging it out over multiple cycle will not be a good ux10:38
sean-k-mooneywhich is why i was expecting this type of change to be a cross project goal10:38
gibiOK, then please propose a revert on the osc patch and lets bring it up on cross project levels10:38
stephenfinI don't think this is a serious issue10:39
stephenfinIf people don't like it, then they'll let us know10:39
gibiIt seems it is serious for sean-k-mooney10:39
stephenfinOtherwise 🤷10:39
jrosserif i put debug=True in [DEFAULT] of nova.conf (for example) should that be sufficient to then be able to see https://github.com/openstack/oslo.messaging/blob/master/oslo_messaging/_drivers/impl_rabbit.py#L608-L610 in the nova-conductor log file?10:41
sean-k-mooneyi just dont want to have to explain in our downstream doc and to customer why openstack server create and openstack resource provider list work differently10:41
sean-k-mooneyjrosser: am im not sure i thought that was only for the nova log level10:42
sean-k-mooneybut maybe its everything10:42
jrossersean-k-mooney: yeah, i'm not seeing those and am struggling to find how to enable oslo.messaging debug in the context of a nova service10:43
gibijrosser: I think you need this config https://docs.openstack.org/nova/latest/configuration/config.html#DEFAULT.default_log_levels10:44
gibithere the oslo.messaging is set to INFO by default10:44
gibiso you have to change that10:44
jrosserah! let me try that10:45
*** ociuhandu has joined #openstack-nova10:51
*** ociuhandu has quit IRC10:55
*** ociuhandu has joined #openstack-nova11:07
*** ignaziocassano1 has joined #openstack-nova11:16
ignaziocassano1ralonsoh looking at this bug https://bugzilla.redhat.com/show_bug.cgi?id=1917675 seems redhat released patches only for rhel8 ....so Think for centos811:18
openstackbugzilla.redhat.com bug 1917675 in openstack-nova "Nova evacuate fails due to timeout waiting for a network-vif-plugged event for instance" [Urgent,On_qa] - Assigned to smooney11:18
ignaziocassano1I am using centos711:19
*** dtantsur is now known as dtantsur|brb11:19
*** __ministry has quit IRC11:19
sean-k-mooneyyou are using packages from RDO not form Redhat11:20
*** __ministry has joined #openstack-nova11:20
sean-k-mooneyignaziocassano1: and RDO support train on 7 and 811:20
ignaziocassano1Yes, I wonder if they will be' released on rdo for centos 711:21
sean-k-mooneythe RDO packages and the OSP packages are not the same11:21
sean-k-mooneyignaziocassano1: yes still have an automated build for train on centos7 so it should be eventually11:21
sean-k-mooneyif we backport it upstream11:21
ignaziocassano1Patching manually seems do not work11:22
*** rcernin has quit IRC11:23
ignaziocassano1Will you backport them on rdo centos 7?11:23
*** rcernin has joined #openstack-nova11:24
ignaziocassano1I would like to migrate on train on centos7 then try to upgrade to centos 811:24
ignaziocassano1I think it is a solution for not reinstall all11:25
sean-k-mooneyim not currently plannign to work on backporting it. i may in the future but i have some other things i need to work on11:26
ignaziocassano1So, what can I do to solve it? Next year we 'll invest money for acquire red hat support, but now we cannot11:28
*** happyhemant has quit IRC11:29
sean-k-mooneyone option is to try propose the backport yourself or see if rodolfo or others have time to backport it. i generally try to help people regardless of if they are a customer or not but this partical set of bugs has made me consider quiting opentack developemnt and doing something else with my life so not that its finally merged i dont really want to spend more time on it11:31
*** ociuhandu has quit IRC11:33
ignaziocassano1Yes, I understand your position11:34
jrossergibi: thanks for the default_log_levels tip, it's working for me now11:35
*** rcernin has quit IRC11:35
gibijrosser: cool. happy to help11:36
*** ociuhandu has joined #openstack-nova11:45
*** ignaziocassano1 has quit IRC11:45
*** k_mouza has quit IRC12:03
*** k_mouza_ has joined #openstack-nova12:03
*** k_mouza_ has quit IRC12:08
*** happyhemant has joined #openstack-nova12:09
*** k_mouza has joined #openstack-nova12:09
*** macz_ has joined #openstack-nova12:28
*** macz_ has quit IRC12:33
*** sapd1_x has joined #openstack-nova12:51
kashyaplyarwood: Can you double-confirm: I don't see any use of QEMU's internal snapshots in the code (`qemu-img snapshot ...`)12:56
kashyaplyarwood: The only known use was removed way back in 2013 in this commit: 8c985874c7885f31871204d3f83ce547fefc5fb6)12:56
lyarwoodkashyap: you mean qcow2 snapshots right?12:56
kashyaplyarwood: I ask because ... a certain enterprise distro might deprecate internal snapshots (for good reasons)12:57
kashyaplyarwood: Yes, exactly.  I recall we used to use it; and I don't see any trace of it (good thing)12:57
*** k_mouza_ has joined #openstack-nova12:58
lyarwoodkashyap: yeah I can't think of any examples where we do13:01
kashyaplyarwood: Cool; I just checked the code, I don't see any other traces.  That's great13:01
lyarwoodkashyap: even the NFS based cinder backends that use qcow2 use backing files13:01
kashyapSweet13:01
*** k_mouza has quit IRC13:02
openstackgerritStephen Finucane proposed openstack/nova master: tests: Create 'nova.tests.fixtures' module  https://review.opendev.org/c/openstack/nova/+/79096513:02
openstackgerritStephen Finucane proposed openstack/nova master: tests: Move ConfFixture  https://review.opendev.org/c/openstack/nova/+/79096613:02
openstackgerritStephen Finucane proposed openstack/nova master: tests: Move remaining non-libvirt fixtures  https://review.opendev.org/c/openstack/nova/+/79096713:02
openstackgerritStephen Finucane proposed openstack/nova master: tests: Split external service fixtures out  https://review.opendev.org/c/openstack/nova/+/79096813:02
openstackgerritStephen Finucane proposed openstack/nova master: tests: Move libvirt-specific fixtures  https://review.opendev.org/c/openstack/nova/+/79096913:02
openstackgerritStephen Finucane proposed openstack/nova master: tests: Add os-brick fixture  https://review.opendev.org/c/openstack/nova/+/79097013:02
openstackgerritStephen Finucane proposed openstack/nova master: WIP: Use jinja2 to generate fake libvirt data  https://review.opendev.org/c/openstack/nova/+/79097113:02
*** macz_ has joined #openstack-nova13:05
*** macz_ has quit IRC13:09
*** whoami-rajat_ is now known as whoami-rajat13:17
*** k_mouza has joined #openstack-nova13:17
*** ratailor__ has quit IRC13:18
*** k_mouza_ has quit IRC13:20
*** dave-mccowan has quit IRC13:24
*** ociuhandu has quit IRC13:25
*** hemanth_n has quit IRC13:28
*** bbowen has joined #openstack-nova13:34
*** hack-char has quit IRC13:46
*** sorrison has quit IRC13:46
*** swp20 has quit IRC13:47
*** sorrison has joined #openstack-nova13:48
*** hack-char has joined #openstack-nova13:49
*** ociuhandu has joined #openstack-nova13:56
*** ociuhandu has quit IRC14:02
*** ociuhandu has joined #openstack-nova14:12
*** ociuhandu_ has joined #openstack-nova14:17
*** ociuhandu has quit IRC14:17
*** dtantsur|brb is now known as dtantsur14:26
stephenfingibi: melwitt, dansmith: If you folks have time this week, could you queue up the first few patches in the remove-sqlalchemy-migrate series? They're all precusor patches that I'd like to get out of the way before we get into the nitty-gritty of alembic integration https://review.opendev.org/q/topic:%22bp%252Fremove-sqlalchemy-migrate%22+(status:open%20OR%20status:merged)14:27
stephenfin(I have patches locally but won't push them yet since they're not _quite_ ready)14:27
gibistephenfin: ack. No promise for this week but I will try14:28
sean-k-mooneystephenfin: you used to have a header you added to RST files to forward declare the header level symbols14:31
sean-k-mooneyi tought that was in one of the spec templates but i cant find it do you rememebr what it was14:31
sean-k-mooneyit was a comment you put at the top of the file then just a buch of header lines that would not render14:31
stephenfinsean-k-mooney: precede the block with ..14:32
stephenfinthat's a comment14:32
stephenfinI think the neutron docs use it for license headers14:32
sean-k-mooneyah ok  we use it for the lisence yes14:32
*** amodi has quit IRC14:44
*** macz_ has joined #openstack-nova15:06
*** macz_ has quit IRC15:10
*** sapd1_x has quit IRC15:10
*** sapd1_x has joined #openstack-nova15:14
*** dklyle has joined #openstack-nova15:20
*** mlavalle has joined #openstack-nova15:22
*** macz_ has joined #openstack-nova15:26
*** gyee has joined #openstack-nova15:26
melwittstephenfin: ack, will put them on my list15:27
-openstackstatus- NOTICE: Any builds with POST_FAILURE result and no available logs between 11:41 and 14:41 UTC today were related to an authentication endpoint problem in one of our providers and can be safely rechecked now15:32
*** ociuhandu_ has quit IRC15:37
stephenfinthanks15:40
*** ociuhandu has joined #openstack-nova15:40
*** lucasagomes has quit IRC16:03
NobodyCamGood morning Nova folks, anyone have a pointer as to why the nova-python-client would not honor timeout values when the network drops?16:06
*** ociuhandu_ has joined #openstack-nova16:07
*** ociuhandu has quit IRC16:11
*** ociuhandu_ has quit IRC16:11
*** slaweq has quit IRC16:11
*** rpittau is now known as rpittau|afk16:12
sean-k-mooneymaybe the socket hangs at the kernel level or soemthign like that but no i dont know why that would happen16:13
sean-k-mooneyNobodyCam: i assume the behavior is if the network drops the nova client just sits there waiting for a respocne that wont come16:13
sean-k-mooneyor are you seeign a different behavior16:14
NobodyCamnope that's what I'm seeing16:14
NobodyCamand Good Morning sean-k-mooney16:14
sean-k-mooneyNobodyCam: so ya my only guess is the socket is staling on the kernel side. i would have expected any timeout you set to be passed to the request libary and for it to be handeled there16:19
sean-k-mooneywe dont start any timers ectra in the nova client to interupt the request ourselves16:19
*** happyhemant has quit IRC16:19
NobodyCam++ makes sense to me16:19
*** ociuhandu has joined #openstack-nova16:21
sean-k-mooneymelwitt: you know the nova client better then i do you have any other ideas why NobodyCam issue would happen?16:24
*** amodi has joined #openstack-nova16:25
melwittsean-k-mooney: if I understand the question correctly, my answer would be the same as what you said earlier. is it that the "timeout" kwarg is being passed to the novaclient bindings and it's not waiting as long as the specified timeout when there's no response?16:27
melwittmy experience with the requests library is that in the case of a non-response, it will stop waiting after about 30 seconds, even if you've no timeout (None) or a longer timeout16:28
sean-k-mooneyright so we pass the timeout to keystone auth here https://github.com/openstack/python-novaclient/blob/d4399d3d36c456c1001b96178ce9b3e7ec082460/novaclient/client.py#L137-L141 which form its docstring it passes to requests https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/session.py#L292-L29516:29
*** ociuhandu has quit IRC16:30
melwittyeah. I think if you tried a bare requests call for the same thing, you'd see the same behavior with the timeout16:30
sean-k-mooneythanks i have skimmed the nova clint code in the past but never really dug into it so just wanted to ensure im not missing anything obvious16:34
melwittno, I don't think you're missing anything16:36
NobodyCamThank you both!16:37
*** k_mouza has quit IRC16:55
*** cz3 is now known as sizzy16:59
*** sizzy is now known as cz316:59
*** ralonsoh has quit IRC17:02
*** MrClayPole has quit IRC17:38
*** ociuhandu has joined #openstack-nova17:40
*** ociuhandu has quit IRC17:46
*** MrClayPole has joined #openstack-nova17:49
*** andrewbonney has quit IRC17:56
*** tbachman has quit IRC18:00
*** tbachman_ has joined #openstack-nova18:00
*** dtantsur is now known as dtantsur|afk18:01
*** grandchild has quit IRC18:06
*** grandchild has joined #openstack-nova18:08
*** tbachman has joined #openstack-nova18:14
*** tbachman_ has quit IRC18:16
*** bbowen has quit IRC18:37
*** links has quit IRC18:43
*** k_mouza has joined #openstack-nova18:56
*** k_mouza has quit IRC19:00
*** hoonetorg has joined #openstack-nova19:07
*** macz_ has quit IRC19:16
*** macz_ has joined #openstack-nova19:32
openstackgerritsean mooney proposed openstack/nova-specs master: [WIP] pci device tracking in placment  https://review.opendev.org/c/openstack/nova-specs/+/79104719:34
sean-k-mooneythat still pretty incomplete but im finsihing up for the day so said i woudl push what i had19:35
*** ociuhandu has joined #openstack-nova19:44
*** slaweq has joined #openstack-nova19:48
*** kashyap has quit IRC19:49
*** Jeffrey4l has quit IRC19:49
*** Jeffrey4l has joined #openstack-nova20:00
*** macz_ has quit IRC20:08
*** macz_ has joined #openstack-nova20:08
*** ociuhandu has quit IRC20:12
*** k_mouza has joined #openstack-nova20:23
*** k_mouza has quit IRC20:27
*** dansmith has quit IRC20:29
*** derekh has quit IRC20:39
*** andrewbogott has joined #openstack-nova21:03
andrewbogottHello all!  I have a question about address records. I'm looking at a unit test in trove which mocks an instance address record like this:21:04
andrewbogotthttps://www.irccloud.com/pastebin/gWoqmguo/21:04
andrewbogottMy actual running cloud shows things that look more like this:21:04
andrewbogotthttps://www.irccloud.com/pastebin/mqZwz6TN/21:04
*** slaweq has quit IRC21:04
andrewbogottMy question is: did the record ever look like the former, and changed?  Or is it possible for it to look either way depending on neutron setup?21:05
*** dansmith has joined #openstack-nova21:06
*** k_mouza has joined #openstack-nova21:27
*** openstackgerrit has quit IRC21:47
*** k_mouza has quit IRC21:51
*** avolkov has quit IRC21:52
*** derekh has joined #openstack-nova22:02
*** k_mouza has joined #openstack-nova22:54
*** k_mouza has quit IRC22:58
*** tosky has quit IRC23:01
*** rcernin has joined #openstack-nova23:07
*** macz_ has quit IRC23:08
*** mlavalle has quit IRC23:56

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!