| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-08-21 | |||
| 13:28:03 | mriedem | and apparently no one has noticed since folsom | |
| 13:28:51 | openstackgerrit | Chris Dent proposed openstack/nova master: Set policy_opt defaults in placement gabbi fixture https://review.openstack.org/594172 | |
| 13:29:05 | mriedem | https://bugs.launchpad.net/python-novaclient/+bug/1295126/comments/3 | |
| 13:29:05 | openstack | Launchpad bug 1295126 in python-novaclient "Admin only shown for args that can be used by non-admin" [Wishlist,Fix released] - Assigned to Verónica Musso (veronica-a-musso) | |
| 13:29:10 | mriedem | "and --instance-name has no effect for both" | |
| 13:30:13 | mriedem | tssurya: alex_xu: i'd probably just deprecate the --instance-name option in nova list, it's not done anything since essex | |
| 13:30:23 | mriedem | adding the support server-side at this point is likely a microversion | |
| 13:30:45 | alex_xu | mriedem: yea, and it is admin-only filter so we can deprecate it | |
| 13:30:48 | tssurya | mriedem: ack, I don't think its that essential a filter | |
| 13:31:14 | tssurya | its not an admin-only.. | |
| 13:33:08 | alex_xu | tssurya: instance_name field only can be see by the admin? | |
| 13:35:24 | mriedem | no, | |
| 13:35:29 | mriedem | OS-EXT-SRV-ATTR:instance_name is also shown for non-admins | |
| 13:35:35 | mriedem | it's in ExtendedServerAttributesController | |
| 13:36:37 | mriedem | oh wait no alex_xu is correc | |
| 13:36:39 | mriedem | *correct | |
| 13:36:47 | mriedem | os_compute_api:os-extended-server-attributes defaults to admin-only | |
| 13:37:06 | mriedem | https://docs.openstack.org/nova/latest/configuration/policy.html | |
| 13:38:04 | mriedem | stephenfin: see https://docs.openstack.org/nova/latest/configuration/policy.html and os_compute_api:os-extended-server-attributes - i thought we had restructured text formatting on policy option help? | |
| 13:38:15 | mriedem | maybe that's only in oslo.config option help? | |
| 13:38:17 | mriedem | efried: ? ^ | |
| 13:39:00 | efried | mriedem: Patch not merged. Lemme grab it... | |
| 13:39:12 | tssurya | oh, well its confusing because the help for the options doesn't say its Admin only and the bug above ^ says they changed it: https://bugs.launchpad.net/python-novaclient/+bug/1295126/comments/6 | |
| 13:39:12 | openstack | Launchpad bug 1295126 in python-novaclient "Admin only shown for args that can be used by non-admin" [Wishlist,Fix released] - Assigned to Verónica Musso (veronica-a-musso) | |
| 13:39:30 | mriedem | tssurya: yup, | |
| 13:39:37 | mriedem | despite that one person saying it was never even used | |
| 13:39:45 | mriedem | tssurya: so just report a bug and deprecate --instance-name | |
| 13:39:47 | mriedem | i'll +2 that | |
| 13:39:54 | tssurya | mriedem: cool | |
| 13:40:02 | mriedem | it predates gerrit so i'm not surprised it's a mess | |
| 13:40:41 | stephenfin | mriedem: Yeah, just oslo.config, I think | |
| 13:40:58 | stephenfin | though I had it in my head oslo.policy wasn't broken in the first place. Obviously not | |
| 13:41:14 | efried | mriedem: | |
| 13:41:14 | efried | - nova patch to twiddle a couple of options to prove it works | |
| 13:41:14 | efried | - oslo.config patch to address complaint that using the rst role in help text shows up ugly in the sample: https://review.openstack.org/#/c/583064/ | |
| 13:41:25 | efried | https://review.openstack.org/#/c/583025/ shoulda been that first link, sorry | |
| 13:41:42 | stephenfin | efried: I think that's a different issue | |
| 13:42:08 | stephenfin | efried: mriedem's asking why newlines and the likes in policy.help aren't being parsed | |
| 13:42:24 | stephenfin | ...in the HTML output. Your patch affects the ini output, right? | |
| 13:43:18 | tssurya | okay, so instance.display_name is name and instance.hostname is OS-EXT-SRV-ATTR:hostname and we don't care about OS-EXT-SRV-ATTR:instance_name. | |
| 13:43:22 | efried | stephenfin: the oslo.config patch, yes. | |
| 13:44:01 | efried | oh, reread what mriedem was actually saying. Yeah, I don't know about that, sorry. | |
| 13:44:09 | efried | I would have asked stephenfin :) | |
| 13:44:28 | mriedem | ha | |
| 13:44:58 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Filter out instances without a host when populating AZ https://review.openstack.org/594050 | |
| 13:45:17 | mriedem | ^ is likely an RC3 issue | |
| 13:47:51 | mriedem | regarding petr's email about install guide testing, | |
| 13:47:55 | stephenfin | mriedem: Agreed | |
| 13:48:20 | mriedem | i wonder how valid it is, or time would be saved, by starting up devstack but not enabling nova, so that you can do that manually after keystone/glance/cinder/neutron are already setup | |
| 13:48:58 | mriedem | i think the only major thing in the install guide in rocky was the placement db | |
| 13:49:29 | sean-k-mooney | mriedem: i think you will hit depency issues | |
| 13:50:00 | mriedem | on other openstack services? | |
| 13:50:06 | mriedem | or things like setting up libvirt? | |
| 13:50:34 | sean-k-mooney | well neutron would expect to be able to talks to placement for things like routed networks | |
| 13:50:48 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Filter out instances without a host when populating AZ https://review.openstack.org/594178 | |
| 13:50:50 | mriedem | routed networks are optional and devstack doesn't set those up anyway | |
| 13:51:01 | mriedem | we definitely *should* have a ci job that uses routed networks | |
| 13:51:13 | mriedem | across a 2-node deploy where each host is in a separate aggregate | |
| 13:51:39 | mriedem | but that would require time and people that care to make sure it continues to work | |
| 13:52:12 | sean-k-mooney | in theroy devstack should be able to help i guess | |
| 13:52:45 | sean-k-mooney | mriedem: is placement installation considered part of the nova install guide | |
| 13:55:40 | dansmith | tssurya: looks like the down-cell stack needs rebasing again | |
| 13:56:00 | dansmith | presumably its review-able regardless? | |
| 13:56:01 | efried | mriedem: https://review.openstack.org/594179 <== alternative uuidsentinel impl | |
| 13:56:20 | tssurya | dansmith: yea, its ready for a first time review | |
| 13:56:26 | tssurya | still working on filtering part | |
| 13:56:32 | dansmith | okay | |
| 13:56:36 | tssurya | but would be nice to get opinions | |
| 13:56:58 | tssurya | I have them as seperate patches for now, will squash them with the version BUMP | |
| 13:57:09 | tssurya | once we review the approach | |
| 13:57:59 | tssurya | and, mriedem: sorry about missing the instance.host None case earlier on and the backport headaches. | |
| 13:58:03 | openstackgerrit | Jiri Suchomel proposed openstack/nova stable/pike: Filter out instances without a host when populating AZ https://review.openstack.org/594184 | |
| 14:00:20 | openstackgerrit | Surya Seetharaman proposed openstack/nova stable/queens: Filter out instances without a host when populating AZ https://review.openstack.org/594185 | |
| 14:02:37 | stephenfin | mriedem: https://bugs.launchpad.net/oslo.policy/+bug/1788183 | |
| 14:02:37 | openstack | Launchpad bug 1788183 in oslo.policy "Rule description not rendered as rST" [Undecided,New] | |
| 14:04:14 | mriedem | tssurya: not your fault, we have reviewers for a reason | |
| 14:04:21 | mriedem | and i obviously missed it as well | |
| 14:04:33 | mriedem | sean-k-mooney: i think so yes | |
| 14:05:09 | mriedem | efried: why not in oslotest? because of the circular dep? | |
| 14:05:54 | efried | mriedem: And because it's... a UUID util. And because just because I can't think of a reason for it to be used outside of test, doesn't mean it can't be. See commit message. | |
| 14:06:33 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Filter out instances without a host when populating AZ https://review.openstack.org/594185 | |
| 14:07:03 | sean-k-mooney | mriedem: then in that case if you wanted to test the nova install guide, and use devstack to help, you would just have devstack install keystone,mysql, and rabbitmq. and perhapse memcached | |
| 14:07:39 | mriedem | but nova also needs neutron | |
| 14:07:46 | mriedem | and i don't want to go through the neutron install guide to test nova's install | |
| 14:07:49 | mriedem | same for glance | |
| 14:08:15 | mriedem | cue kevin fox to say it should all be one monolithic install | |
| 14:09:06 | sean-k-mooney | well i would not expect the install guide for nova to cover the glance or neutron parts | |
| 14:09:23 | sean-k-mooney | i also would not assmue you could boot a vm after finishing it | |
| 14:09:44 | sean-k-mooney | i would just assumne i had the nova compontes deployed and fuctioning | |
| 14:10:06 | sean-k-mooney | e.g. nova hypervior list should show all the resouces but openstack server create would fail | |
| 14:10:28 | mriedem | well, if i'm installing nova, i would like to be able to create a vm by the end of it | |
| 14:10:35 | mriedem | otherwise i don't know if i f'ed up the install somewhere | |
| 14:11:08 | sean-k-mooney | in that case it does have to be a multi service install guide | |
| 14:11:12 | mriedem | also, https://docs.openstack.org/nova/latest/install/controller-install-ubuntu.html#install-and-configure-components refers to the neutron install guide | |
| 14:13:19 | sean-k-mooney | i guess refering to the other guide also works. that said untill nova networks if fully dead neutron is technical not a nova depency | |
| 14:13:45 | sean-k-mooney | but i could see adding neutron to the devstack install. | |
| 14:14:24 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Filter out instances without a host when populating AZ https://review.openstack.org/594184 | |
| 14:14:46 | sean-k-mooney | glace i guess would also be required because there is no way to boot a vm otherwise. unless you used the fake drivers | |
| 14:27:42 | mriedem | jroll: pretty sure this has always been true yeah? https://bugs.launchpad.net/nova/+bug/1787509 | |
| 14:27:42 | openstack | Launchpad bug 1787509 in OpenStack Compute (nova) "Baremetal filters and default filters cannot be used simultaneously in the same nova" [Undecided,New] | |