| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-02 | |||
| 12:40:38 | openstackgerrit | Matthew Booth proposed openstack/nova master: Ensure errors_out_migration errors out migration https://review.openstack.org/479802 | |
| 12:41:43 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test resize to same host with placement api https://review.openstack.org/489973 | |
| 12:41:52 | mdbooth | gibi: Reigned in my instincts with https://review.openstack.org/#/c/489637/ :) | |
| 12:42:54 | gibi | mdbooth: looking.... | |
| 12:44:21 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Improve stable-api doc with current API state https://review.openstack.org/489926 | |
| 12:49:15 | alex_xu | nova api meeting is in 10 mins at #openstack-meeeting-4 | |
| 13:00:46 | alex_xu | nova api meeting is running | |
| 13:02:43 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.rebuild_scheduled notification https://review.openstack.org/473929 | |
| 13:22:55 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Avoid error log on 405 response https://review.openstack.org/490021 | |
| 13:25:22 | openstackgerrit | Matthew Booth proposed openstack/nova master: Functional test for auto revert resize https://review.openstack.org/489654 | |
| 13:25:23 | openstackgerrit | Matthew Booth proposed openstack/nova master: fake_notifier: Refactor wait_for_versioned_notification https://review.openstack.org/489637 | |
| 13:26:56 | edleafe | Man, that new OpenStack logo in Gerrit looks huge | |
| 13:27:31 | gibi | edleafe: first I didn't know why my review tabs looks soo strange | |
| 13:27:59 | mdbooth | edleafe: And compression artifact-y | |
| 13:28:15 | mdbooth | Is it scaled up, or something? | |
| 13:28:46 | mdbooth | Interestingly, it's actually a gorgeous SVG | |
| 13:28:55 | gibi | it is not artifacty for me | |
| 13:28:56 | mdbooth | So it just that Chrome sucks at displaying it to me | |
| 13:29:00 | gibi | can be a browser issue | |
| 13:37:07 | cdent | edleafe: I responded to your comments on the aggregate map update thing. I feel like I’m not understanding you and you’re not understanding me. | |
| 13:41:07 | edleafe | cdent: sorry, still caffeinating | |
| 13:42:13 | cdent | i know how that can be | |
| 13:42:15 | edleafe | cdent: I was trying to say that another way of reducing the need to update the aggregate map would be to make those calls through the report client - at least the ones from Nova. This way the client "knows" when things change | |
| 13:42:28 | cdent | “those calls”? | |
| 13:42:53 | edleafe | cdent: when we set an aggregate in placement that nova cares about | |
| 13:42:58 | cdent | oh | |
| 13:43:20 | cdent | yeah, no, how would we know | |
| 13:44:03 | sdague_ | efried: you have a working devstack run on your change? | |
| 13:44:35 | efried | sdague_ I put four links in the change set. | |
| 13:44:57 | sdague_ | ah, there it is | |
| 13:46:09 | efried | sdague_ One pair alters devstack to make glance use the new code. The other pair builds on that and proves ironic. | |
| 13:46:32 | efried | sdague_ Each pair has one half using ksa creds to do real discovery; the other half using endpoint_override explicitly. | |
| 13:47:12 | edleafe | cdent: just throwing out ideas to reduce the number of calls. We call so often precisely because we don't have a way of knowing otherwise | |
| 13:47:34 | sdague_ | hmmm... ok, I guess I didn't see the change I was expecting, which was just pulling out the api_servers set | |
| 13:50:28 | cdent | edleafe: i like thrown ideas | |
| 13:51:34 | sdague_ | efried: https://review.openstack.org/490031 is what I was thinking | |
| 13:51:38 | sdague_ | which I actually want to merge | |
| 13:54:24 | efried | sdague_ That's this one: https://review.openstack.org/#/c/488444/ | |
| 13:54:43 | efried | sdague_ I don't think yours will work as-is, because you're not providing auth/session/adapter creds. | |
| 13:55:06 | smatzek | mriedem, Recent changes to devstack setup related to cells have broke the affinity filters. This is breaking Trove's gate. My knowledge of cells, affinity filters, and the change are insufficient and could use pointers on how to get the filters working again. https://bugs.launchpad.net/devstack/+bug/1708171 | |
| 13:55:06 | openstack | Launchpad bug 1708171 in devstack "Nova Affinity filters no longer work" [Undecided,New] | |
| 13:56:29 | efried | sdague_ Note that https://review.openstack.org/#/c/488444/3/lib/nova@542 is where I get rid of api_servers by renaming it - I wanted to be able to see what the value would have been, which is why I didn't just remove the line. | |
| 13:56:29 | sdague_ | efried: hmmm... how was glance working previously without those creds? | |
| 13:56:48 | efried | sdague_ Because api_servers was setting explicit endpoint, so no discovery was done. | |
| 13:57:09 | sdague_ | ok, this seems weird | |
| 13:57:29 | sdague_ | we need another set of creds to do discovery? | |
| 13:57:39 | sdague_ | that just feels wrong | |
| 13:57:50 | sdague_ | I must be missing something | |
| 13:59:33 | efried | sdague_ The service catalog ultimately lives in a ksa auth. | |
| 14:00:50 | efried | sdague_ And the discovery process uses a Session to talk to the discovery endpoint to grab the version document. | |
| 14:01:23 | sdague_ | ok, but that all happens in a user context right? | |
| 14:01:23 | efried | sdague_ And the Adapter business is how we tell it what combo of service type, interface, etc. we're looking for. | |
| 14:01:32 | sdague_ | can't we first order piggy back on that? | |
| 14:01:36 | efried | mordred may be able to explain it better. | |
| 14:01:38 | smatzek | dansmith, can you take a look at bug 1708171 and give me some pointers as you authored the transport_url change in devstack ^ ? | |
| 14:01:38 | openstack | bug 1708171 in devstack "Nova Affinity filters no longer work" [Undecided,New] https://launchpad.net/bugs/1708171 | |
| 14:02:10 | mordred | efried, sdague_: reading scrollback | |
| 14:03:19 | efried | sdague_ The user context may (or may not) have an appropriate auth in it. The util is set up to take that if it's present. | |
| 14:03:22 | sdague_ | I'm all fine with a glance service user to get us past snapshot timeouts, but it seems weird to have the net change be here. "Hey, great feature, remove this one config line and add these 10, including passwords, to every system" feels less compelling | |
| 14:03:37 | dansmith | smatzek: yep, known issue, you'll need to disable the multi-cell layout in your job if you need that | |
| 14:03:38 | dansmith | mriedem: ^ | |
| 14:03:39 | sdague_ | efried: doesn't have appropriate auth to find glance? | |
| 14:03:45 | mordred | sdague_, efried: so - I think we can totally use the existing auth | |
| 14:04:05 | efried | sdague_ I don't disagree the switchover feels burdensome for the glance case. | |
| 14:04:09 | mordred | the thing we might need that's new is adapter params so an admin can *override* defaults | |
| 14:04:20 | sdague_ | mordred: I'm fine with that | |
| 14:04:20 | dansmith | smatzek: https://review.openstack.org/#/c/487478/ | |
| 14:04:21 | mordred | (which I left some notes on as comments in the efried change) | |
| 14:04:30 | sdague_ | I just want to be able to run without that | |
| 14:05:01 | sdague_ | because, to the best of my knowledge every glance action happens within a user context in the current nova flows | |
| 14:05:26 | mordred | yes. we should make sure that a) the admin can still provide api_servers and all works as before b) the admin can provide nothing and a proper setup will work c) the admin can provide override values for the adapter parameters and those will affect what discovery finds correctly | |
| 14:05:39 | sdague_ | especially because screwing up 10 service auth config variables is actually *very* common | |
| 14:05:52 | mordred | basically everyone gets them all wrong :) | |
| 14:05:58 | sdague_ | we get a regular trickle of bugs in which are people having done that in neutron all the time | |
| 14:06:08 | sdague_ | i.e. the neutron part of the nova config | |
| 14:06:13 | sdague_ | then 500s all over the place | |
| 14:06:19 | mordred | efried: I'll take another pass through your patch with the above in mind - I may not have tracked the auth flow completely the last time | |
| 14:06:49 | efried | It's worth pointing out that one of my motivations in this change set was to be as non-intrusive as possible to the code. E.g. in glance, isolate the change to where it finds the service URL, as opposed to doing a far-reaching rework to find a good auth to use, etc. | |
| 14:06:50 | mordred | sdague_: incidentally - morgan had a question the other day which I don't think is terribly tractable at the moment... | |
| 14:06:51 | larivee | /join #openstack-i18n | |
| 14:07:00 | sdague_ | the reality is the only service I think Nova talks to outside of a user context is neutron | |
| 14:07:20 | sdague_ | because it does background processing in a periodic to catch certain changes | |
| 14:07:28 | mordred | sdague_: but currently the config sections are glance/cinder/ironic - but we've got this whole "use service-types" thing going on | |
| 14:07:37 | sdague_ | mordred: sure | |
| 14:07:52 | mordred | sdague_: maybe for the S cycle we should do a transition from glance to image - but certainly not for right now | |
| 14:07:54 | sdague_ | I was just thrown with the idea that 10 lines of config needed to be added here | |
| 14:08:00 | mordred | yah | |
| 14:08:06 | sdague_ | mordred: yeh, that's low priority on the naming | |
| 14:08:10 | sdague_ | just nice to have | |
| 14:08:11 | mordred | I mean - there are a bunch of lines of potential config that need to be possible to set | |
| 14:08:27 | mordred | but I agree, we need to make sure things work without them being set too | |
| 14:08:44 | sdague_ | I just want to get us down to super minimal configs, because every line of config we can pull out of manditory setup docs makes it easier to get right | |
| 14:08:50 | mordred | yup | |
| 14:08:54 | smatzek | dansmith, thanks, so I can set singleconductor and it should work. The comment block says this option will be removed in the future. I assume the affinity filters will be fixed to not need it before that's removed? | |
| 14:08:55 | mordred | 100% agree | |
| 14:09:20 | mordred | and that's actually why we made some of the ksa changes - the ability to specify version ranges and lists of intefaces, for instance ... | |
| 14:09:28 | dansmith | smatzek: we have several things that we have to fix before we can remove that thing, yeah | |
| 14:09:35 | sdague_ | mordred: yep | |
| 14:09:43 | efried | sdague_ So in this impl, a) api_servers is still supported, and takes precedence; b) if you set endpoint_override, you *shouldn't* need the other fields (I should edit https://review.openstack.org/#/c/489671/2/lib/nova to verify that) - that should be a straight swap. | |
| 14:09:48 | mordred | is so that nova can say "I can handle v1-v2 of glance and prefer internal interface then public interface" | |