*** mlavalle has quit IRC | 00:03 | |
*** avolkov has quit IRC | 00:05 | |
gmann | melwitt: ack. I will check after finishing the testing of policy upgrade check. | 00:11 |
---|---|---|
melwitt | awesome thanks | 00:17 |
*** ociuhandu has joined #openstack-nova | 00:24 | |
*** ociuhandu has quit IRC | 00:30 | |
*** sean-k-mooney has quit IRC | 00:31 | |
gmann | gibi: bauzas : uploaded the upgrade check result of fresh devstack env on patch and those are as expected. tested 3 cases 1. policy file in json format with new defaults only bug scenario http://paste.openstack.org/show/792909/ 2. policy file with both defaults present http://paste.openstack.org/show/792910/ 3. policy file generated in yaml format - http://paste.openstack.org/show/792911/ | 00:49 |
*** sapd1 has quit IRC | 01:00 | |
*** sapd1 has joined #openstack-nova | 01:00 | |
gmann | case4: no policy file - http://paste.openstack.org/raw/792913/ | 01:05 |
*** Liang__ has joined #openstack-nova | 01:08 | |
*** gyee has quit IRC | 01:10 | |
openstackgerrit | xuyuanhao proposed openstack/nova master: the vms can not be force deleted when vm_status is soft-delete and task-state=deleting https://review.opendev.org/724260 | 02:02 |
*** tinwood has quit IRC | 02:10 | |
*** tinwood has joined #openstack-nova | 02:12 | |
*** maciejjozefczyk has quit IRC | 02:15 | |
*** maciejjozefczyk has joined #openstack-nova | 02:15 | |
*** mkrai has joined #openstack-nova | 02:31 | |
*** JamesBenson has quit IRC | 02:37 | |
*** mkrai has quit IRC | 02:51 | |
*** mkrai has joined #openstack-nova | 02:58 | |
*** JamesBenson has joined #openstack-nova | 03:05 | |
gmann | melwitt: tested locally and it lgtm from backword compatibility point of view and without microversion. but it is fixing more APIs than listed in commit msg so need test coverage for those. | 03:12 |
*** psachin has joined #openstack-nova | 03:34 | |
*** mkrai has quit IRC | 03:36 | |
*** mkrai_ has joined #openstack-nova | 03:36 | |
*** yedongcan has joined #openstack-nova | 03:50 | |
melwitt | gotcha thanks gmann | 03:52 |
*** dasp_ has joined #openstack-nova | 03:58 | |
*** dasp has quit IRC | 04:00 | |
*** amodi has quit IRC | 04:03 | |
*** ratailor has joined #openstack-nova | 04:20 | |
*** jangutter_ has quit IRC | 04:25 | |
*** jangutter has joined #openstack-nova | 04:26 | |
*** mkrai_ has quit IRC | 04:30 | |
*** mkrai__ has joined #openstack-nova | 04:30 | |
*** evrardjp has quit IRC | 04:35 | |
*** evrardjp has joined #openstack-nova | 04:36 | |
*** jangutter_ has joined #openstack-nova | 04:37 | |
*** jangutter has quit IRC | 04:41 | |
*** JamesBenson has quit IRC | 04:43 | |
*** gouthamr_ has joined #openstack-nova | 04:46 | |
*** vishalmanchanda has joined #openstack-nova | 05:07 | |
*** jangutter_ has quit IRC | 05:11 | |
*** jmlowe has quit IRC | 05:15 | |
*** jmlowe has joined #openstack-nova | 05:26 | |
*** tetsuro has joined #openstack-nova | 05:29 | |
*** sapd1_x has joined #openstack-nova | 05:30 | |
*** damien_r has joined #openstack-nova | 05:36 | |
*** udesale has joined #openstack-nova | 05:41 | |
*** damien_r has quit IRC | 05:41 | |
*** dklyle has quit IRC | 05:57 | |
*** jangutter has joined #openstack-nova | 06:02 | |
*** ociuhandu has joined #openstack-nova | 06:03 | |
*** ociuhandu has quit IRC | 06:07 | |
*** ociuhandu has joined #openstack-nova | 06:18 | |
*** sapd1_x has quit IRC | 06:22 | |
*** ociuhandu has quit IRC | 06:30 | |
*** nightmare_unreal has joined #openstack-nova | 06:45 | |
*** xek has joined #openstack-nova | 06:56 | |
*** belmoreira has joined #openstack-nova | 06:58 | |
*** slaweq has joined #openstack-nova | 06:58 | |
*** iurygregory has joined #openstack-nova | 06:58 | |
*** ociuhandu has joined #openstack-nova | 07:02 | |
*** ociuhandu has quit IRC | 07:11 | |
*** ociuhandu has joined #openstack-nova | 07:11 | |
gibi | good morning | 07:12 |
*** PrinzElvis has joined #openstack-nova | 07:13 | |
PrinzElvis | hi all | 07:13 |
PrinzElvis | https://www.irccloud.com/pastebin/KYjd5FUo/Rabbitmq_error_from_rocky_to_stein | 07:15 |
*** tesseract has joined #openstack-nova | 07:17 | |
*** ociuhandu has quit IRC | 07:24 | |
*** ralonsoh has joined #openstack-nova | 07:27 | |
*** ociuhandu has joined #openstack-nova | 07:27 | |
*** tosky has joined #openstack-nova | 07:28 | |
*** rpittau|afk is now known as rpittau | 07:29 | |
gibi | PrinzElvis: the only thing seems a bit strange to me is "{% endfor %}/{{ om_rpc_vhost }}" as that will be expanded to "//" as om_rpc_vhost is "/" | 07:29 |
gibi | PrinzElvis: but I'm no way expert on the rabbit config | 07:30 |
*** mkrai__ has quit IRC | 07:32 | |
*** ociuhandu has quit IRC | 07:39 | |
PrinzElvis | gibi: Ok I removed this "//" in my "transport_url = rabbit://openstack:2fe0525ec5db4f1b97d4d055527501ec@10.10.1.11:5672,openstack:2fe0525ec5db4f1b97d4d055527501ec@10.10.1.12:5672,openstack:2fe0525ec5db4f1b97d4d055527501ec@10.10.1.13:5672// " | 07:44 |
nightmare_unreal | artom: can you have another look at : https://review.opendev.org/#/c/715395/ . Thanks . Also i don't understand what can cause nova-live-migration zuul job to fail ? what does it even check ? | 07:45 |
PrinzElvis | gibi: and is work | 07:45 |
gibi | PrinzElvis: then I think the problem is in the ansible code you pasted | 07:47 |
gibi | gmann, dansmith: I'm +2 now on the policy upgrade check patch https://review.opendev.org/#/c/723645/ | 07:49 |
*** ratailor is now known as ratailor|lunch | 07:50 | |
*** mkrai__ has joined #openstack-nova | 07:52 | |
PrinzElvis | gibi> this code is from stable/stein: https://github.com/openstack/kolla-ansible/blob/stable/stein/ansible/group_vars/all.yml | 07:52 |
*** mkrai__ has quit IRC | 07:54 | |
*** mkrai_ has joined #openstack-nova | 07:54 | |
*** ociuhandu has joined #openstack-nova | 07:54 | |
gibi | PrinzElvis: I suggest to contact the kolla developers on #openstack-kolla channel | 07:55 |
PrinzElvis | gibi> Thks | 07:56 |
*** damien_r has joined #openstack-nova | 08:06 | |
*** PrinzElvis has quit IRC | 08:07 | |
*** PrinzElvis has joined #openstack-nova | 08:07 | |
*** PrinzElvis has quit IRC | 08:11 | |
*** PrinzElvis has joined #openstack-nova | 08:12 | |
*** ccamacho has joined #openstack-nova | 08:12 | |
*** martinkennelly has joined #openstack-nova | 08:15 | |
*** avolkov has joined #openstack-nova | 08:20 | |
*** ratailor|lunch is now known as ratailor | 08:30 | |
*** Liang__ has quit IRC | 08:32 | |
*** mkrai_ has quit IRC | 08:45 | |
*** mkrai__ has joined #openstack-nova | 08:45 | |
*** jsuchome has joined #openstack-nova | 08:46 | |
*** N3l1x has quit IRC | 08:56 | |
*** masterpe has quit IRC | 08:56 | |
*** ociuhandu has quit IRC | 08:58 | |
*** ociuhandu has joined #openstack-nova | 08:58 | |
*** martinkennelly has quit IRC | 09:00 | |
*** ociuhandu has quit IRC | 09:04 | |
*** yedongcan has quit IRC | 09:06 | |
*** ociuhandu has joined #openstack-nova | 09:16 | |
*** dtantsur|afk is now known as dtantsur | 09:23 | |
*** derekh has joined #openstack-nova | 09:25 | |
*** mkrai__ has quit IRC | 09:28 | |
*** mkrai__ has joined #openstack-nova | 09:28 | |
*** huaqiang has quit IRC | 09:36 | |
*** brtknr has quit IRC | 09:37 | |
*** brtknr has joined #openstack-nova | 09:42 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-audit: Use cliff instead of homegrown argparse bleh https://review.opendev.org/724332 | 09:46 |
*** masterpe has joined #openstack-nova | 09:47 | |
*** martinkennelly has joined #openstack-nova | 10:05 | |
*** ociuhandu has quit IRC | 10:07 | |
*** ociuhandu has joined #openstack-nova | 10:13 | |
*** jaosorior has quit IRC | 10:17 | |
*** huaqiang has joined #openstack-nova | 10:21 | |
*** rpittau is now known as rpittau|bbl | 10:21 | |
kashyap | sean-k-mooney: Hope my response clarifies on some of our disagreements here: https://review.opendev.org/#/c/631154/7 | 10:35 |
kashyap | sean-k-mooney: I wasn't assuming anything particular on your part, but asking to explicitly clarify with me what I meant | 10:35 |
kashyap | sean-k-mooney: As noted there, at one point in the past, QEMU *did* allow 'i440fx' + plain UEFI boot | 10:35 |
kashyap | (As noted in the spec comment, Nova doesn't allow that; we know that. Not debating that at all.) | 10:36 |
kashyap | Anyway, we can sort it out on the change. No rush to respond | 10:36 |
*** derekh has quit IRC | 10:41 | |
*** derekh has joined #openstack-nova | 10:42 | |
*** ociuhandu has quit IRC | 10:48 | |
*** ociuhandu has joined #openstack-nova | 10:49 | |
*** mkrai__ has quit IRC | 11:02 | |
*** rcernin has quit IRC | 11:06 | |
*** sean-k-mooney has joined #openstack-nova | 11:07 | |
*** udesale_ has joined #openstack-nova | 11:12 | |
*** tetsuro has quit IRC | 11:12 | |
*** ociuhandu has quit IRC | 11:13 | |
*** ociuhandu has joined #openstack-nova | 11:14 | |
*** udesale has quit IRC | 11:15 | |
*** ociuhandu has quit IRC | 11:18 | |
*** raildo has joined #openstack-nova | 11:21 | |
artom | nightmare_unreal, yeah, I'll look | 11:43 |
nightmare_unreal | Thanks :) | 11:43 |
artom | nightmare_unreal, nova-live-migration checks... live migration :) | 11:43 |
artom | It's unlikely to be related to your patch, most likely random failure | 11:43 |
nightmare_unreal | yes that's what I meant. WHy it was failing because I didn't touched anything with migration | 11:44 |
openstackgerrit | Jiri Suchomel proposed openstack/nova master: Add ability to download Glance images into the libvirt image cache via RBD https://review.opendev.org/574301 | 11:49 |
*** ociuhandu has joined #openstack-nova | 11:49 | |
*** ociuhandu has quit IRC | 11:54 | |
*** bbowen_ has joined #openstack-nova | 11:57 | |
*** bbowen has quit IRC | 11:57 | |
*** nweinber has joined #openstack-nova | 11:58 | |
*** maciejjozefczyk_ has joined #openstack-nova | 12:08 | |
artom | nightmare_unreal, somethings things fail :) | 12:08 |
*** tkajinam has quit IRC | 12:09 | |
artom | In this case looks like a server evacuation timed out: https://zuul.opendev.org/t/openstack/build/8ac593dc86074d128b16ae0796e9e458/log/job-output.txt#8481 | 12:10 |
*** ratailor has quit IRC | 12:10 | |
*** maciejjozefczyk has quit IRC | 12:11 | |
lyarwood | artom: it failed | 12:11 |
*** ociuhandu has joined #openstack-nova | 12:12 | |
lyarwood | artom: the timeout is just because we are waiting for the instance to go to ACTIVE but it's always in ERROR | 12:12 |
artom | lyarwood, oh, right | 12:12 |
* lyarwood had thought this was fixed | 12:12 | |
artom | In any case, it cannot possibly because of the changes to nova-manage heal_allocations :) | 12:12 |
lyarwood | '[' ERROR '!=' ERROR ']' | 12:13 |
lyarwood | erhm | 12:13 |
lyarwood | oh ignore that | 12:13 |
lyarwood | artom: but yeah sorry | 12:14 |
lyarwood | unrelated | 12:14 |
*** rpittau|bbl is now known as rpittau | 12:22 | |
lyarwood | this looks like another VIF timeout issue, I wonder if we also need to remove the OVS ports etc before restarting the agents and attempting the rescue. | 12:22 |
lyarwood | evacuation even | 12:22 |
*** mriedem has joined #openstack-nova | 12:23 | |
sean-k-mooney | lyarwood: you should not need too | 12:25 |
*** mkrai has joined #openstack-nova | 12:25 | |
sean-k-mooney | we cenrtenly cant assume that will happen in a evacuate case | 12:26 |
lyarwood | well the host should be fenced in an evacuation | 12:27 |
lyarwood | https://review.opendev.org/#/c/723853/ - I was going to add that as a precondition FWIW | 12:28 |
lyarwood | but yeah I don't think that would actually help here | 12:28 |
*** rcernin has joined #openstack-nova | 12:31 | |
*** derekh has quit IRC | 12:32 | |
*** lpetrut has joined #openstack-nova | 12:34 | |
*** maciejjozefczyk_ is now known as maciejjozefczyk | 12:35 | |
*** slaweq has quit IRC | 12:35 | |
*** slaweq has joined #openstack-nova | 12:36 | |
*** derekh has joined #openstack-nova | 12:38 | |
*** slaweq_ has joined #openstack-nova | 12:38 | |
artom | Who handles http://status.openstack.org/reviews/#nova ? | 12:40 |
artom | "Page refreshed at 2020-03-20 06:38:57 UTC" | 12:41 |
artom | Kinda stale ;) | 12:41 |
gibi | artom: I suggest to ask infra | 12:41 |
*** slaweq has quit IRC | 12:41 | |
gibi | I've noticed the staleness of it but never get the time to dig around | 12:41 |
sean-k-mooney | i tink it updates automaticaly nightly | 12:42 |
gibi | it is just a looooong night :) | 12:42 |
sean-k-mooney | oh its 03 not 04 | 12:42 |
artom | Yeah, this ain't Game of Thrones | 12:42 |
sean-k-mooney | adn the 20th not 30th | 12:42 |
sean-k-mooney | ya just misreda the date | 12:43 |
sean-k-mooney | i mix up numbers much less often then letters but it somethimes happens | 12:43 |
*** ociuhandu has quit IRC | 12:44 | |
*** ociuhandu has joined #openstack-nova | 12:45 | |
*** dpawlik has joined #openstack-nova | 12:46 | |
*** dpawlik has quit IRC | 12:46 | |
*** dpawlik has joined #openstack-nova | 12:46 | |
kashyap | Urgh | 12:54 |
kashyap | Rabit-Holes-- | 12:54 |
kashyap | sean-k-mooney: I'm just reading your comment: right now 'i440fx' with UEFI *does not* work. I wrote to the OVMF maintainer to ask about what's the deal there | 12:54 |
*** lbragstad_ has joined #openstack-nova | 12:55 | |
sean-k-mooney | kashyap: i think it was just not implemented | 12:55 |
kashyap | sean-k-mooney: So, don't assume that: "it works with both" _today_. As I mentioned in my comment it was working at _one_ point_ in time... | 12:55 |
kashyap | sean-k-mooney: So, the core problem is this: | 12:55 |
sean-k-mooney | given its like 20 years old im not suprised it was not a priortiy | 12:55 |
kashyap | The 'i440fx' chipset does not have SMM emulation: hence Secure Boot is not possible with it. If someone adds SMM to 'i440fx', SB should work 'fine' for it. | 12:56 |
kashyap | (But indeed it's not a priority to do that) | 12:56 |
sean-k-mooney | kashyap: by the way im not assumeing ti works with both | 12:57 |
kashyap | I'll comment the rest on the change. | 12:57 |
kashyap | sean-k-mooney: You say on the change in the past OVMF_CODE.secboot.fd didn't work with 'i440fx' | 12:57 |
kashyap | That's of course correct -- it never wored | 12:57 |
kashyap | s/wored/worked/ | 12:57 |
*** lbragstad has quit IRC | 12:57 | |
sean-k-mooney | yes the .secboot one did not | 12:57 |
sean-k-mooney | and the release note you quoted state that the OVMF_CODE.fd file did | 12:58 |
kashyap | The non-secboot.fd used to work in the past. Right now, it drops me directly to the UEFI shell. | 12:58 |
kashyap | sean-k-mooney: Yes, it did in December-2016. Today, it didn't. Still checking what's the state of it | 12:58 |
kashyap | There's only one responsive OVMF maintainer/developer that I know; and he's terribly overworked | 12:58 |
sean-k-mooney | well OVMF_CODE.fd is just uefi firmware | 12:59 |
sean-k-mooney | secure boot is an optional feature | 12:59 |
sean-k-mooney | it might be that the droped support form loadign the firmware form rom | 12:59 |
sean-k-mooney | i think pflash was not supported on i440fx | 12:59 |
sean-k-mooney | i tried to get uefi without secure boot wokking with the pc type a few times in teh past | 13:00 |
sean-k-mooney | but i have only had success using q35 | 13:00 |
sean-k-mooney | using the ovmf binarys in ubuntu 16.04 and 18 | 13:00 |
sean-k-mooney | .04 | 13:00 |
sean-k-mooney | i have not really gone out of my way to get upstream copies or anything like that but i have had to check uefi stuff on 1-2 ocations in the past and just always ended up going to q35 | 13:01 |
sean-k-mooney | anyway i think we can ignore that part of the spec as it really not relevent to chanign the default in nova | 13:02 |
sean-k-mooney | it is relevent for your secure boot spec but not this one | 13:03 |
*** mkrai has quit IRC | 13:03 | |
kashyap | 1 sec; bbiab | 13:09 |
kashyap | sean-k-mooney: I've checked how UEFI / OVMF is built for Ubuntu and Debian and SUSE as part of some earlier work: | 13:10 |
kashyap | https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=932269 | 13:10 |
openstack | Debian bug 932269 in ovmf "Ship the firmware "descriptor files" as part of the 'ovmf' package" [Normal,Fixed] | 13:10 |
kashyap | (And other bzs) | 13:10 |
*** JamesBenson has joined #openstack-nova | 13:10 | |
sean-k-mooney | kashyap: sure but its really not relevent to this spec at all | 13:11 |
kashyap | It's all well-known mess. Anyway, I'm going into a rabbit hole | 13:11 |
sean-k-mooney | can we jsut drop this topic and remove it form the spec or put it in the refrences section | 13:11 |
kashyap | sean-k-mooney: I won't just drop it, I want to highlight the advantage it brings. Having given talks at various conferences, admins often ask me: 'tell me more about what 'q35' brings' | 13:12 |
sean-k-mooney | its not material to the topic of changing the default | 13:12 |
kashyap | But anyway, for now I'll take a break... | 13:12 |
kashyap | sean-k-mooney: Yes, it's not. Whoever said that? | 13:12 |
sean-k-mooney | that what the problem desictiopn section of the spec is ment to cover | 13:13 |
kashyap | Yeah, I'll move it to a separate section - it's just a matter for formatting. I wasn't intentioanlly making it "part of problem description" | 13:13 |
*** lpetrut has quit IRC | 13:14 | |
sean-k-mooney | ok that is what https://github.com/openstack/nova-specs/blob/master/specs/victoria-template.rst#references is for. | 13:18 |
artom | If this was an internal call I'd cut you off and move on to the next topic ;) | 13:19 |
sean-k-mooney | hehe that is why i said can we drop this topic. i ment talking about it not the content | 13:19 |
kashyap | sean-k-mooney: Aside: 'pflash' is very much supported on 'i440fx' on QEMU ("-drive if=pflash") | 13:22 |
*** psachin has quit IRC | 13:22 | |
sean-k-mooney | good to know is it the other way around then. i tought one of the two machine types only supported one option | 13:23 |
sean-k-mooney | e.g. one supported pflash and the other supproted pflash and rom | 13:23 |
sean-k-mooney | so maybe q35 only support rom or i jsut had an error in my libvirt xml when i tried it in the past | 13:23 |
kashyap | Perhaps; for later reading, if you want to learn more, look up: "Configuring pflash devices for OVMF firmware" (an outstanding 'qemu-devel' thread) -- it has all the truths from a maintainer | 13:26 |
*** rcernin has quit IRC | 13:29 | |
*** mkrai has joined #openstack-nova | 13:35 | |
*** ociuhandu has quit IRC | 13:53 | |
*** ociuhandu has joined #openstack-nova | 13:56 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Modify PciDevice.uuid generation code https://review.opendev.org/530487 | 14:00 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Add an online migration for PciDevice.uuid https://review.opendev.org/530905 | 14:00 |
*** eharney has quit IRC | 14:05 | |
*** eharney has joined #openstack-nova | 14:13 | |
*** lpetrut has joined #openstack-nova | 14:13 | |
*** tkajinam has joined #openstack-nova | 14:16 | |
*** spatel has joined #openstack-nova | 14:21 | |
*** mkrai has quit IRC | 14:23 | |
*** dtantsur has quit IRC | 14:24 | |
*** dtantsur has joined #openstack-nova | 14:29 | |
*** francoisp has quit IRC | 14:29 | |
sean-k-mooney | stephenfin: my javascript is terible but you know the code injector thing you got use to use for bugzilla. http://paste.openstack.org/show/792950/ fixes gerrit so you can copy text form gerrit comments again | 14:38 |
sean-k-mooney | just create a new rule for code injector with review\.opendev\.org as the host and that as the content | 14:39 |
stephenfin | Oh, sweet. Thanks! | 14:39 |
*** grandchild has joined #openstack-nova | 14:39 | |
*** mlavalle has joined #openstack-nova | 14:39 | |
sean-k-mooney | acutlly i think i can make it simpler | 14:40 |
sean-k-mooney | stephenfin: ya so this work and is less dumb http://paste.openstack.org/show/792952/ | 14:43 |
sean-k-mooney | i proably could just make it modify the one element that was chagne but it works so its fine | 14:44 |
*** lpetrut has quit IRC | 14:46 | |
*** dklyle has joined #openstack-nova | 14:49 | |
openstackgerrit | Thomas Goirand proposed openstack/nova master: Add a /healthcheck URL https://review.opendev.org/724684 | 14:57 |
gmann | dansmith: replied on this comment if that make sense ? -https://review.opendev.org/#/c/723645/8/nova/cmd/status.py@391 | 14:58 |
gmann | i think checking single base rule is enough to catch the auto generated file with new default. | 14:59 |
gmann | dansmith: or we can check for these set of base rule which are default of every other rules - https://github.com/openstack/nova/blob/730eeb5fe002c093a5b477e095c3374457820007/nova/policies/base.py#L48-L54 | 15:00 |
dansmith | gmann: okay, it was genuinely a question, not an assertion that we should do something different.. do we have any policy rules that were added in U that wouldn't be in the policy file unless they were added by the generator? | 15:00 |
gmann | dansmith: yeah we have few which were made granular. but if old name rule present in file which will be with old default keep working | 15:01 |
gmann | example - https://github.com/openstack/nova/blob/master/nova/policies/agents.py | 15:02 |
*** slaweq_ is now known as slaweq | 15:03 | |
gmann | 'os_compute_api:os-agents' was old and 'os_compute_api:os-agents:*' are new rule | 15:03 |
dansmith | gmann: yeah, so if we have a totally new rule name, and that's in the file with the new default value, then that's a really good indication that they just generated the file right? | 15:04 |
dansmith | gmann: I guess the other way to look at it is: if we check *all* the rules, and warn if any are scope-checing, but the config doesn't have scope turned on, then that's a warning, regardless right? | 15:05 |
gmann | dansmith: yeah and SYSTEM_ADMIN_API which are checking is also new rule. | 15:05 |
dansmith | meaning, regardless of whether they generated the file, or if they think they've written the policy file correctly | 15:05 |
gmann | enforce_scope value check i can add. so checking all the base rule which are defaults for all other rules with enforce_scope check can cover that case also - https://github.com/openstack/nova/blob/730eeb5fe002c093a5b477e095c3374457820007/nova/policies/base.py#L48-L54 | 15:08 |
*** xek has quit IRC | 15:08 | |
dansmith | okay | 15:08 |
gmann | otherwise verifying the whole file with correct or not is very difficult as they can have their own new override value with scope or without | 15:08 |
dansmith | I know, but if they do so with scope, but have scope disabled, that won't work right? | 15:09 |
dansmith | or do you just mean that parsing all the potential options is difficult? | 15:09 |
gmann | yeah parsing. | 15:09 |
dansmith | either way, it's fine.. if system_admin_api is new, and you check the config variable, then that's good enough I think | 15:09 |
dansmith | okay, gotcha | 15:09 |
gmann | ok, modifying. | 15:10 |
dansmith | thanks for doing this | 15:10 |
gmann | dansmith: np!, thanks for all those review, learnt a lot of thing on upgrade stuff :) | 15:11 |
dansmith | :) | 15:11 |
*** salmankhan has joined #openstack-nova | 15:20 | |
openstackgerrit | Merged openstack/nova stable/stein: Add retry to cinder API calls related to volume detach https://review.opendev.org/722783 | 15:23 |
*** gyee has joined #openstack-nova | 15:35 | |
*** sapd1_x has joined #openstack-nova | 15:37 | |
*** hemna has quit IRC | 15:38 | |
*** hemna has joined #openstack-nova | 15:38 | |
kashyap | sean-k-mooney: FYI, one last on this: i440fx + regular UEFI boot (with OVMF) is a valid upstream configuration -- OVMF maintainer just confirmed | 15:41 |
kashyap | sean-k-mooney: I just did a fresh test, and it works. (Previously, my test was broken: I didn't have an EFI partition) | 15:41 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: WIP: objects: Add migrate-on-load behavior for legacy NUMA objects https://review.opendev.org/724381 | 15:41 |
kashyap | sean-k-mooney: Evidence: https://kashyapc.fedorapeople.org/libvirt_plain_UEFI_i440fx_f31_a.xml and https://kashyapc.fedorapeople.org/QEMU_plain_UEFI_i440fx_f31_a.txt | 15:42 |
*** ioni has quit IRC | 15:43 | |
*** sapd1_x has quit IRC | 15:44 | |
openstackgerrit | Thomas Goirand proposed openstack/nova master: Add a /healthcheck URL https://review.opendev.org/724684 | 15:47 |
*** belmoreira has quit IRC | 15:49 | |
gibi | nova meeting starts in less then 10 minutes on #openstack-meeting-3 | 15:50 |
openstackgerrit | Thomas Goirand proposed openstack/nova master: Add a /healthcheck URL https://review.opendev.org/724684 | 15:51 |
melwitt | elod, lyarwood: could I pls get review on https://review.opendev.org/723050 when you have a chance later? | 16:04 |
lyarwood | melwitt: ack looking | 16:05 |
elod | melwitt: will do that in ~1-2 hrs | 16:06 |
melwitt | thanks | 16:07 |
*** rpittau is now known as rpittau|afk | 16:07 | |
openstackgerrit | Thomas Goirand proposed openstack/nova master: Add a /healthcheck URL https://review.opendev.org/724684 | 16:10 |
sean-k-mooney | kashyap: ok i guess i just has an invalid config i was try to convert an existing instance but now that i think about it i proably also did not have a biosboot partion or a gpt partion table for that matter in the vm since it was originally booted wit a legacy bios | 16:15 |
sean-k-mooney | so ya that is proably the same issue i hit | 16:16 |
kashyap | Yes, having an EFI partition is critical :) | 16:16 |
* kashyap bbiab | 16:16 | |
*** tbachman has quit IRC | 16:23 | |
*** tbachman has joined #openstack-nova | 16:24 | |
stephenfin | lyarwood, melwitt: Two noddy patches for a stable core here https://review.opendev.org/#/c/722961/ https://review.opendev.org/#/c/722962/ | 16:25 |
* lyarwood wonders if noddy translates for people in NA | 16:26 | |
lyarwood | stephenfin: ack looking | 16:26 |
lyarwood | oh openstack/placement | 16:27 |
lyarwood | stephenfin: I'm not a stable core theree | 16:27 |
lyarwood | there* | 16:27 |
stephenfin | then who is? | 16:27 |
stephenfin | I honestly expected to be in that group :/ | 16:27 |
melwitt | ah, me neither. and no I did not know the word "noddy" :) | 16:27 |
lyarwood | I now have the song stuck in my head, thanks stephenfin | 16:28 |
lyarwood | stephenfin: https://review.opendev.org/#/admin/groups/1938,members - we might want to fix this | 16:29 |
lyarwood | stephenfin: something for the ptg, the project just came up in the meeting anyway FWIW | 16:29 |
*** ociuhandu has quit IRC | 16:30 | |
lyarwood | stephenfin: https://review.opendev.org/#/admin/groups/1936,members - tbh you and others who are already cores should be copied across | 16:30 |
*** haleyb has quit IRC | 16:31 | |
stephenfin | efried: any chance of adding nova-stable-maint to placement-stable-maint, assuming you think that makes sense? | 16:32 |
lyarwood | or that | 16:32 |
stephenfin | why wait ¯\_(ツ)_/¯ | 16:32 |
stephenfin | efried: https://review.opendev.org/#/admin/groups/540,members and https://review.opendev.org/#/admin/groups/1938,members fwiw | 16:33 |
*** tkajinam has quit IRC | 16:34 | |
*** evrardjp has quit IRC | 16:35 | |
*** evrardjp has joined #openstack-nova | 16:36 | |
efried | stephenfin: I don't think I haven any control over stable anything. | 16:36 |
stephenfin | you're in placement-stable-maint so this might be the sole exception :) | 16:37 |
*** ioni has joined #openstack-nova | 16:37 | |
efried | o rite | 16:37 |
*** udesale_ has quit IRC | 16:39 | |
efried | stephenfin: I don't think it makes sense to include by group. I could cherry pick names that make sense, like dansmith and melwitt. But if someone is in nova-stable-maint but not in stable-maint-core it's because they know nova, which doesn't necessarily mean they know placement. | 16:40 |
efried | Added dansmith and melwitt. | 16:41 |
stephenfin | I've found these things to be self-selecting, personally. I'm stable core in all oslo stuff but there are backports I won't touch with a stick because I don't grok them (anything oslo.concurrency, for one) | 16:41 |
stephenfin | but that's fair | 16:41 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add nova-status upgrade check and reno for policy new defaults https://review.opendev.org/723645 | 16:44 |
efried | Having had two no-longer-working-in-openstack names on the list before was understandably brittle, esp considering stable-maint-core really only has one member who's active. But Dan and Mel should be able to bootstrap it from here as appropriate. | 16:44 |
*** haleyb has joined #openstack-nova | 16:45 | |
*** lbragstad_ is now known as lbragstad | 16:46 | |
gibi | efried: thanks for fixing that up | 16:46 |
dansmith | bauzas: are these proposed yet? I don't see them if you're the owner | 16:48 |
bauzas | dansmith: artom | 16:48 |
bauzas | dansmith: https://review.opendev.org/#/q/topic:placement-audit-backport+(status:open+OR+status:merged) | 16:49 |
bauzas | he proposed the patches, I just expressed the opportunity to un-DNM them | 16:49 |
dansmith | that's quite a stack of dependencies too | 16:50 |
artom | dansmith, the stein one is the last self-contained one, and even then it's kinda fugly | 16:50 |
artom | rocky's a horror (picture show) | 16:50 |
artom | :D | 16:50 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add nova-status upgrade check and reno for policy new defaults https://review.opendev.org/723645 | 16:53 |
*** maciejjozefczyk has quit IRC | 16:54 | |
dansmith | artom: so looking at it, the train one isn't terrible, and if it's really likely that people will need this then I guess it's okay, although I feel like having outstanding issues would be a stronger case | 17:04 |
dansmith | especially since they can fix one-off issues with osc-placement quite easy, and there are community tools to do it | 17:04 |
openstackgerrit | OpenStack Release Bot proposed openstack/os-vif stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/724712 | 17:04 |
openstackgerrit | OpenStack Release Bot proposed openstack/os-vif stable/ussuri: Update TOX_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/724713 | 17:04 |
openstackgerrit | OpenStack Release Bot proposed openstack/os-vif master: Update master for stable/ussuri https://review.opendev.org/724714 | 17:04 |
openstackgerrit | OpenStack Release Bot proposed openstack/os-vif master: Add Python3 victoria unit tests https://review.opendev.org/724715 | 17:04 |
artom | dansmith, so, full disclosure, I don't really care one or another :P | 17:04 |
dansmith | the stein one, while self-contained is already pretty smelly | 17:04 |
artom | dansmith, as I said, it was purely procedural for me - I wanted to do it one release at a time, and upstream CI on it | 17:05 |
dansmith | and beyond stein I think it's unreasonable without a major issue, personally | 17:05 |
dansmith | artom: ack | 17:05 |
bauzas | lemme look why we needed this command | 17:05 |
bauzas | ie. which bugs leaked the allocations | 17:05 |
bauzas | some of them were old and are fixed, some of them still exist IIRC | 17:05 |
artom | dansmith, so I'd be perfectly comfortable doing it downstream-only | 17:06 |
*** dtantsur is now known as dtantsur|afk | 17:06 | |
artom | dansmith, assuming you don't destroy the case for that as well ;) | 17:06 |
artom | I probably didn't do enough "due diligence", and just blindly went ahead and started cherry-picking | 17:06 |
*** nightmare_unreal has quit IRC | 17:06 | |
dansmith | artom: you know I just love crushing hopes and dreams right? | 17:06 |
dansmith | (I really don't despite what you probably think) | 17:06 |
artom | dansmith, I know. How do you think I became such an empty husk? :) | 17:07 |
* dansmith sobs in the corner | 17:07 | |
bauzas | https://review.opendev.org/#/c/670112/14/nova/tests/functional/test_nova_manage.py are the cases we verify | 17:07 |
* bauzas wishes he could have noted as a comment which bugs those checks were related | 17:08 | |
bauzas | so I've tested three cases | 17:16 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Note the ``hw_numa_nodes`` image property https://review.opendev.org/683849 | 17:16 |
bauzas | 1/ deleted instances leak allocations | 17:16 |
bauzas | 2/ resize confirm can also leak allocations | 17:16 |
bauzas | 3/ deleted compute evacuate too | 17:17 |
bauzas | now, the question is, which bugs are those 3 checks related with ? | 17:17 |
bauzas | the last one is easy | 17:17 |
*** tesseract has quit IRC | 17:17 | |
bauzas | https://bugs.launchpad.net/nova/+bug/1829479 | 17:18 |
openstack | Launchpad bug 1829479 in OpenStack Compute (nova) "The allocation table has residual records when instance is evacuated and the source physical node is removed" [Medium,In progress] - Assigned to Matt Riedemann (mriedem) | 17:18 |
bauzas | dansmith: artom: just to finish, I think https://bugs.launchpad.net/nova/+bug/1829479 impacts all releases up to now | 17:22 |
openstack | Launchpad bug 1829479 in OpenStack Compute (nova) "The allocation table has residual records when instance is evacuated and the source physical node is removed" [Medium,In progress] - Assigned to Matt Riedemann (mriedem) | 17:22 |
bauzas | which makes the audit command a good workaround for it | 17:22 |
bauzas | anyway, /me disappears | 17:22 |
dansmith | okay I'm confused.. are we still arguing to backport this or what? | 17:22 |
dansmith | did we fix the leak due to evacuation or are we saying the fix for that is just to have the audit command? | 17:23 |
dansmith | obviously for things that happened before such a fix, audit could help | 17:24 |
bauzas | dansmith: nope, we haven't fixed it yet | 17:26 |
dansmith | okay I would be in favor of backporting *that* :) | 17:26 |
bauzas | the audit command just cleans the mess | 17:26 |
bauzas | anyway, I need a beer | 17:28 |
*** vishalmanchanda has quit IRC | 17:29 | |
*** dpawlik has quit IRC | 17:38 | |
*** salmankhan1 has joined #openstack-nova | 17:44 | |
*** salmankhan has quit IRC | 17:47 | |
mriedem | https://review.opendev.org/#/c/678100/ | 17:49 |
*** salmankhan has joined #openstack-nova | 17:50 | |
*** salmankhan1 has quit IRC | 17:51 | |
*** jsuchome has quit IRC | 17:54 | |
openstackgerrit | sean mooney proposed openstack/nova master: [DNM] testing with force_legacy_port_binding workaround https://review.opendev.org/724387 | 17:57 |
*** salmankhan has quit IRC | 18:01 | |
* bauzas wonders if invoking evacuate issues three times will make mriedem appear in a mirror | 18:01 | |
melwitt | mriedem: want an e-r stat? https://review.opendev.org/724719 | 18:03 |
*** ociuhandu has joined #openstack-nova | 18:03 | |
mriedem | bauzas: no just linking in bugs that are assigned to me | 18:04 |
*** ccamacho has quit IRC | 18:08 | |
mriedem | melwitt: commented | 18:08 |
*** ociuhandu has quit IRC | 18:08 | |
mriedem | it's weird but the before and after on that query are on the same change, and in the new query they are all coming from the same file (n-cpu in the subnode) | 18:08 |
mriedem | is that expected? | 18:09 |
mriedem | was qemu or libvirt bumped in devstack on master? | 18:09 |
melwitt | mriedem: ah dangit ... I focused on the error log message and didn't notice the warning log is what's in the e-r query. so the query doesn't need an update. sorry :( | 18:11 |
openstackgerrit | sean mooney proposed openstack/nova master: silence amqp heartbeat warning https://review.opendev.org/724188 | 18:13 |
openstackgerrit | sean mooney proposed openstack/nova master: remove support of oslo.messagiang 9.8.0 warning message https://review.opendev.org/724725 | 18:13 |
melwitt | and since the bug was greyed out on http://status.openstack.org/elastic-recheck/#1840159 at the time, that reinforced my thinking that the query didn't pick it up. sigh | 18:13 |
*** ociuhandu has joined #openstack-nova | 18:15 | |
mriedem | \o/ | 18:21 |
*** jmlowe has quit IRC | 18:24 | |
*** jmlowe has joined #openstack-nova | 18:27 | |
gmann | dansmith: gibi this is ready - https://review.opendev.org/#/c/723645/ | 18:33 |
*** ralonsoh has quit IRC | 18:38 | |
*** gyee has quit IRC | 18:45 | |
dansmith | gmann: a couple more wording clarification tings | 18:49 |
dansmith | *things | 18:49 |
*** gyee has joined #openstack-nova | 18:55 | |
*** hemna has quit IRC | 19:07 | |
*** hemna has joined #openstack-nova | 19:07 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add nova-status upgrade check and reno for policy new defaults https://review.opendev.org/723645 | 19:18 |
gmann | dansmith: updated ^^ | 19:18 |
*** damien_r has quit IRC | 19:18 | |
dansmith | gmann: okay lemme just fix another thing in-line to avoid you having to do it | 19:19 |
gmann | dansmith: sure, thanks | 19:20 |
dansmith | gmann: is it "recommended" or "required" to enable enforce_scope if you have scope checking rules? | 19:20 |
dansmith | I thought required | 19:21 |
gmann | dansmith: this is separate flaf not scope one - enforce_new_defaults | 19:21 |
gmann | flag | 19:21 |
gmann | to switch to new default without overwriting the file | 19:21 |
dansmith | gmann: right, but if you have scope-checking rules and want to keep them, then you need enforce_scope=True right? | 19:22 |
dansmith | you're checking enforce_scope, but if that's disabled, you tell them to enable enforce_new_defaults | 19:22 |
gmann | dansmith: ok, so enforce_scope=True is required and (enforce_new_defaults=True OR file overwrite) to move to new things | 19:23 |
dansmith | right, I think you're telling them the wrong thing | 19:23 |
dansmith | let me comment for context and you can see if I'm right | 19:23 |
gmann | in that sentence i was targeting only file overwrite alternate via flag | 19:23 |
gmann | but i agree we can add enforce_scope=True also for full context | 19:24 |
dansmith | but the sentence is about keeping the new format | 19:25 |
dansmith | U cinnebted | 19:25 |
dansmith | whoa | 19:25 |
dansmith | "I commented" | 19:25 |
*** derekh has quit IRC | 19:25 | |
gmann | dansmith: replied, https://review.opendev.org/#/c/723645/11/nova/cmd/status.py@358 | 19:33 |
gmann | added scope and new flag way. | 19:33 |
dansmith | gmann: ++ | 19:34 |
gmann | my intention is they are aware of new flag enforce_new_defaults t o avoid legacy way of overwriting the file to new default | 19:34 |
dansmith | gmann: I will try really hard to +2 the next one | 19:34 |
gmann | ok, updating. | 19:34 |
dansmith | gmann: ack, I just think the message should include the error, and "how to fix the error" | 19:34 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add nova-status upgrade check and reno for policy new defaults https://review.opendev.org/723645 | 19:40 |
gmann | dansmith: ^^ updated | 19:41 |
dansmith | gmann: I think you forgot to remove the old sentence | 19:44 |
gmann | ohh :( | 19:45 |
dansmith | otherwise that looks good to me | 19:45 |
dansmith | melwitt: you around to help me put gmann out of his misery? | 19:46 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add nova-status upgrade check and reno for policy new defaults https://review.opendev.org/723645 | 19:47 |
gmann | dansmith: done, sorry for that. | 19:47 |
dansmith | gmann: thanks for your patience with me, +2d | 19:47 |
dansmith | melwitt: ^ | 19:47 |
dansmith | I gotta stretch my legs, bbl | 19:48 |
gmann | dansmith: thanks | 19:49 |
melwitt | yeah I'll get it | 19:51 |
*** ociuhandu has quit IRC | 19:57 | |
*** slaweq has quit IRC | 19:57 | |
*** ociuhandu has joined #openstack-nova | 19:58 | |
*** slaweq has joined #openstack-nova | 19:58 | |
*** ociuhandu has quit IRC | 20:03 | |
*** ociuhandu has joined #openstack-nova | 20:07 | |
melwitt | gmann: I found some misspellings and grammar things and what seem to be a few bugs, but I also suggested some additional info. let me know what you think. if you agree with them, I can update and approve if you want. I know you have been through a lot of updates with the policy docs things | 20:13 |
gmann | melwitt: ok, checking.. | 20:13 |
melwitt | the main thing I'm worried about is if operators just enable enforce_scope = True when they see the status check, without realizing that means end users will have to request scoped tokens from keystone in order to access the formerly admin, now system scoped admin APIs. if I understand correctly | 20:15 |
*** mgariepy has quit IRC | 20:22 | |
*** umbSublime has joined #openstack-nova | 20:26 | |
*** slaweq has quit IRC | 20:29 | |
*** mgariepy has joined #openstack-nova | 20:32 | |
gmann | melwitt: thanks. replied. i linked the new doc there which i can move on those line so that they have complete steps info. | 20:33 |
gmann | mentioning those in upgrade checks is too much as they need to do read-only roles things also. | 20:34 |
gmann | is that fine ? | 20:34 |
melwitt | I dunno ... will depend on how carefully the operators regard that message | 20:35 |
melwitt | we can try it, just let's be ready for when/if people miss the "you have to request scoped tokens" part | 20:36 |
melwitt | like blarg admin APIs no longer work! | 20:37 |
gmann | ok, let me add those lines and then doc link. | 20:37 |
melwitt | I could be wrong. let's just say I've been burned by not 100% overly clear reno/tool output ... nova-consoleauth *cough cough* | 20:38 |
*** raildo_ has joined #openstack-nova | 20:46 | |
*** martinkennelly has quit IRC | 20:46 | |
*** martinkennelly has joined #openstack-nova | 20:47 | |
*** raildo has quit IRC | 20:49 | |
*** cgoncalves has quit IRC | 20:53 | |
*** Jeffrey4l has quit IRC | 20:53 | |
*** gary_perkins has quit IRC | 20:53 | |
*** johanssone has quit IRC | 20:53 | |
*** openstackgerrit has quit IRC | 20:53 | |
*** gary_perkins has joined #openstack-nova | 20:53 | |
*** Jeffrey4l has joined #openstack-nova | 20:53 | |
*** johanssone has joined #openstack-nova | 20:54 | |
*** cgoncalves has joined #openstack-nova | 20:55 | |
*** sean-k-mooney has quit IRC | 20:58 | |
*** sean-k-mooney has joined #openstack-nova | 21:05 | |
*** smcginnis has quit IRC | 21:06 | |
*** smcginnis has joined #openstack-nova | 21:07 | |
*** nweinber has quit IRC | 21:17 | |
*** raildo_ has quit IRC | 21:19 | |
gmann | melwitt: ^^ updated, please check if looks fine. linked the main doc as 'educate' in reno but in upgrade check msg i need to paste the complete link - https://review.opendev.org/#/c/723645/15 | 21:22 |
melwitt | gmann: thanks. weird it didn't post an update in channel | 21:26 |
melwitt | oh our gerritbot left | 21:26 |
gmann | melwitt: yeah i was also surprised initially and rechecking my local branch that i did submit or not :) | 21:28 |
melwitt | haha yeah | 21:28 |
melwitt | I mentioned it in #opendev | 21:28 |
melwitt | gmann: the updates look good. I will watch in zuul so I can sanity check the generated doc previews and then I'll approve it after that | 21:31 |
gmann | melwitt: sure, thanks | 21:31 |
*** mriedem has quit IRC | 21:39 | |
*** mriedem has joined #openstack-nova | 21:39 | |
*** mriedem has left #openstack-nova | 21:40 | |
*** grandchild has quit IRC | 21:40 | |
*** openstackgerrit has joined #openstack-nova | 21:40 | |
openstackgerrit | Ghanshyam Mann proposed openstack/os-vif master: [Community goal] Update contributor documentation https://review.opendev.org/722392 | 21:40 |
gmann | working now \o/ | 21:41 |
gmann | sean-k-mooney: ^^ updated as per your comment on 722392 | 21:41 |
sean-k-mooney | yep looking now | 21:41 |
sean-k-mooney | omg you have a white space how coudl you :P https://review.opendev.org/#/c/722392/3/doc/source/contributor/contributing.rst@29 | 21:42 |
sean-k-mooney | gmann: if you have time to fix https://review.opendev.org/#/c/722392/3/doc/source/contributor/contributing.rst those nits im happy with the content | 21:46 |
openstackgerrit | Ghanshyam Mann proposed openstack/os-vif master: [Community goal] Update contributor documentation https://review.opendev.org/722392 | 21:46 |
gmann | sean-k-mooney: done | 21:46 |
sean-k-mooney | awsome | 21:47 |
sean-k-mooney | ah i see you have already rechecked the other patch | 21:47 |
sean-k-mooney | i proably got lost with the zuul restart | 21:47 |
gmann | yeah, i thought that was merged | 21:48 |
sean-k-mooney | ya same | 21:48 |
sean-k-mooney | i guess not but it should merge soon | 21:48 |
sean-k-mooney | then we can merge stephens cahnge which is blocked by that | 21:49 |
gmann | sean-k-mooney: seems we need to backport that on ussuri brach also - https://review.opendev.org/#/c/724712/ | 21:51 |
sean-k-mooney | ya that makes sense i dont think we cap the sphinx version in the upperconstratit to prevent the issue | 21:52 |
gmann | we missed to recheck the fix and ussuri release one merged first - https://review.opendev.org/#/c/723687/ | 21:53 |
sean-k-mooney | ya one it merges on master we can cherry pick it and fast appove | 21:55 |
sean-k-mooney | then rebase teh git review patch on top | 21:55 |
sean-k-mooney | im not sure if i will be around much longer today be we can get that done tommorow | 21:55 |
gmann | +1 | 21:58 |
*** rcernin has joined #openstack-nova | 22:04 | |
*** larainema has quit IRC | 22:05 | |
openstackgerrit | Merged openstack/nova stable/ussuri: zuul: Switch to the Zuulv3 grenade job https://review.opendev.org/724189 | 22:14 |
melwitt | gmann: added a comment on the nova-status review, I had been thinking the non-existent policy file would also result in taking all of the defaults | 22:20 |
*** ociuhandu has quit IRC | 22:29 | |
*** ociuhandu has joined #openstack-nova | 22:30 | |
*** martinkennelly has quit IRC | 22:31 | |
*** rcernin has quit IRC | 22:33 | |
*** rcernin has joined #openstack-nova | 22:34 | |
*** ociuhandu has quit IRC | 22:35 | |
*** tkajinam has joined #openstack-nova | 22:46 | |
*** tbachman has quit IRC | 22:49 | |
*** tbachman has joined #openstack-nova | 22:52 | |
*** tosky has quit IRC | 23:11 | |
*** spatel has quit IRC | 23:17 | |
gmann | melwitt: sorry, I was away. | 23:22 |
gmann | you mean file location in nova.conf but does not exist or with defaults conf (oslo_policy.policy_file) with no file. | 23:23 |
*** tkajinam has quit IRC | 23:43 | |
*** tkajinam has joined #openstack-nova | 23:43 | |
melwitt | gmann: um... I'm not sure, I just meant like when there's no policy.json file, it uses the defaults in code? | 23:48 |
*** ociuhandu has joined #openstack-nova | 23:57 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!