| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-24 | |||
| 22:55:45 | mriedem | only thing on ^ is we'd have to cleanup the commit message | |
| 22:56:23 | stevemar | mriedem: fi-lgtm | |
| 22:57:02 | stevemar | andreykurilin: i rebased it, should be bumped out of the gate (don't think it even got that far) | |
| 22:57:26 | dansmith | bauzas: it's not starting the placement api, looks like | |
| 22:57:35 | dansmith | bauzas: I see no placement-api log | |
| 22:57:38 | stevemar | andreykurilin: how would bumping OSC min solve the issue? | |
| 22:58:00 | andreykurilin | mriedem: but why we have multiple networks with latest openstacksdk and do not have them with old version? | |
| 22:59:38 | mriedem | andreykurilin: because the sdk broke the filtering on available networks :) | |
| 22:59:47 | andreykurilin | mriedem: oh | |
| 23:00:00 | mriedem | at least i think that's what kevinbenton found when he looked at that issue before | |
| 23:00:18 | mriedem | his notes are in that patch | |
| 23:00:20 | stevemar | andreykurilin: mriedem can we just get the nova functional tests working with the newer version? | |
| 23:00:31 | andreykurilin | mriedem: can you update commit message? | |
| 23:00:40 | stevemar | i guess that's what https://review.openstack.org/#/c/418190/ does | |
| 23:00:45 | mriedem | stevemar: well shouldn't that be a major version bump on the sdk? | |
| 23:01:06 | stevemar | mriedem: they have a stance that since they are pre 1.0 they are allowed to break compatibility | |
| 23:02:09 | stevemar | mriedem: like how gmail was in beta for a decade | |
| 23:02:32 | mriedem | stevemar: jesus h c | |
| 23:02:38 | stevemar | osc and senlinclient *need* the latest version | |
| 23:02:39 | bauzas | dansmith: roger, I'll try to investigate tomorrow morning but I'm just exhausted | |
| 23:02:40 | mriedem | well they should shit or get off that pot | |
| 23:02:40 | andreykurilin | stevemar: I'm ok to merge mriedem fix for our gates and then look at the process of merging g-r update. I do not want to see g-r update at first, since we need to merge one patch before cutting new release | |
| 23:02:50 | andreykurilin | heh | |
| 23:02:57 | stevemar | mriedem: soon (tm) | |
| 23:03:04 | stevemar | andreykurilin: i would looove that | |
| 23:03:45 | stevemar | andreykurilin: rally is good right? | |
| 23:04:06 | andreykurilin | stevemar: yes. setting quotas works. thanks | |
| 23:04:18 | openstack | Launchpad bug 1658571 in OpenStack Compute (nova) "Microversion 2.37 break 2.32 usage" [High,In progress] - Assigned to Artom Lifshitz (notartom) | |
| 23:04:18 | Kevin_Zheng | mriedem: https://bugs.launchpad.net/nova/+bug/1658571 | |
| 23:05:25 | andreykurilin | stevemar: PS: I wonder why are so dependent on pre 1.0 library which can do whatever they want(as you said) .. | |
| 23:05:36 | andreykurilin | *why we are | |
| 23:05:46 | mriedem | andreykurilin: yeah good question | |
| 23:06:09 | mriedem | stevemar: so is OSC going to have a thing in it's release notes saying, oh btw, when you create private tenant networks in this release things are doing to be different? | |
| 23:06:10 | stevemar | andreykurilin: theres some history there | |
| 23:06:25 | mriedem | Kevin_Zheng: hi | |
| 23:06:34 | mriedem | Kevin_Zheng: did you see the ML thread? | |
| 23:07:22 | andreykurilin | stevemar: if they want to play with us in the same sandbox, they should follow the same rules | |
| 23:07:50 | Kevin_Zheng | mriedem: not yet, left early yesterday | |
| 23:08:01 | stevemar | mriedem: we can add that to the release notes, but i think we rename the column/info properly | |
| 23:09:12 | bauzas | mriedem: what kind of important note should we write for telling to ops "folks, you need to change your cloud before upgrading to Ocata"? | |
| 23:09:24 | bauzas | mriedem: given the upgrade section is already huuuuuge | |
| 23:09:49 | bauzas | possibly in the prelude I guess | |
| 23:10:00 | dansmith | bauzas: made changes.. start_placement() isn't getting called which means no a2ensite, restart, etc | |
| 23:10:08 | mriedem | bauzas: yeah and probably in the placement devref | |
| 23:10:26 | bauzas | dansmith: kewl thanks ! | |
| 23:10:34 | mriedem | bauzas: we'll have something in the prelude section for placement and cells v2 for sure | |
| 23:10:47 | mriedem | i'd also like docs reflected in the devref which we have for placement and cells v2 | |
| 23:10:49 | mriedem | for stuff like this | |
| 23:10:54 | bauzas | dansmith:jam+1 | |
| 23:11:05 | dansmith | heh | |
| 23:11:13 | stevemar | mriedem: plus, everyone uses neutron anyway :P | |
| 23:13:15 | Kevin_Zheng | mriedem: just read, so a new microversion? | |
| 23:13:26 | mriedem | Kevin_Zheng: yes | |
| 23:13:35 | mriedem | Kevin_Zheng: but i think we can handle that as a bug fix after the feature freeze | |
| 23:13:41 | mriedem | so we're not rushing toward thursday | |
| 23:13:55 | Kevin_Zheng | Cool | |
| 23:14:11 | bauzas | dansmith: wrt grenade, how can we make sure that we don't run a placement API in the compute-only for the multinode job ? | |
| 23:14:14 | mriedem | stevemar: this is like earth-shattering issue #4 today so i've lost my sense of humor about it :) | |
| 23:14:21 | Kevin_Zheng | But Chinese New Year comes this week | |
| 23:14:30 | stevemar | mriedem: samesies | |
| 23:14:32 | dansmith | bauzas: by not starting one in the subnode? | |
| 23:14:38 | mriedem | Kevin_Zheng: i think artom is planning to work on it | |
| 23:14:44 | Kevin_Zheng | Ok | |
| 23:14:57 | dansmith | bauzas: the subnode only has ENABLED_SERVICES='n-cpu' or something | |
| 23:15:08 | bauzas | dansmith: yeah, but since I'm updating from-newton, am I sure it's not for the subnode? | |
| 23:15:18 | dansmith | yes | |
| 23:15:41 | bauzas | dansmith: okay, but I'd still want to have the subnode running placement_compute_only | |
| 23:15:49 | dansmith | bauzas: if that was a problem, all the db creation stuff above your code would run on the subnode too | |
| 23:16:51 | andreykurilin | bauzas mriedem: can you look at https://review.openstack.org/#/c/424256/ ? neutron job stuck, but all others passed. | |
| 23:18:46 | mriedem | andreykurilin: looks like it's running | |
| 23:19:07 | andreykurilin | mriedem: ~2,5 hours... | |
| 23:19:17 | andreykurilin | usually it takes less than an hour | |
| 23:19:21 | mriedem | oh hrm | |
| 23:19:24 | mriedem | you might want to ask in -infra | |
| 23:20:05 | mriedem | stevemar: so on https://review.openstack.org/#/c/418190/ you want me to basically just reword the commit to say, the sdk has changed things in a backward incompatible way which breaks osc used in devstack runs, so we have to workaround that now? | |
| 23:21:02 | mriedem | i need to get cleaned up before dinner, full of salt water | |
| 23:29:03 | andreykurilin | mriedem: quick question: does version negotiation require to be authenticated ? | |
| 23:29:42 | stevemar | mriedem: let dtroyer take a looksy, i told him about it in -sdks | |
| 23:29:49 | stevemar | mriedem: i'm damn exhausted | |
| 23:32:33 | andreykurilin | mriedem: and please, check https://review.openstack.org/#/c/424745/ . Not sure is that what you want or not. | |
| 23:34:04 | mriedem | andreykurilin: that varies by service, would need to test it out | |
| 23:36:47 | mriedem | why do we need to change grenade for placement? i thought from-newton was already doing something there? | |
| 23:37:53 | mriedem | nvm i guess we didn't, | |
| 23:38:00 | mriedem | we made placement enabled by default in d-g on master | |
| 23:38:04 | mriedem | but grenade would start on stable/newton | |
| 23:38:50 | mriedem | andreykurilin: on https://review.openstack.org/#/c/424745/ i think there are also some things in the python API code that i need to look at, | |
| 23:38:53 | mriedem | but probably won't have time right now | |
| 23:39:36 | andreykurilin | np | |
| 23:39:51 | bauzas | mriedem: because placement is not defaulted by devstack in newton | |
| 23:40:14 | bauzas | mriedem: that's why I started to write a script for from-newton | |
| 23:40:19 | dtroyer | andreykurilin: your last comment on https://review.openstack.org/#/c/424847/, you are about to release another novaclient? | |
| 23:40:19 | mriedem | yup | |
| 23:40:39 | andreykurilin | dtroyer: yes | |
| 23:41:35 | andreykurilin | dtroyer: we need just to merge https://review.openstack.org/#/c/424256/ . It is last known(for me) blocker | |
| 23:41:38 | dtroyer | any chance https://review.openstack.org/#/c/418190/ can go in too? AIUI that'ss unblock the SDK stuff | |
| 23:41:42 | mriedem | bauzas: was thinking about this https://github.com/openstack-infra/devstack-gate/commit/c1b64743b8404dad93d4c06bcb26cf87cbc5240c | |
| 23:42:20 | mriedem | andreykurilin: so i'll update the commit message in https://review.openstack.org/#/c/418190/ quick | |
| 23:42:26 | mriedem | since it sounds like we need it regardless | |
| 23:42:30 | andreykurilin | dtroyer: ^ | |
| 23:42:48 | bauzas | mriedem: yup, me too | |