*** brinzhang_ has joined #openstack-nova | 00:06 | |
*** brinzhang has quit IRC | 00:09 | |
*** ivve has quit IRC | 00:10 | |
*** zer0c00l has joined #openstack-nova | 00:34 | |
*** aspiers has quit IRC | 00:35 | |
*** xek has quit IRC | 00:37 | |
*** ccamacho has quit IRC | 00:51 | |
*** Liang__ has joined #openstack-nova | 00:57 | |
openstackgerrit | Arthur Dayne proposed openstack/nova master: libvirt:volume:Disallow AIO=native when no 'O_DIRECT' is available https://review.opendev.org/682772 | 00:58 |
---|---|---|
*** Liang__ is now known as LiangFang | 00:59 | |
*** LiangFang has quit IRC | 01:01 | |
*** Liang__ has joined #openstack-nova | 01:01 | |
*** abaindur has quit IRC | 01:14 | |
*** jawad_axd has joined #openstack-nova | 01:21 | |
*** nanzha has joined #openstack-nova | 01:22 | |
*** jawad_axd has quit IRC | 01:26 | |
*** dpawlik has joined #openstack-nova | 01:40 | |
*** igordc has quit IRC | 01:42 | |
*** jawad_axd has joined #openstack-nova | 01:42 | |
*** Xuchu has joined #openstack-nova | 01:44 | |
*** dpawlik has quit IRC | 01:45 | |
*** jawad_axd has quit IRC | 01:47 | |
*** tbachman has joined #openstack-nova | 01:57 | |
*** ileixe has quit IRC | 02:01 | |
*** jawad_axd has joined #openstack-nova | 02:03 | |
*** larainema has joined #openstack-nova | 02:04 | |
*** ileixe has joined #openstack-nova | 02:05 | |
*** jawad_axd has quit IRC | 02:07 | |
openstackgerrit | Arthur Dayne proposed openstack/nova master: libvirt:volume:Disallow AIO=native when no 'O_DIRECT' is available https://review.opendev.org/682772 | 02:09 |
*** brinzhang has joined #openstack-nova | 02:21 | |
*** brinzhang_ has quit IRC | 02:24 | |
*** markvoelker has joined #openstack-nova | 02:36 | |
*** markvoelker has quit IRC | 02:41 | |
*** igordc has joined #openstack-nova | 02:45 | |
openstackgerrit | Qiu Fossen proposed openstack/nova-specs master: support instance tag fuzzy query https://review.opendev.org/691651 | 02:45 |
openstackgerrit | Qiu Fossen proposed openstack/nova-specs master: support instance tag fuzzy query https://review.opendev.org/691651 | 02:49 |
openstackgerrit | Qiu Fossen proposed openstack/nova-specs master: support instance tag fuzzy query https://review.opendev.org/691651 | 02:50 |
openstackgerrit | Qiu Fossen proposed openstack/nova-specs master: support instance tag fuzzy query https://review.opendev.org/691651 | 02:51 |
*** mdbooth has quit IRC | 02:54 | |
*** mdbooth has joined #openstack-nova | 02:56 | |
*** brinzhang_ has joined #openstack-nova | 03:03 | |
*** brinzhang has quit IRC | 03:07 | |
*** ileixe has quit IRC | 03:11 | |
*** ileixe has joined #openstack-nova | 03:15 | |
*** artom has quit IRC | 03:16 | |
*** brinzhang has joined #openstack-nova | 03:17 | |
*** brinzhang_ has quit IRC | 03:21 | |
*** tbachman has quit IRC | 03:31 | |
*** tbachman has joined #openstack-nova | 03:31 | |
*** Xuchu has quit IRC | 03:31 | |
*** Xuchu has joined #openstack-nova | 03:32 | |
*** Guest14611 has quit IRC | 03:50 | |
*** benj has joined #openstack-nova | 03:51 | |
*** benj is now known as Guest85345 | 03:51 | |
*** ricolin has joined #openstack-nova | 03:57 | |
*** brinzhang_ has joined #openstack-nova | 04:17 | |
*** brinzhang has quit IRC | 04:21 | |
*** dave-mccowan has quit IRC | 04:22 | |
openstackgerrit | Arthur Dayne proposed openstack/nova master: libvirt:volume:Disallow AIO=native when no 'O_DIRECT' is available https://review.opendev.org/682772 | 04:56 |
*** markvoelker has joined #openstack-nova | 04:58 | |
*** markvoelker has quit IRC | 05:03 | |
*** brinzhang has joined #openstack-nova | 05:18 | |
*** brinzhang_ has quit IRC | 05:21 | |
*** igordc has quit IRC | 05:31 | |
*** dviroel has quit IRC | 05:37 | |
*** igordc has joined #openstack-nova | 05:38 | |
*** sridharg has joined #openstack-nova | 05:38 | |
*** dpawlik has joined #openstack-nova | 05:42 | |
*** brinzhang_ has joined #openstack-nova | 05:46 | |
*** markvoelker has joined #openstack-nova | 05:46 | |
*** dpawlik has quit IRC | 05:46 | |
*** brinzhang has quit IRC | 05:49 | |
*** markvoelker has quit IRC | 05:56 | |
*** markvoelker has joined #openstack-nova | 05:56 | |
*** brinzhang has joined #openstack-nova | 05:59 | |
openstackgerrit | Qiu Fossen proposed openstack/nova-specs master: support instance tag fuzzy query https://review.opendev.org/691651 | 05:59 |
*** markvoelker has quit IRC | 06:00 | |
*** brinzhang_ has quit IRC | 06:02 | |
openstackgerrit | Qiu Fossen proposed openstack/nova-specs master: support instance tag fuzzy query https://review.opendev.org/691651 | 06:12 |
*** mkrai has joined #openstack-nova | 06:14 | |
*** jawad_axd has joined #openstack-nova | 06:14 | |
*** brinzhang has quit IRC | 06:17 | |
*** brinzhang has joined #openstack-nova | 06:17 | |
*** abaindur has joined #openstack-nova | 06:22 | |
*** dpawlik has joined #openstack-nova | 06:30 | |
brinzhang | stephenfin: alex_xu: https://review.opendev.org/#/c/682772/ this patach find the root issue and update the changes, change the conf.driver_io in the libvirt driver, not change the conf.driver_io in any disk_driver in https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L174-L200 | 06:35 |
brinzhang | stephenfin: alex_xu: do you have time to review this, and give any suggestion? | 06:36 |
*** Luzi has joined #openstack-nova | 06:38 | |
*** luksky has joined #openstack-nova | 06:43 | |
*** sapd1 has joined #openstack-nova | 06:45 | |
*** brinzhang_ has joined #openstack-nova | 06:48 | |
*** brinzhang_ has quit IRC | 06:49 | |
*** sapd1 has quit IRC | 06:49 | |
*** brinzhang_ has joined #openstack-nova | 06:49 | |
*** tkajinam has quit IRC | 06:50 | |
*** tkajinam has joined #openstack-nova | 06:51 | |
*** brinzhang has quit IRC | 06:51 | |
*** igordc has quit IRC | 06:52 | |
openstackgerrit | Daniel Pawlik proposed openstack/nova stable/train: Added openssh-client into bindep https://review.opendev.org/691808 | 07:02 |
openstackgerrit | Daniel Pawlik proposed openstack/nova stable/stein: Added openssh-client into bindep https://review.opendev.org/691809 | 07:02 |
*** jawad_axd has quit IRC | 07:04 | |
*** nanzha has quit IRC | 07:04 | |
*** nanzha has joined #openstack-nova | 07:08 | |
openstackgerrit | Qiu Fossen proposed openstack/nova-specs master: support instance tag fuzzy query https://review.opendev.org/691651 | 07:14 |
*** jawad_axd has joined #openstack-nova | 07:15 | |
*** jawad_ax_ has joined #openstack-nova | 07:16 | |
*** jawad_axd has quit IRC | 07:20 | |
*** brinzhang has joined #openstack-nova | 07:27 | |
*** rcernin has quit IRC | 07:28 | |
*** brinzhang_ has quit IRC | 07:30 | |
*** ccamacho has joined #openstack-nova | 07:33 | |
*** pcaruana has joined #openstack-nova | 07:37 | |
*** slaweq has joined #openstack-nova | 07:42 | |
openstackgerrit | sunjiyun-inspur proposed openstack/nova master: On branch vm-query-notsameproject https://review.opendev.org/691629 | 07:49 |
*** tesseract has joined #openstack-nova | 07:49 | |
openstackgerrit | Arthur Dayne proposed openstack/nova master: libvirt:driver:Disallow AIO=native when no 'O_DIRECT' is available https://review.opendev.org/682772 | 07:51 |
*** tkajinam has quit IRC | 07:54 | |
*** trident has quit IRC | 07:55 | |
*** trident has joined #openstack-nova | 08:01 | |
openstackgerrit | Qiu Fossen proposed openstack/nova-specs master: support instance tag fuzzy query https://review.opendev.org/691651 | 08:03 |
*** damien_r has joined #openstack-nova | 08:03 | |
*** maciejjozefczyk has joined #openstack-nova | 08:04 | |
*** yaawang has quit IRC | 08:10 | |
*** yaawang has joined #openstack-nova | 08:10 | |
*** ivve has joined #openstack-nova | 08:13 | |
*** brinzhang_ has joined #openstack-nova | 08:13 | |
*** brinzhang has quit IRC | 08:17 | |
*** brinzhang has joined #openstack-nova | 08:34 | |
*** brinzhang_ has quit IRC | 08:37 | |
*** ralonsoh has joined #openstack-nova | 08:44 | |
*** priteau has joined #openstack-nova | 08:46 | |
*** brinzhang_ has joined #openstack-nova | 08:49 | |
*** brinzhang has quit IRC | 08:51 | |
*** dtantsur|afk is now known as dtantsur | 08:55 | |
*** Luzi has quit IRC | 09:07 | |
*** rpittau|afk is now known as rpittau | 09:08 | |
*** tkajinam has joined #openstack-nova | 09:15 | |
*** tkajinam has quit IRC | 09:16 | |
openstackgerrit | Qiu Fossen proposed openstack/nova-specs master: Support fuzzy querying instance by tag https://review.opendev.org/691651 | 09:21 |
*** Luzi has joined #openstack-nova | 09:22 | |
*** Liang__ has quit IRC | 09:23 | |
*** Xuchu has quit IRC | 09:32 | |
*** CeeMac has joined #openstack-nova | 09:37 | |
*** ociuhandu has joined #openstack-nova | 09:47 | |
*** ociuhandu has quit IRC | 09:50 | |
*** brinzhang has joined #openstack-nova | 09:59 | |
*** brinzhang_ has quit IRC | 10:02 | |
*** pcaruana has quit IRC | 10:03 | |
sean-k-mooney | brinzhang: i guess that is one way to adress it. the other way is to say that its invalid to set the cache mode to 'writeback', 'writethrough'or 'unsafe' | 10:03 |
sean-k-mooney | e.g. use cache mode = node or direct_sysnc | 10:04 |
brinzhang | sean-k-mooney: yeah, it's the one way to resolve this issue. | 10:07 |
*** lpetrut has joined #openstack-nova | 10:07 | |
sean-k-mooney | this is also arguably a libvirt bug given we currently dont set teh driver io mode | 10:07 |
sean-k-mooney | im pretty sure i have hit that warning in the past and that is how i fixed it for the lvm driver | 10:08 |
sean-k-mooney | but using thread io is also accpetable | 10:08 |
sean-k-mooney | thread io is a little more expensive in terms of qemu cpu overhead | 10:08 |
brinzhang | I am not sure all of the volume driver can be set thread | 10:09 |
sean-k-mooney | well we dont want to default to threaded | 10:09 |
sean-k-mooney | that would be incorrect | 10:09 |
brinzhang | But if donot set driver_io to thread, the bug | 10:10 |
brinzhang | the bug is also exist, do you have another suggestion to fix it? | 10:10 |
sean-k-mooney | no what i mean is we dont want to defualt to therad in general outside of this edgecase | 10:10 |
brinzhang | I know set thread more expensive | 10:10 |
sean-k-mooney | by the way i dont think | 10:11 |
sean-k-mooney | if (hasattr(conf, 'driver_io') and | 10:11 |
sean-k-mooney | conf.driver_io == "native" and conf.driver_cache in | 10:11 |
sean-k-mooney | ['writeback', 'writethrough', 'unsafe']): | 10:11 |
sean-k-mooney | will work | 10:11 |
sean-k-mooney | do you know where conf.driver_io == "native" is set | 10:11 |
sean-k-mooney | oh its set in the volume plugins | 10:12 |
brinzhang | it's in the volume driver, such as iscsi https://github.com/openstack/nova/blob/master/nova/virt/libvirt/volume/iscsi.py#L57 | 10:12 |
sean-k-mooney | yes im not sure its valid to override that | 10:13 |
sean-k-mooney | the code you added woudl basically change that to threaded on systems without direct io | 10:13 |
sean-k-mooney | brinzhang: ok i would want to see at least a release not to call out the behavior change | 10:14 |
brinzhang | Yeah, we test it in iscsi, change "native" to "thread" on my env, it's ok on the function, | 10:15 |
sean-k-mooney | and ideally i think we would also want a warning in init_host signally to the operator that we are falling back to threaded io | 10:15 |
sean-k-mooney | brinzhang: threaded i think will always be supported as it was teh legacy way fo doing io | 10:15 |
sean-k-mooney | so ya it should work with isci | 10:16 |
brinzhang | sean-k-mooney: thanks reply my doubt | 10:16 |
sean-k-mooney | brinzhang: ill leave some comment on the review but i think i would be ok with this as the direction for the fix. it will enable most cases to just work and as long as we warn the opertaotr of the behavior change i think its ok | 10:17 |
brinzhang | sean-k-mooney: yeah, thanks. | 10:18 |
*** priteau has quit IRC | 10:20 | |
*** pcaruana has joined #openstack-nova | 10:24 | |
*** artom has joined #openstack-nova | 10:36 | |
*** tbachman has quit IRC | 10:40 | |
openstackgerrit | Maciej Kucia proposed openstack/nova master: SR-IOV passthrough: Check PF only if VF is enabled https://review.opendev.org/476642 | 11:27 |
openstackgerrit | sunjiyun-inspur proposed openstack/nova master: query diff user in same project https://review.opendev.org/691849 | 11:27 |
*** arxcruz is now known as arxcruz|lunch | 11:37 | |
*** damien_r has quit IRC | 11:49 | |
*** ociuhandu has joined #openstack-nova | 11:51 | |
*** ociuhandu has quit IRC | 11:55 | |
*** jaosorior has joined #openstack-nova | 11:56 | |
*** markvoelker has joined #openstack-nova | 11:59 | |
*** brinzhang_ has joined #openstack-nova | 12:02 | |
*** mkrai has quit IRC | 12:03 | |
*** markvoelker has quit IRC | 12:03 | |
*** brinzhang has quit IRC | 12:05 | |
*** markvoelker has joined #openstack-nova | 12:05 | |
*** damien_r has joined #openstack-nova | 12:05 | |
*** tbachman has joined #openstack-nova | 12:06 | |
*** larainema has quit IRC | 12:22 | |
*** belmoreira has joined #openstack-nova | 12:27 | |
*** dpawlik has quit IRC | 12:27 | |
*** jaosorior has quit IRC | 12:39 | |
*** pcaruana has quit IRC | 12:45 | |
*** jmlowe has quit IRC | 12:48 | |
*** munimeha1 has quit IRC | 12:51 | |
*** xek has joined #openstack-nova | 12:55 | |
*** arxcruz|lunch is now known as arxcruz | 12:56 | |
*** dklyle has quit IRC | 12:57 | |
*** david-lyle has joined #openstack-nova | 12:57 | |
*** pcaruana has joined #openstack-nova | 12:58 | |
*** dpawlik has joined #openstack-nova | 13:04 | |
*** dave-mccowan has joined #openstack-nova | 13:05 | |
*** jmlowe has joined #openstack-nova | 13:10 | |
*** liuyulong has joined #openstack-nova | 13:11 | |
*** sridharg has quit IRC | 13:11 | |
openstackgerrit | Arthur Dayne proposed openstack/nova master: libvirt:driver:Disallow AIO=native when 'O_DIRECT' is not available https://review.opendev.org/682772 | 13:11 |
*** francoisp has joined #openstack-nova | 13:22 | |
openstackgerrit | Arthur Dayne proposed openstack/nova master: libvirt:driver:Disallow AIO=native when 'O_DIRECT' is not available https://review.opendev.org/682772 | 13:24 |
*** nweinber has joined #openstack-nova | 13:27 | |
*** david-lyle is now known as dklyle | 13:28 | |
*** mriedem has joined #openstack-nova | 13:29 | |
*** jaosorior has joined #openstack-nova | 13:47 | |
*** pcaruana has quit IRC | 13:52 | |
*** pcaruana has joined #openstack-nova | 13:55 | |
*** Luzi has quit IRC | 13:55 | |
openstackgerrit | sean mooney proposed openstack/nova stable/stein: make config drives sticky bug 1835822 https://review.opendev.org/691864 | 13:56 |
openstack | bug 1835822 in OpenStack Compute (nova) "vms loose acess to config drive with CONF.force_config_drive=True after hard reboot" [Medium,Fix released] https://launchpad.net/bugs/1835822 - Assigned to sean mooney (sean-k-mooney) | 13:56 |
*** jaosorior has quit IRC | 13:59 | |
mriedem | huh, the changePassword server action api requires you specify adminPass but the value can be an empty string, in which case the server (nova-compute service) generates a password, but its not returned to the caller... | 14:02 |
mriedem | that's....helpful | 14:02 |
*** jaosorior has joined #openstack-nova | 14:03 | |
*** brinzhang_ has quit IRC | 14:03 | |
sean-k-mooney | very much so i can see why some people migt find that irrating | 14:03 |
*** brinzhang_ has joined #openstack-nova | 14:04 | |
sean-k-mooney | also i abandond ^ since i notice the other config dirve backports were abandoned too | 14:04 |
efried | mriedem: to what extent is it allowable to run osc version $later against a cloud at version $earlier ? | 14:05 |
ganso | Hi folks. I am about to open a new nova bug that my customer is experiencing. They are running "openstack usage list --start 2019-08-01 --end 2019-08-31 --os-compute-api-version 2.40" and it stays in infinite loop. With --debug, I could see that the marker is never updated, therefore the server always sends the same page result over and over. The env is Queens, this was tested with OpenStack CLI 3.19 and 4.0. I decided to check here in case someone | 14:05 |
ganso | has seen this problem before and it is an obvious one before I open the bug. Unfortunately I don't have access to the environment so it is hard for me to collect more data, and I couldn't reproduce in my lab. So, has anyone seen this weird behavior before? | 14:05 |
mriedem | so i guess the question is, would we consider it a bug that you can generate and change the admin password on a guest without getting the value back? | 14:05 |
mriedem | because if that's a bug, we might not require a microversion to change it | 14:05 |
*** brinzhang_ has quit IRC | 14:05 | |
*** dklyle has quit IRC | 14:05 | |
mriedem | efried: osc is meant to run against old server versions | 14:05 |
*** brinzhang_ has joined #openstack-nova | 14:06 | |
mriedem | dtroyer is pretty adament about backward compat with old clouds | 14:06 |
mriedem | even for like nova-network support | 14:06 |
efried | how much older? The real question is: when can we drop nova-network support from osc? | 14:06 |
efried | d'oh | 14:06 |
mriedem | why are you asking? | 14:06 |
mriedem | we dropped nova-network support from novaclient and dean had to fix that all up in osc to continue supporting nova-net | 14:06 |
efried | because osc generates option helps based on whether it detects the 'network' (neutron) service or not. | 14:07 |
sean-k-mooney | can you tell form the api that it is nova networks? | 14:07 |
efried | which is fine when you're running in a cloud | 14:07 |
efried | but not so much when you're generating html docs to publish. | 14:07 |
*** brinzhang_ has quit IRC | 14:07 | |
*** dklyle has joined #openstack-nova | 14:07 | |
sean-k-mooney | efried: well the sphinx docs for osc should generate for all intree options right | 14:07 |
mriedem | ganso: never heard of it but i wouldn't be surprised with markers, though it would be good to know more details about the marker, e.g. if it's a deleted server | 14:08 |
sean-k-mooney | and then plugins have there own docs | 14:08 |
*** links has joined #openstack-nova | 14:08 | |
sean-k-mooney | efried: do you wnat to skip the nova networks docs? | 14:08 |
mriedem | efried: option help or entire commands? | 14:08 |
efried | both neutron and nova-net options are in osc proper, non-plugin. Yes, I would love to be able to code that decision as: "If I'm not running in a cloud, assume I'm building docs, and since nova-network is dead, only generate the neutron options" | 14:09 |
efried | mriedem: uhm, not totally sure about that, would have to dig a bit more. | 14:09 |
mriedem | i'd say either drop nova-network stuff from published docs or you'd have to mark anything as nova-net as nova-net specific i guess? which probably sucks. | 14:09 |
ganso | mriedem: hmm that is an interesting detail. So, the marker I have is exactly the last instance returned from the first request. In the dict returned by the API, the instance is marked as "active". The CLI then uses this instance's ID for the next request URL, but then the API just returns the same page, with the same instance as last one. And over and over. | 14:10 |
efried | right | 14:10 |
sean-k-mooney | efried: given the last time i deployed with nova net osc was not a thing i have never seen the nova net commands/opetions do you have an example | 14:11 |
ganso | mriedem: Even though the instance is marked as "active" in the returned API, it is possible that the instance has been deleted but was active in the queried period? | 14:11 |
efried | sean-k-mooney: I'm not sure whether the docs have any of those today. Which would make my job easier, as what I'm doing will just produce the equivalent. Let me go dig some... | 14:12 |
efried | sean-k-mooney: fyi https://review.opendev.org/#/c/691767/1/openstackclient/network/common.py@174 | 14:12 |
*** dklyle has quit IRC | 14:12 | |
mriedem | ganso: i would expect that if the instance is deleted then it's vm_state is not active | 14:13 |
*** eharney has quit IRC | 14:14 | |
mriedem | or you're really unlucky in hitting a tight race window | 14:14 |
mriedem | i just asked about deleted markers because we've seen bugs with those so i wouldn't be surprised if that is the issue | 14:14 |
ganso | mriedem: hmm ok so the usage returns the current instance state. About races, the customer can consistently reproduce this, it happens every time | 14:15 |
mriedem | do you have this fix? https://review.opendev.org/#/c/632516/ | 14:15 |
mriedem | that is in queens 17.0.10 | 14:15 |
ganso | mriedem: yes. Customer is running 17.0.11 | 14:16 |
sean-k-mooney | efried: for what its worth looking at https://docs.openstack.org/python-openstackclient/pike/cli/command-list.html and drilling down it port and network resouce i dont see nova network sepcific commands | 14:16 |
mriedem | https://docs.openstack.org/python-openstackclient/pike/cli/command-objects/server.html#server-ssh is nova-net specific | 14:17 |
efried | sean-k-mooney: one example is https://docs.openstack.org/python-openstackclient/latest/cli/command-objects/security-group.html#security-group-list | 14:17 |
efried | In this case, the nova-network options (just --all-projects) are a subset of the neutron options (which is what the doc has) | 14:17 |
efried | And now I'm noticing that they're tagged. | 14:17 |
sean-k-mooney | mriedem: you can use the ssh command with neutron | 14:17 |
sean-k-mooney | mriedem i use it all the time when im too lazy to lookup the ip | 14:17 |
sean-k-mooney | efried why is that nova networks specific. it should list the neutorn security groups | 14:19 |
sean-k-mooney | and --all-project shoudl still work in that context if you are an admin | 14:19 |
efried | sean-k-mooney: I'm saying, this is an example where the same subcommand has "different" options for nova-net vs neutron. | 14:19 |
efried | except they overlap | 14:19 |
sean-k-mooney | ah right | 14:19 |
efried | except the current doc says --all-projects is ignored | 14:20 |
sean-k-mooney | Network version 2 ignores this option and will always display information for all projects (admin only). | 14:20 |
efried | which doesn't gel with what the opt help says | 14:20 |
sean-k-mooney | that is the delta | 14:20 |
efried | oh, yes it does | 14:20 |
sean-k-mooney | people might not click that "networkign v2" mean neutron v2 | 14:20 |
efried | yeah, no kidding | 14:20 |
efried | okay, so nova-net only supports --all-projects, and neutron supports all the others but ignores --all-projects. (But neutron still adds the opt so it won't blow up if you use it.) | 14:22 |
sean-k-mooney | efried: ya that seam to be the case | 14:23 |
efried | okay, so to make this work the way I want, to get actual parity, I would have to produce both helps, marked with their respective support statements :( | 14:25 |
dansmith | ...which is what we should do if we're responsible backwards-compatible software engineers amirite? | 14:25 |
efried | yes -- again assuming we still want an ussuri-level osc to support nova-net | 14:26 |
dansmith | I think you mean s/ussuri-level// | 14:26 |
efried | python-openstackclient is cycle-with-intermediary. Its deliverables are aligned with openstack releases. I'm making changes to osc that will release with ussuri. So I think I mean, if it's legal to deploy an ussuri-level osc in a cloud that still runs nova-net... | 14:28 |
*** Sundar has joined #openstack-nova | 14:28 | |
dansmith | well, I think the point is.. it's a client and I think the goal for most is to have clients be backwards compatible with stable APIs | 14:29 |
efried | but nova-net was only deprecated in newton. Which means there will be people who will run it until it don't run no mo. Which means stein. And surely you can run ussuri osc against a stein cloud. | 14:30 |
efried | So I can assume removing nova-net support from osc is a ways off in the future, and I have to deal with it. | 14:30 |
sean-k-mooney | efried: dansmith as a compromise assuming moveing to the sdk in ocs contindues wehn the major version bump fo that happens we could drop nova-net support as part of that | 14:31 |
dansmith | efried: fwiw just the other day I was talking to someone running a very large production mitaka cloud, working to deploy train for their next phase.. it would suck super bad to not have a client be able to talk to both of them | 14:31 |
efried | for now I don't actually care "when" or even "if"; all I needed to know was "not now". | 14:31 |
*** Liang__ has joined #openstack-nova | 14:31 | |
efried | dansmith: ack | 14:31 |
efried | btw, for what I'm working on now, it's not a question of dropping actual support for nova-net. It's just a question of whether docs.o.o/osc/ussuri{and later}/ will contain the options for nova-net. | 14:33 |
efried | but the answer is the same. | 14:33 |
dansmith | well, if it's in the client and not documented (even if not generated) that also kinda sucks, but less so, I'll agree | 14:34 |
*** dviroel has joined #openstack-nova | 14:34 | |
efried | dansmith: the CLI helps only produce the content for one or the other, by detecting whether the 'network' (neutron) endpoint exists. | 14:35 |
dansmith | ...right | 14:35 |
efried | but of course we're not in your cloud when we're generating docs | 14:36 |
dansmith | sure | 14:36 |
efried | thanks for the talk, all. The path forward is clear, if messy. | 14:36 |
dansmith | mriedem: are you looking for me to pass all the notification payload things in via init, or only the two things you noted in the review? | 14:40 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove dead set_admin_password code to generate password https://review.opendev.org/691873 | 14:44 |
*** mkrai_ has joined #openstack-nova | 14:45 | |
mriedem | dansmith: just the few things noted | 14:51 |
mriedem | the image lists are generated after the fact | 14:51 |
*** mlavalle has joined #openstack-nova | 14:51 | |
dansmith | mriedem: okay I'm not sure I see the distinction, but that's fine | 14:51 |
mriedem | it was a nit, ignore if you want | 14:53 |
*** mkrai_ has quit IRC | 14:54 | |
dansmith | it's already done, I just wanted to know the convention for next time, but if it's just "feels right" then that's fine too | 14:54 |
*** slaweq has quit IRC | 14:54 | |
*** slaweq has joined #openstack-nova | 14:56 | |
*** Liang__ has quit IRC | 14:57 | |
*** gyee has joined #openstack-nova | 14:57 | |
*** mkrai has joined #openstack-nova | 14:57 | |
*** eharney has joined #openstack-nova | 14:59 | |
*** Liang__ has joined #openstack-nova | 14:59 | |
*** xek_ has joined #openstack-nova | 15:00 | |
*** ociuhandu has joined #openstack-nova | 15:00 | |
*** dklyle has joined #openstack-nova | 15:01 | |
*** links has quit IRC | 15:02 | |
*** Sundar has quit IRC | 15:02 | |
*** xek has quit IRC | 15:02 | |
mriedem | like the spanx i'm wearing right now, it just feels right | 15:03 |
* dansmith nods | 15:04 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove dead set_admin_password code to generate password https://review.opendev.org/691873 | 15:04 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix changePassword adminPass="" broken schema validation https://review.opendev.org/691880 | 15:04 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix changePassword adminPass="" broken schema validation https://review.opendev.org/691880 | 15:06 |
*** ociuhandu has quit IRC | 15:07 | |
mriedem | oh god how many duplicates do we have of this bug https://bugs.launchpad.net/nova/+bug/1850153 | 15:09 |
openstack | Launchpad bug 1850153 in OpenStack Compute (nova) "nova resize operation doesn't support disk resize for emphemeral disk and swap disk" [Undecided,New] | 15:09 |
efried | mriedem: so passing in an empty password does *not* generate? | 15:09 |
mriedem | efried: nope | 15:09 |
mriedem | and you can't pass adminPass=None | 15:09 |
efried | passing in null theoretically would have, but that gets kicked by schema... yeah. | 15:09 |
efried | adminPass="" will actually set the password to the empty string. | 15:10 |
sean-k-mooney | yep | 15:10 |
sean-k-mooney | peopel try to cheat schema validation on the neutron side setting "" | 15:10 |
mriedem | which is probably a security vulnerability | 15:10 |
sean-k-mooney | to pass null and end up setting it to the litral empty sting | 15:10 |
mriedem | as noted in the commit, the changePassword API originally did not allow "" | 15:11 |
mriedem | it was regressed when jsonschema was applied | 15:11 |
*** mdbooth has quit IRC | 15:11 | |
mriedem | it wasn't caught because there was no test for empty string | 15:11 |
sean-k-mooney | i suppose it would not be that hard to validate the lenght of the sting | 15:11 |
*** ociuhandu has joined #openstack-nova | 15:12 | |
*** mdbooth has joined #openstack-nova | 15:12 | |
efried | isn't there something where setting empty password causes it to prompt you to set a real password the next time you log in? | 15:13 |
efried | maybe I'm remembering some old shell hack from 20y ago. | 15:13 |
sean-k-mooney | ya so we should proably update https://github.com/openstack/nova/blob/master/nova/api/validation/parameter_types.py#L364-L371 | 15:14 |
mriedem | sean-k-mooney: do you see the patch i proposed that just scrolled above? | 15:15 |
sean-k-mooney | e.g. add 'minLenght':1 | 15:15 |
sean-k-mooney | mriedem: nope | 15:15 |
efried | mriedem: sorry, where in https://review.opendev.org/#/c/35625/ is a nonempty password enforced? | 15:16 |
sean-k-mooney | i assume it does that or something similar | 15:16 |
sean-k-mooney | https://review.opendev.org/#/c/691880/ | 15:16 |
mriedem | efried: https://review.opendev.org/#/c/35625/10/nova/api/openstack/compute/plugins/v3/admin_password.py@61 | 15:16 |
mriedem | efried: oh well i guess that would be True | 15:16 |
mriedem | "" is a basestring | 15:16 |
efried | mriedem: that enforces ... yeah | 15:16 |
efried | so empty string has always been allowed | 15:17 |
*** liuyulong has quit IRC | 15:17 | |
openstackgerrit | Dan Smith proposed openstack/nova master: Log some stats for image pre-cache https://review.opendev.org/688173 | 15:18 |
openstackgerrit | Dan Smith proposed openstack/nova master: Add Aggregate image caching progress notifications https://review.opendev.org/691390 | 15:18 |
sean-k-mooney | the admin password requires the qemu guest agent to work too right. | 15:18 |
*** dpawlik has quit IRC | 15:18 | |
sean-k-mooney | e.g. by defualt it wont work unless you enable it and install it in your guest image at least for libvirt | 15:19 |
*** Liang__ has quit IRC | 15:19 | |
efried | Which is kinda okay imo, I'm not convinced it's always wrong to use "". | 15:19 |
mriedem | looks like that was originally this https://github.com/openstack/nova/blob/icehouse-eol/nova/api/openstack/compute/servers.py#L1490 | 15:19 |
dansmith | gibi: mriedem: are either of you aware of any example notification-consuming code out there? thinking I might write up an example tool that caches images and watches for completion via notifications | 15:19 |
sean-k-mooney | efried: well at least on the linux side it can be locked down via ssh/pam setting in the guest | 15:19 |
mriedem | and this is where the generate password stuff happened https://github.com/openstack/nova/blob/icehouse-eol/nova/api/openstack/compute/servers.py#L1484 | 15:20 |
gibi | dansmith: no I don't know about such tool | 15:20 |
*** ociuhandu has quit IRC | 15:20 | |
mriedem | so the regression was actually that you could specify adminPass=None and it would generate a password for you, which is probably how that got down into the compute manager code | 15:20 |
dansmith | gibi: bummer, okay | 15:20 |
mriedem | gibi: your versioned notifications talk had a project on github you wip'ed up for consuming notifications didn't you? | 15:21 |
mriedem | in devstack | 15:21 |
gibi | mriedem: yeah that was an example tool | 15:21 |
gibi | mriedem: nothing production ready | 15:21 |
*** Sundar has joined #openstack-nova | 15:21 | |
dansmith | gibi: not looking for production, looking for example :) | 15:21 |
gibi | https://github.com/gibizer/nova-notification-demo | 15:21 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Pass RequestContext to oslo_policy https://review.opendev.org/674038 | 15:21 |
gibi | dansmith: sorry I skipped the 'example' word in your question | 15:22 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing admin_actions policies https://review.opendev.org/657698 | 15:22 |
dansmith | gibi: thanks, I'll see if I can steal enough out of that to get something working | 15:22 |
*** ociuhandu has joined #openstack-nova | 15:22 | |
gibi | dansmith: I'm happy to help | 15:23 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing os-services policies https://review.opendev.org/669181 | 15:23 |
*** jawad_ax_ has quit IRC | 15:24 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in os-services https://review.opendev.org/645427 | 15:24 |
*** jawad_axd has joined #openstack-nova | 15:25 | |
*** ociuhandu has quit IRC | 15:26 | |
*** jawad_axd has quit IRC | 15:29 | |
*** xek__ has joined #openstack-nova | 15:41 | |
mriedem | is anyone against just putting something in the API reference about https://bugs.launchpad.net/nova/+bug/1558880 being a known limitation? there are at least 3 known duplicates of that bug now and i don't see it being fixed anytime soon. | 15:42 |
openstack | Launchpad bug 1558880 in OpenStack Compute (nova) "instance can not resize ephemeral from mitaka to stein and master" [Medium,Confirmed] | 15:42 |
*** xek_ has quit IRC | 15:43 | |
*** mkrai has quit IRC | 15:45 | |
*** slaweq has quit IRC | 15:49 | |
*** slaweq has joined #openstack-nova | 15:54 | |
gibi | mriedem: I agree to document it as a known limitation | 15:54 |
*** jmlowe has quit IRC | 15:55 | |
*** mgariepy has quit IRC | 15:57 | |
*** nanzha has quit IRC | 15:58 | |
*** mgariepy has joined #openstack-nova | 15:59 | |
*** nanzha has joined #openstack-nova | 15:59 | |
*** igordc has joined #openstack-nova | 16:01 | |
*** mkrai has joined #openstack-nova | 16:01 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use admin neutron client to gather port resource requests https://review.opendev.org/691005 | 16:02 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use admin neutron client to see if instance has qos ports https://review.opendev.org/691900 | 16:04 |
efried | mriedem: This looks weird https://docs.openstack.org/api-ref/compute/?expanded=list-hypervisors-detail#list-hypervisors | 16:08 |
mriedem | caller, can you be more specific? | 16:08 |
efried | 'marker' is listed twice, once as integer, once as string. In the former it says it's available until 2.52 and also new in 2.53 | 16:08 |
mriedem | it's because marker and id changed from an int id to a uuid in 2.53 | 16:09 |
mriedem | for multi-cell listing | 16:09 |
efried | oh, sorry, I misread 2.33 as 2.53. | 16:09 |
mriedem | iow, the int id is not unique across cells | 16:09 |
efried | This looks fine. | 16:09 |
* efried crawls back into hole. | 16:09 | |
efried | mriedem: but while I have your attention... adding support for new qparams into SDK, who's responsible for that? | 16:10 |
efried | do we have like a known "we have SDK support up to XXX microversion" somewhere? | 16:10 |
mriedem | nope | 16:11 |
efried | e.g. with_servers in that same API doesn't seem to exist in the SDK compute proxy's `hypervisors()` method. | 16:11 |
mriedem | i did that for osc | 16:11 |
mriedem | https://etherpad.openstack.org/p/compute-api-microversion-gap-in-osc | 16:12 |
efried | ack | 16:12 |
mriedem | getting anyone to care about that is also pretty non-starty | 16:12 |
efried | but you're not surprised that `with_servers` isn't in the SDK yet? | 16:12 |
mriedem | hell no | 16:12 |
efried | okay. | 16:12 |
mriedem | as i've said before, the only thing that surprises me in openstack anymore is when things work | 16:12 |
*** ivve has quit IRC | 16:13 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Reset instance to current vm_state if rolling back in resize_instance https://review.opendev.org/691908 | 16:17 |
*** igordc has quit IRC | 16:18 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use admin neutron client to gather port resource requests https://review.opendev.org/691005 | 16:19 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use admin neutron client to see if instance has qos ports https://review.opendev.org/691900 | 16:20 |
*** jaosorior has quit IRC | 16:24 | |
*** nanzha has quit IRC | 16:31 | |
*** Sundar has quit IRC | 16:35 | |
*** xek__ has quit IRC | 16:36 | |
*** tesseract has quit IRC | 16:38 | |
*** damien_r has quit IRC | 16:43 | |
*** lpetrut has quit IRC | 16:49 | |
*** AJaeger has joined #openstack-nova | 16:51 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add known limitation about resize not resizing ephemeral disks https://review.opendev.org/691915 | 16:51 |
AJaeger | nova team, could I get a second +2 on https://review.opendev.org/#/c/689020/ to update the openSUSE Zuul job, please? Once that is in, I'll backport as well - these are needed to move forward on the infra side. | 16:52 |
AJaeger | thanks, dansmith ! | 16:56 |
dansmith | AJaeger: np | 16:56 |
*** nanzha has joined #openstack-nova | 16:59 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: re-work resize action post-conditions https://review.opendev.org/691918 | 17:00 |
*** maciejjozefczyk has quit IRC | 17:04 | |
*** nanzha has quit IRC | 17:05 | |
*** tbachman has quit IRC | 17:14 | |
*** igordc has joined #openstack-nova | 17:16 | |
gmann | johnthetubaguy melwitt: replied on unified limit spec added comment of hangout call. this lgtm now. I think it is all you have captured in spec but it will be good if you can explain the plan per cycle (what we discussed in hangout, i wrote it in comment). also you need to change the cycle name in spec -https://review.opendev.org/#/c/602201/15/specs/ussuri/approved/unified-limits-nova.rst@523 | 17:18 |
*** jmlowe has joined #openstack-nova | 17:19 | |
*** ivve has joined #openstack-nova | 17:21 | |
*** rpittau is now known as rpittau|afk | 17:25 | |
*** jawad_axd has joined #openstack-nova | 17:26 | |
*** gmann is now known as gmann_afk | 17:29 | |
sean-k-mooney | gmann: "- at some cycle in future - remove the APIs." | 17:30 |
sean-k-mooney | gmann_afk: have we ever actully got around to removeing an api since we adopted micorverions | 17:31 |
sean-k-mooney | e.g. you can always fall back to 2.0 and get them | 17:31 |
dansmith | sean-k-mooney: we have removed the network stuff, AFAIK | 17:32 |
dansmith | returning 409 Gone for those | 17:32 |
sean-k-mooney | the nova net apis? | 17:32 |
sean-k-mooney | ya i guess | 17:33 |
sean-k-mooney | we just usually keep them for quite a long time | 17:33 |
*** tbachman has joined #openstack-nova | 17:33 | |
*** gmann_afk is now known as gmann | 17:35 | |
gmann | sean-k-mooney: removing API i mean 410 and keep the endpoints. | 17:36 |
gmann | like nova-net, cloudpipe, nova-cert APIs. but yes when it will happen do not know | 17:36 |
sean-k-mooney | i guess there have been a few | 17:37 |
sean-k-mooney | ya that is what i assumed you ment | 17:37 |
gmann | i clarified in comment. | 17:38 |
*** ociuhandu has joined #openstack-nova | 17:40 | |
*** dtantsur is now known as dtantsur|afk | 17:41 | |
*** ociuhandu has quit IRC | 17:45 | |
*** mkrai has quit IRC | 17:57 | |
mriedem | dansmith: you're going to have to recheck your top notification patch on https://bugs.launchpad.net/nova/+bug/1850514 | 18:05 |
openstack | Launchpad bug 1850514 in OpenStack Compute (nova) "ReshapeForPCPUsTest.test_vcpu_to_pcpu_reshape intermittently fails with "Cannot 'migrate' instance while it is in vm_state building"" [Medium,Confirmed] | 18:05 |
dansmith | yeah I know | 18:05 |
mriedem | ok, posting a fix for that bug | 18:11 |
mriedem | it's using the _wait_for_state_change i hate | 18:11 |
mriedem | but all of the libvirt functional tests use | 18:12 |
*** ociuhandu has joined #openstack-nova | 18:12 | |
*** tbachman has quit IRC | 18:17 | |
*** ociuhandu has quit IRC | 18:18 | |
*** tbachman has joined #openstack-nova | 18:18 | |
*** tbachman has quit IRC | 18:23 | |
*** jaosorior has joined #openstack-nova | 18:24 | |
*** jaosorior has quit IRC | 18:26 | |
*** nweinber_ has joined #openstack-nova | 18:27 | |
*** jmlowe has quit IRC | 18:28 | |
*** nweinber has quit IRC | 18:29 | |
*** tbachman has joined #openstack-nova | 18:30 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix race in test_vcpu_to_pcpu_reshape https://review.opendev.org/691941 | 18:33 |
mriedem | dansmith: efried: ^ fixes that gate bug | 18:33 |
*** elod has quit IRC | 18:33 | |
*** elod has joined #openstack-nova | 18:33 | |
*** jmlowe has joined #openstack-nova | 18:33 | |
*** abaindur has quit IRC | 18:36 | |
*** nweinber__ has joined #openstack-nova | 18:45 | |
*** nweinber_ has quit IRC | 18:47 | |
dansmith | mriedem: I haven't looked, but why does that fix it? because it might catch the ACTIVE before the transition or something? | 18:48 |
efried | dansmith: yes, that's how I understand it. | 18:51 |
efried | because it catches "not ACTIVE" immediately, rather. | 18:52 |
efried | mriedem: +2 | 18:52 |
dansmith | yeah okay | 18:56 |
dansmith | oh I should read the damn commit message :P | 18:57 |
mriedem | always start with the commit message... | 19:00 |
mriedem | at least on my patches | 19:00 |
dansmith | well, I like to look at code first, but sometimes forget to go back | 19:00 |
dansmith | mriedem: thanks for recheckin' mah thing | 19:01 |
mriedem | hey, it's my pleasure | 19:01 |
mriedem | ;P | 19:01 |
*** spatel has joined #openstack-nova | 19:02 | |
*** spatel has quit IRC | 19:02 | |
*** tbachman has quit IRC | 19:02 | |
mriedem | brinzhang keeps pushing this spec since berlin and i just don't see the justification for it in case others want to pile on https://review.opendev.org/#/c/580336/ | 19:05 |
mriedem | it also requires changing what is now the swap volume only api | 19:05 |
mriedem | which could get a bit weird | 19:05 |
*** Guest85345 is now known as benj_ | 19:08 | |
*** ralonsoh has quit IRC | 19:08 | |
*** abaindur has joined #openstack-nova | 19:16 | |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Add functional regression test for bug 1849409 https://review.opendev.org/690720 | 19:25 |
openstack | bug 1849409 in OpenStack Compute (nova) rocky "openstack server list --deleted --limit -1 hangs" [High,In progress] https://launchpad.net/bugs/1849409 - Assigned to Matt Riedemann (mriedem) | 19:25 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Fix listing deleted servers with a marker https://review.opendev.org/690721 | 19:25 |
mriedem | melwitt: i worked around the rocky-only intermittent failure you noticed in ^ | 19:27 |
*** eharney has quit IRC | 19:29 | |
artom | mriedem, re: brinzhang's spec - isn't that heat's job? | 19:35 |
artom | (Not updating volume attachments, deleting related things as a unit) | 19:38 |
artom | (Which is what they want... right?) | 19:38 |
artom | Oh, just read your last comment | 19:39 |
dansmith | artom: this is more about instances that were not created with the flag set the way they want it, right/ | 19:39 |
artom | I was thinking "make this volume go away when its instance does" | 19:40 |
dansmith | right but you can already do that today | 19:40 |
dansmith | if you get the flag set when you create the instance | 19:40 |
artom | Not "oops, I messed up, please keep this volume when its instance goes away" | 19:40 |
dansmith | if you don't, then you might want to change it later, which I thought is the point of this spec | 19:40 |
artom | dansmith, right, they want to set it after | 19:40 |
artom | ... besides the wrong API stuff that mriedem raised, seems reasonable to me... | 19:42 |
*** panda|pto has quit IRC | 19:55 | |
*** panda has joined #openstack-nova | 19:58 | |
*** elod has quit IRC | 20:02 | |
*** elod has joined #openstack-nova | 20:02 | |
mriedem | the use case is you boot from volume with delete_on_termination=True, do some awesome stuff in the volume and then realize you don't want it deleted when you delete the server so you want to change delete_on_termination=False, | 20:08 |
mriedem | which you can do since 2.79 with non-root volumes, | 20:08 |
mriedem | and you can create a snapshot of the server which will create volume snapshots since forever | 20:09 |
mriedem | so it seems like a waste of effort to bake that all into an API that is already solely just dealing with swap volume semantics | 20:09 |
*** AJaeger has left #openstack-nova | 20:10 | |
mriedem | obviously we *can* do this but i'm saying it doesn't seem justifiable that we *should* do it | 20:10 |
gmann | efried: can you remove -2 on this, spec is merged and I am rebasing the series now - https://review.opendev.org/#/c/674038/ | 20:11 |
mriedem | another spec that is a good example of things we can do but i'm not sure we should https://review.opendev.org/#/c/682302/ | 20:12 |
mriedem | but we kind of opened pandora's box when adding changePassword in the first place | 20:12 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles and mapping in policy base class https://review.opendev.org/645452 | 20:13 |
*** dpawlik has joined #openstack-nova | 20:14 | |
mriedem | heh http://lists.openstack.org/pipermail/openstack-discuss/2019-October/010429.html | 20:15 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add policy deprecation fixture and Suppress warnings in tests https://review.opendev.org/676670 | 20:15 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in os-services API policies https://review.opendev.org/648480 | 20:15 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Pass the target in os-services APIs policy https://review.opendev.org/676688 | 20:16 |
*** dpawlik has quit IRC | 20:19 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing admin_actions policies https://review.opendev.org/657698 | 20:20 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in Admin Actions https://review.opendev.org/657823 | 20:20 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in Admin Action API policies https://review.opendev.org/676682 | 20:20 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Ensure we pass a target in admin actions https://review.opendev.org/663095 | 20:21 |
efried | gmann: done | 20:23 |
gmann | johnthetubaguy: melwitt ^^ rebased the policy patches. this and all its base patches are one set of os-services API policy change - https://review.opendev.org/#/c/676688/2 | 20:24 |
gmann | efried: thanks | 20:24 |
*** CeeMac has quit IRC | 20:27 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing admin_actions policies https://review.opendev.org/657698 | 20:27 |
*** eharney has joined #openstack-nova | 20:27 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in Admin Actions https://review.opendev.org/657823 | 20:28 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in Admin Action API policies https://review.opendev.org/676682 | 20:28 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Ensure we pass a target in admin actions https://review.opendev.org/663095 | 20:28 |
*** markvoelker has quit IRC | 20:35 | |
*** markvoelker has joined #openstack-nova | 20:45 | |
*** nweinber__ has quit IRC | 20:53 | |
*** dtruong has quit IRC | 21:01 | |
*** dtruong has joined #openstack-nova | 21:01 | |
*** pcaruana has quit IRC | 21:15 | |
openstackgerrit | sean mooney proposed openstack/nova master: Add transform_image_metadata request filter https://review.opendev.org/665775 | 21:21 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add cross-cell resize policy rule and enable in API https://review.opendev.org/638269 | 21:40 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Enable cross-cell resize in the nova-multi-cell job https://review.opendev.org/656656 | 21:40 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add negative test to delete server during cross-cell resize claim https://review.opendev.org/688832 | 21:40 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: flatten rbd image during cross-cell move spawn at dest https://review.opendev.org/691991 | 21:40 |
mriedem | gibi: when you get a chance, can you come back on these cross-cell resize patches that you were +2 on before but have been rebased https://review.opendev.org/#/c/635668/ https://review.opendev.org/#/c/635684/ | 21:42 |
*** elod has quit IRC | 21:45 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: flatten rbd image during cross-cell move spawn at dest https://review.opendev.org/691991 | 21:45 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add cross-cell resize policy rule and enable in API https://review.opendev.org/638269 | 21:45 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Enable cross-cell resize in the nova-multi-cell job https://review.opendev.org/656656 | 21:45 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add negative test to delete server during cross-cell resize claim https://review.opendev.org/688832 | 21:45 |
*** elod has joined #openstack-nova | 21:47 | |
*** slaweq has quit IRC | 21:52 | |
mriedem | smcginnis: btw, still waiting on https://review.opendev.org/#/c/687058/ for queens EM, it's been getting rechecked for a couple of weeks now - random unrelated failures and a long queue for nova to get ci run | 21:53 |
mriedem | i think it if continues to fail into tomorrow i say we just cut it and let that not be part of the last release, it's a rando 3rd party volume driver fix that is low priority | 21:53 |
smcginnis | mriedem: Thanks, I had meant to check up on things today. | 21:54 |
*** elod has quit IRC | 21:54 | |
smcginnis | Dang, (10 rechecks) | 21:54 |
*** elod has joined #openstack-nova | 21:56 | |
mriedem | dansmith: http://status.openstack.org/elastic-recheck/#1844929 continues to hurt the gate pretty badly - any ideas on ways that we could maybe punch/tickle the scheduler/cell db post-upgrade in grenade before running smoke tests to see if we can flush that out somehow? | 21:57 |
*** rcernin has joined #openstack-nova | 22:02 | |
*** mriedem has quit IRC | 22:06 | |
openstackgerrit | Merged openstack/nova master: Switch to opensuse-15 nodeset https://review.opendev.org/689020 | 22:06 |
*** gyee has quit IRC | 22:27 | |
openstackgerrit | Sundar Nadathur proposed openstack/nova-specs master: Fix a bug in the sequence diagram. https://review.opendev.org/692001 | 22:39 |
*** slaweq has joined #openstack-nova | 22:50 | |
*** gyee has joined #openstack-nova | 22:52 | |
*** slaweq has quit IRC | 22:54 | |
*** abaindur has quit IRC | 23:04 | |
*** abaindur has joined #openstack-nova | 23:05 | |
*** tkajinam has joined #openstack-nova | 23:08 | |
*** ivve has quit IRC | 23:13 | |
openstackgerrit | Merged openstack/nova stable/queens: Fixes multi-registry config in Quobyte driver https://review.opendev.org/687058 | 23:18 |
openstackgerrit | Merged openstack/nova master: Fix race in test_vcpu_to_pcpu_reshape https://review.opendev.org/691941 | 23:18 |
*** abaindur has quit IRC | 23:26 | |
*** luksky has quit IRC | 23:26 | |
*** abaindur has joined #openstack-nova | 23:27 | |
*** abaindur has quit IRC | 23:33 | |
*** mlavalle has quit IRC | 23:34 | |
*** tbachman has joined #openstack-nova | 23:45 | |
*** tkajinam has quit IRC | 23:48 | |
*** brinzhang has joined #openstack-nova | 23:49 | |
*** gyee has quit IRC | 23:54 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!