| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-02 | |||
| 11:06:05 | bauzas | so either people aren't yet there and just running old versions | |
| 11:06:13 | bauzas | or we have less people testing those | |
| 11:06:36 | sdague | yeh, it does change at different parts of the cycle, once we get RCs I expect more honestly | |
| 11:06:53 | sdague | though, we also have less randomly misconfigured cloud Invalid bugs | |
| 11:06:56 | sdague | so that's nice | |
| 11:08:16 | bauzas | yup | |
| 11:13:32 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test resize to same host with placement api https://review.openstack.org/489973 | |
| 11:30:53 | mamandle | bauzas: alex_xu: mriedem: can you please take a look at https://review.openstack.org/#/c/489848/, its cherrypick of https://review.openstack.org/#/c/483911/ to Ocata. Thanks. | |
| 11:37:45 | openstackgerrit | Chris Dent proposed openstack/nova master: Always use application/json accept header in report client https://review.openstack.org/489772 | |
| 11:46:28 | tonyb | bauzas: Any chance you can look at: https://review.openstack.org/489078 It's need to get the current glanceclient/glance_store into pike | |
| 11:47:26 | tonyb | bauzas: I'll nag sdague and mriedem when I catch them tomorrow | |
| 12:00:11 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Improve stable-api doc with current API state https://review.openstack.org/489926 | |
| 12:02:06 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/477091 | |
| 12:22:42 | openstackgerrit | John Garbutt proposed openstack/nova-specs master: WIP: Backlog spec on preemptible servers https://review.openstack.org/438640 | |
| 12:38:59 | gibi | whoa gerrit now has the new OpenStack logo | |
| 12:40:35 | openstackgerrit | Matthew Booth proposed openstack/nova master: Split Compute.errors_out_migration into a separate contextmanager https://review.openstack.org/485734 | |
| 12:40:35 | openstackgerrit | Matthew Booth proposed openstack/nova master: Fix scope of errors_out_migration in resize_instance https://review.openstack.org/487495 | |
| 12:40:36 | openstackgerrit | Matthew Booth proposed openstack/nova master: Functional test for auto revert resize https://review.openstack.org/489654 | |
| 12:40:36 | openstackgerrit | Matthew Booth proposed openstack/nova master: Automatically revert resize which fails on destination https://review.openstack.org/462521 | |
| 12:40:37 | openstackgerrit | Matthew Booth proposed openstack/nova master: Fix scope of errors_out_migration in finish_resize https://review.openstack.org/487515 | |
| 12:40:37 | openstackgerrit | Matthew Booth proposed openstack/nova master: fake_notifier: Refactor wait_for_versioned_notification https://review.openstack.org/489637 | |
| 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 | openstack | Launchpad bug 1708171 in devstack "Nova Affinity filters no longer work" [Undecided,New] | |
| 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:56:29 | sdague_ | efried: hmmm... how was glance working previously without those creds? | |
| 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: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 | efried | sdague_ And the Adapter business is how we tell it what combo of service type, interface, etc. we're looking for. | |
| 14:01:23 | sdague_ | ok, but that all happens in a user context right? | |
| 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 | openstack | bug 1708171 in devstack "Nova Affinity filters no longer work" [Undecided,New] https://launchpad.net/bugs/1708171 | |
| 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: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 | dansmith | smatzek: https://review.openstack.org/#/c/487478/ | |
| 14:04:20 | sdague_ | mordred: I'm fine with that | |
| 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... | |