| *** mriedem1 has joined #openstack-relmgr-office | 00:18 | |
| *** mriedem has quit IRC | 00:21 | |
| *** bswartz has joined #openstack-relmgr-office | 00:22 | |
| *** mriedem has joined #openstack-relmgr-office | 00:51 | |
| *** mriedem1 has quit IRC | 00:53 | |
| openstackgerrit | Dean Troyer proposed openstack/releases: OpenStackClient 1.7.1 https://review.openstack.org/229694 | 00:57 |
|---|---|---|
| *** armax has quit IRC | 01:29 | |
| *** dims__ has quit IRC | 02:29 | |
| *** dims_ has joined #openstack-relmgr-office | 02:29 | |
| *** mriedem has quit IRC | 02:51 | |
| *** dims__ has joined #openstack-relmgr-office | 03:15 | |
| *** dims___ has joined #openstack-relmgr-office | 03:16 | |
| *** dims____ has joined #openstack-relmgr-office | 03:18 | |
| *** dims_ has quit IRC | 03:18 | |
| *** dims__ has quit IRC | 03:20 | |
| *** dims___ has quit IRC | 03:21 | |
| *** spzala has quit IRC | 03:34 | |
| *** armax has joined #openstack-relmgr-office | 03:36 | |
| *** dims____ has quit IRC | 03:37 | |
| *** aukhan has joined #openstack-relmgr-office | 04:25 | |
| *** stevemar has joined #openstack-relmgr-office | 04:30 | |
| ttx | bswartz: approving stuff now | 06:33 |
| *** armax has quit IRC | 06:39 | |
| ttx | stevemar: your approval of https://review.openstack.org/#/c/229647/ means we need to open a RC2 window for keystone to account for it. | 07:30 |
| ttx | stevemar: so ping me when around to discuss what else to approve there | 07:32 |
| *** openstackgerrit has quit IRC | 07:46 | |
| *** openstackgerrit has joined #openstack-relmgr-office | 07:47 | |
| *** thingee has joined #openstack-relmgr-office | 07:57 | |
| *** thingee has quit IRC | 08:31 | |
| *** dims has joined #openstack-relmgr-office | 09:55 | |
| *** Kiall has quit IRC | 09:55 | |
| *** Kiall has joined #openstack-relmgr-office | 09:56 | |
| *** stevemar has quit IRC | 10:08 | |
| *** stevemar has joined #openstack-relmgr-office | 10:09 | |
| *** stevemar has quit IRC | 10:13 | |
| sdague | ttx: morning | 10:30 |
| ttx | o/ | 10:32 |
| sdague | so... I think we have a path forward which is going to require a new novaclient release | 10:34 |
| sdague | https://review.openstack.org/#/c/229669 will have to land first | 10:34 |
| sdague | is there anything else we should know / do given where we are in the release cycle? | 10:34 |
| sdague | to build context here: we can't switch all the upgrade infrastructure from liberty -> master because osc/novaclient boot from volume with a preprepared volume does not currently work | 10:38 |
| sdague | and that's used in our cinder upgrade verification | 10:38 |
| ttx | sdague: is it going to require to change requirements ? | 10:38 |
| ttx | like bump the minimal version ? | 10:39 |
| sdague | ideally, probably yes. Though it is only this one operation that is broken (as far as we can tell), so we could probably live without it | 10:39 |
| ttx | I'd rather do that ASAP so that other RC2s can include the bump then | 10:41 |
| ttx | let's wait for dhellmann and just do that | 10:41 |
| sdague | ok, we still need to actually land the patch, melanie did a WIP on it | 10:43 |
| sdague | unless johnthetubaguy wants to just update the commit message to make it landable | 10:44 |
| sdague | I'll let that be his call as PTL | 10:44 |
| ttx | ack | 10:44 |
| * ttx lunches | 10:44 | |
| *** aukhan has quit IRC | 11:07 | |
| flaper87 | ttx: https://review.openstack.org/#/c/228227/ | 11:28 |
| flaper87 | ttx: and https://review.openstack.org/#/c/229672/ | 11:28 |
| flaper87 | ttx: unless I'm missing something, it should be ok to unblock these patches now | 11:29 |
| *** dtantsur|afk is now known as dtantsur | 11:47 | |
| ttx | flaper87: looking | 11:50 |
| ttx | flaper87, nikhil: Yeah, I should open a RC2 milestone there today. Anything else you want to land there ? | 11:52 |
| ttx | oh there is a CVE coming up, we should include that as well | 11:52 |
| * ttx opens milestone | 11:52 | |
| *** gordc has joined #openstack-relmgr-office | 12:01 | |
| flaper87 | ttx: yeah, originally the CVE was the main reason for having an RC2 | 12:07 |
| bswartz | ttx: ping | 12:17 |
| bswartz | ttx: I saw you marked a bug as "fix released" even though RC isn't released yet | 12:18 |
| bswartz | should I be marking each bug released as the fix merges into stable/liberty? | 12:18 |
| ttx | bswartz: o/ | 12:25 |
| ttx | bswartz: I can do that. But yes, we track if a bug makes it to the release branch by moving it from FixCommitted to fixReleased | 12:25 |
| ttx | since Launchpad limits who can create series tasks, it simplifies tracking | 12:25 |
| ttx | FixCommitted = in master branch | 12:26 |
| ttx | FixReleased = in release branch (stable/liberty) | 12:26 |
| ttx | bswartz: I think we are missing a few backports. I stand by to approve them when submitted | 12:26 |
| *** spzala has joined #openstack-relmgr-office | 12:31 | |
| bswartz | ttx: okay it's confusing because there's no obvious way to check what was backported and what still needed it | 12:32 |
| bswartz | ttx: I found the ones that were missing and cherry picked them | 12:32 |
| bswartz | I'll start marking the bugs released as they merge | 12:33 |
| bswartz | oh you already did it | 12:33 |
| ttx | yep. | 12:34 |
| sdague | ttx: so, as the clients actually branched, I'm assuming the stable/liberty backport was right here - https://review.openstack.org/#/q/Ie8c56c28492793990ef7ed6dc54768cef9e28a98,n,z | 12:40 |
| sdague | that will need a stable maint +A on it | 12:41 |
| ttx | I'm on it | 12:41 |
| *** stevemar has joined #openstack-relmgr-office | 12:41 | |
| ttx | sdague: oh, waiting for the master landing first | 12:41 |
| sdague | sure, master is in the gate now | 12:42 |
| sdague | is there a reason not to have them run in parallel? | 12:42 |
| *** stevemar has quit IRC | 12:46 | |
| ttx | sdague: what do we do if master fails ? | 12:46 |
| ttx | Waiting is about the only way you make sure you never introduce a regression in master (i.e. fix something in stable/liberty but not in master) | 12:47 |
| sdague | recheck, it's passed tests a few times | 12:47 |
| *** stevemar has joined #openstack-relmgr-office | 12:47 | |
| ttx | doesn't mean something won't happen that will prevent the merging of the patch as it stands. | 12:47 |
| sdague | that risk seems extremely small | 12:48 |
| ttx | We can parallelize it but then we need to check back that the master patch actually landed | 12:48 |
| ttx | so it becomes a two-step process | 12:48 |
| ttx | it's fine when you only watch one | 12:48 |
| sdague | yep, which we will, as we need new versions of novaclient on both branches | 12:48 |
| ttx | when you watch two dozens, like I'm right now, not so fun | 12:48 |
| sdague | I'll keep an eye on these | 12:48 |
| ttx | sdague: ok then | 12:49 |
| sdague | because novaclient doesn't get picked up by us, or users, except at releases | 12:49 |
| sdague | if it was the server, I would be more cautious, but as a library with a low patch volume, I think we're fine | 12:49 |
| ttx | sdague: approved. I put your name on the blame line | 12:50 |
| sdague | :) | 12:50 |
| sdague | that's fine | 12:51 |
| sdague | this is basically my high priority interupt today until it's all sorted, as all the grenade branch switch work is blocked on it | 12:51 |
| ttx | (we may ned that extra hour this will likely buy us) | 12:51 |
| ttx | nee* | 12:51 |
| *** stevemar has quit IRC | 12:52 | |
| *** mriedem has joined #openstack-relmgr-office | 13:23 | |
| dhellmann | ttx, sdague : good morning, I'm catching up on scrollback | 13:34 |
| ttx | dhellmann: the priority is to get the requirements changes in, since we can't really close the currently-open RC2s without importing the requirements changes | 13:35 |
| ttx | dhellmann: I think that means the pmython-novaclient that sdague is working on... and the reno import | 13:35 |
| dhellmann | ttx: ack | 13:35 |
| ttx | the latter might need a kick, was blocked by a dims question | 13:36 |
| ttx | Also might be worth reviewing if anything else is worth a requirements bump | 13:36 |
| ttx | dhellmann: if you confirm that https://review.openstack.org/#/c/229059/ is good to go, I can approve it. It's in master anyway so if a mistake was made we need to fix it there too | 13:36 |
| sdague | just to be clear, this is currently where we blow up in doing the branch shift, but we don't know what will fail behind it once this is changed | 13:37 |
| dhellmann | ttx, dims: I believe the blacklist only applies to constraints, not to the requirements specification | 13:37 |
| dhellmann | those are unconstrained libs, iiuc | 13:37 |
| dhellmann | sdague: when this is sorted out maybe we can spend a little time reasoning about where these branches should be created in the process (last seems like too late?) | 13:38 |
| sdague | well, it used to be done post release | 13:38 |
| sdague | it can't be done until the branches exist on the projects | 13:38 |
| sdague | but, I agree, figuring out ways to get ahead of this would be good | 13:39 |
| sdague | master -> master upgrades would actually have caught this issue | 13:39 |
| dhellmann | master -> master? like "install everything except the patch under test then upgrade to the patch under test"? | 13:40 |
| sdague | yeh, or even just install master and try to run the upgrade process back to master | 13:40 |
| ttx | dims: could you discuss your -1 on https://review.openstack.org/#/c/229059/ here ? We'd like that fix is ASAP | 13:41 |
| sdague | the reality is, this mostly is about the fact that osc (and the clients in general) have really limitted testing | 13:41 |
| sdague | we do more osc ops during grenade than most places, as it was trying to demonstrate best practices for interacting with openstack | 13:41 |
| sdague | but, I do agree we should probably do a post mortem here and figure out the best way to make this smoother | 13:44 |
| dhellmann | sdague: maybe rather than new features, we should prioritize more tests for osc this cycle | 13:44 |
| dhellmann | yeah, when it's all working | 13:44 |
| bswartz | +1 for test coverage | 13:44 |
| sdague | the problem is it's easy to say +1 for test coverage, but very few people show up to do that :) | 13:45 |
| bswartz | many projects have no functional test coverage in their clients | 13:45 |
| bswartz | just unit tests and functional test coverage of the server | 13:45 |
| bswartz | In manila we've spend a significant amount of time on functional test coverage in both the client and the server, and something like 75% of our test cases are negative test cases | 13:46 |
| bswartz | all it takes is a core team that stands up and refuses to merge new features until the test coverage gets better | 13:49 |
| ttx | mestery: please have a look at https://bugs.launchpad.net/neutron/+bug/1501772 | 13:49 |
| openstack | Launchpad bug 1501772 in neutron (Ubuntu) "Metadata proxy process errors with binary user_data" [Undecided,New] | 13:49 |
| mestery | ttx: Looking ... | 13:50 |
| mestery | ttx: On it | 13:50 |
| *** armax has joined #openstack-relmgr-office | 13:52 | |
| *** armax has quit IRC | 13:52 | |
| ttx | redrobot: ping me when around for a RC2 opening | 13:52 |
| ttx | nikhil: around ? | 13:54 |
| *** stevemar has joined #openstack-relmgr-office | 13:54 | |
| *** armax has joined #openstack-relmgr-office | 13:55 | |
| *** stevemar has quit IRC | 13:59 | |
| redrobot | ttx o/ | 14:04 |
| ttx | redrobot: are the bugfixes you want to include merged in master already ? | 14:04 |
| redrobot | ttx only 1/2 so far. the other is in review. | 14:05 |
| ttx | redrobot: shall we wait a bit more then ? | 14:05 |
| redrobot | ttx yeah, it should be landing later today hopefully | 14:05 |
| ttx | ok, ping me when ready | 14:05 |
| * dims will be back in a bit. running an errand | 14:10 | |
| *** dims has quit IRC | 14:10 | |
| *** david-lyle has quit IRC | 14:11 | |
| *** stevemar has joined #openstack-relmgr-office | 14:11 | |
| *** sigmavirus24_awa is now known as sigmavirus24 | 14:12 | |
| stevemar | ttx: hey, yes, that was intentional, there are 2 more fixes to land back for rc2, one for translations and this one here: https://review.openstack.org/#/c/215870/ | 14:12 |
| *** david-lyle has joined #openstack-relmgr-office | 14:13 | |
| ttx | stevemar: ok, will target | 14:15 |
| ttx | stevemar: could you research and let me know which milestone to target those two blueprints to: | 14:15 |
| ttx | https://blueprints.launchpad.net/keystone/+spec/remove-oslo-incubator | 14:15 |
| ttx | https://blueprints.launchpad.net/keystone/+spec/remove-py26-hacks | 14:15 |
| ttx | stevemar: I pushed the translations one already | 14:16 |
| stevemar | ttx: thanks for the translation merge, qq about one of those blueprints | 14:18 |
| stevemar | ttx: looks like https://blueprints.launchpad.net/keystone/+spec/remove-oslo-incubator went into liberty-3 | 14:20 |
| stevemar | doesn't look like i can set the milestone to one in the past | 14:20 |
| ttx | yeah, you need to reactivate the milestone, I'll do it, it's a bit painful to do | 14:21 |
| ttx | just tell me which is what and I'l lfix it | 14:21 |
| stevemar | ttx: looks like that one was liberty-3. sorry about that. | 14:22 |
| stevemar | ttx: let me ask when the other one went in, might have been the same release | 14:23 |
| stevemar | err milestone | 14:23 |
| ttx | ok | 14:23 |
| ttx | keep me posted | 14:23 |
| *** stevemar has quit IRC | 14:41 | |
| *** stevemar has joined #openstack-relmgr-office | 14:42 | |
| *** stevemar has quit IRC | 14:46 | |
| *** stevemar has joined #openstack-relmgr-office | 14:46 | |
| *** stevemar_ has joined #openstack-relmgr-office | 14:49 | |
| nikhil | ttx: hi, was in a meeting. now here | 14:50 |
| *** stevemar has quit IRC | 14:51 | |
| *** stevemar has joined #openstack-relmgr-office | 14:55 | |
| *** stevemar_ has quit IRC | 14:58 | |
| *** TravT has quit IRC | 14:58 | |
| ttx | nikhil: o/ | 14:59 |
| ttx | nikhil: I opened the RC2 milestone | 14:59 |
| ttx | nikhil: https://launchpad.net/glance/+milestone/liberty-rc2 | 15:00 |
| ttx | nikhil: first one is CVE we expect for later today. Anything else you'd like to see in there before the respin ? | 15:00 |
| *** TravT has joined #openstack-relmgr-office | 15:01 | |
| nikhil | ttx: looking | 15:02 |
| mriedem | ttx: per bug 1501435 i'm going to be posting a novaclient 2.30.1 release request | 15:06 |
| openstack | bug 1501435 in python-novaclient "osc 1.7 no longer can boot a server from volume" [Critical,Fix committed] https://launchpad.net/bugs/1501435 - Assigned to John Garbutt (johngarbutt) | 15:06 |
| mriedem | just fyi | 15:06 |
| ttx | mriedem: sounds good. dhellmann ^ | 15:06 |
| mriedem | and then i'm going to be raising the upper-constraints cap in g-r on stable/liberty from 2.30.0 to 2.31.0 so we have room to fix 2.30.x | 15:06 |
| dhellmann | mriedem: the constraints cap isn't an upper bound, it's an exact value that is the package version we use in the tests | 15:07 |
| mriedem | dhellmann: so that will be automatically updated to 2.30.1? | 15:07 |
| dhellmann | mriedem: do you mean raising the cap in stable/liberty? we shouldn't have a cap any more | 15:07 |
| ttx | mriedem: you mean the raise the minimum, no ? | 15:07 |
| mriedem | i meant just upper-constraints | 15:07 |
| mriedem | sdague: do we need to raise minimum required novaclient for liberty to 2.30.1? | 15:08 |
| dhellmann | mriedem: not quickly, you want to submit that update, too | 15:08 |
| ttx | mriedem: ok | 15:08 |
| mriedem | dhellmann: yeah that was my plan | 15:08 |
| dhellmann | mriedem: if the only thing broken by the old version is osc, then you don't need to raise the minimum for a bug fix | 15:08 |
| dhellmann | mriedem: there are some instructions on submitting a requirements change along with release requests in http://docs.openstack.org/releases/instructions.html#requesting-a-release now | 15:08 |
| mriedem | yar | 15:09 |
| sdague | dhellmann: yeh, raised minimums are probably not needed as long as our jobs will get the new version with upper constraint going up | 15:09 |
| openstackgerrit | Matt Riedemann proposed openstack/releases: liberty: release 2.30.1 for python-novaclient https://review.openstack.org/229952 | 15:11 |
| sdague | mriedem: do you also want to do a master release now? | 15:12 |
| mriedem | here is the g-r change https://review.openstack.org/#/c/229954/ | 15:13 |
| mriedem | wait wrong branch | 15:13 |
| sdague | mriedem: heh | 15:13 |
| mriedem | oh, .gitreview isn't updated on g-r | 15:14 |
| mriedem | yar https://review.openstack.org/#/c/229956/ | 15:15 |
| mriedem | ok, working on master branch release now | 15:16 |
| mriedem | sdague: looking at the diffs in novaclient on master, i don't think there is anything that would really warrant a minor version bump, but we can't release it as 2.30.1, and 2.30.2 would be weird, so i'll just make that 2.31.0 | 15:20 |
| mriedem | 3350a57 is probably ok for a minor | 15:20 |
| mriedem | dhellmann: how do i prime the novaclient releases yaml in the releases repo for mitaka? just copy python-novaclient.yaml from liberty and add to a mitaka subdir? | 15:22 |
| dhellmann | mriedem: yeah, start a new file in a mitaka directory with the single release you want | 15:25 |
| mriedem | dhellmann: there should only be the one release in there? | 15:25 |
| dhellmann | mriedem: yes, to start that cycle you should only have one release | 15:26 |
| mriedem | ok | 15:26 |
| openstackgerrit | Matt Riedemann proposed openstack/releases: mitaka: release python-novaclient 2.31.0 https://review.openstack.org/229959 | 15:26 |
| mriedem | sdague: johnthetubaguy: ^ | 15:27 |
| ttx | dhellmann: when should we push reno support in stable/liberty ? | 15:28 |
| ttx | if it needs to be in the release we should push them for projects that have a RC2 currently opened | 15:29 |
| ttx | (glance, keystone, manila) | 15:29 |
| dhellmann | ttx: for stable/liberty we only need to add it to the test requirements and then we can add notes files without any build infrastructure around them | 15:29 |
| dhellmann | so no sphinx changes, no real requirements updates, etc | 15:29 |
| dhellmann | I don't know if we do releases for test requirements changes, so I don't know if we need to do that before rc2? | 15:29 |
| ttx | dhellmann: we could do it either way. Just seems simpler to include it as we respin a RC2 anyway ? | 15:30 |
| *** dims has joined #openstack-relmgr-office | 15:30 | |
| dhellmann | ttx: wfm | 15:30 |
| ttx | otherwise means we need to do it post-release which is a bit of a pain | 15:30 |
| ttx | since we'd rather drink ourselves to death when that comes | 15:31 |
| johnthetubaguy | mriedem: thank you, they both look good to me now | 15:31 |
| sdague | so, https://review.openstack.org/#/c/229952/1 is jenkins +1, dhellmann / ttx can we get a +A on that | 15:31 |
| dhellmann | yeah, I have a meeting starting right now so I was going to do it after that | 15:32 |
| ttx | sdague: better do it when we actually push it through the releasemachine | 15:32 |
| dims | dhellmann: want me to push that out? | 15:32 |
| dhellmann | dims: sure, thanks! | 15:32 |
| ttx | dims: wfm | 15:32 |
| dims | k looking | 15:32 |
| ttx | dims: you can +2/Approve on releases now right ? | 15:32 |
| dhellmann | dims: beware that the current release script is going to submit a constraints update, too, and mriedem has already done that | 15:33 |
| sdague | ok, I'm just eager to get that going so we can see what other monsters sit behind it | 15:33 |
| dhellmann | ttx: he could before, he's in library-releases | 15:33 |
| ttx | ok cool | 15:33 |
| dims | dhellmann: ack | 15:33 |
| sdague | so that we can actually start the liberty -> master testing and ensure we're not regressing | 15:33 |
| ttx | notmyname: hi! how is it looking in Swiftland ? | 15:33 |
| dhellmann | dims: I didn't update the script with the process change, so if you want to edit it locally and submit the tool change patch that'd be great | 15:33 |
| notmyname | ttx: aaaagghhhh! | 15:33 |
| notmyname | ;-) | 15:33 |
| ttx | notmyname: that's fine, we can cut stable/liberty from your last release :)Ã | 15:34 |
| notmyname | ttx: we're working hard. we've been making good progress. got a couple more to land | 15:34 |
| notmyname | ttx: nah, we'll be able to have something more recent than that :-) | 15:34 |
| ttx | notmyname: ok, good :) | 15:34 |
| dims | dhellmann: let me see what shakes out when i run ./release_postversion.sh | 15:34 |
| dhellmann | dims: ++ | 15:35 |
| ttx | notmyname: still an ETA on Friday, or rather Monday ? | 15:35 |
| notmyname | ttx: however, I'm learning (or relearning) some interesting things this morning about distros | 15:35 |
| ttx | notmyname: I spent the last 5 years to forget everything about distros. | 15:35 |
| notmyname | ttx: time zones make it fun. I plan on having a SHA in your email inbox before your monday am. ie patches sent to the gate by EoD my friday | 15:35 |
| ttx | notmyname: I might pick some up on Saturday morning | 15:36 |
| notmyname | ok | 15:36 |
| bswartz | ttx: added 2 more things to manila liberty-RC2 https://launchpad.net/manila/+milestone/liberty-rc2 https://review.openstack.org/#/q/project:openstack/manila+branch:stable/liberty,n,z | 15:36 |
| ttx | notmyname: we can communicate through the openstack/releases repo. Push the SHA there when you're happy | 15:36 |
| notmyname | ttx: yeah. canonical has never packaged swift's kilo version. and it seems debian has "liberty RC1" but that doesn't include swift (which might not be bad, because it seems swift is not in the expiramental part of debian) | 15:36 |
| notmyname | ttx: yeah. that's what I mean. will do :-) | 15:37 |
| ttx | bswartz: noted | 15:37 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack-infra/release-tools: Fix ImportError: No module named simplejson https://review.openstack.org/229969 | 15:42 |
| dims | sdague: 2.30.1 is on pypi now - https://pypi.python.org/pypi/python-novaclient | 15:42 |
| sdague | dims: thanks! | 15:44 |
| openstackgerrit | Merged openstack/releases: liberty: release 2.30.1 for python-novaclient https://review.openstack.org/229952 | 15:45 |
| dims | sdague: mriedem: need to fix the requirements patch - https://review.openstack.org/#/c/229962/ with the correct version number | 15:49 |
| mriedem | dims: wrong change | 15:49 |
| dims | sdague: mriedem: also do we want to bad novaclient versions in g-r? | 15:49 |
| dims | s/want to bad/want to ban bad/ | 15:50 |
| mriedem | no | 15:50 |
| sdague | dims: honestly, I think it's fine to leave it | 15:50 |
| dims | mriedem: sdague: ok | 15:50 |
| dims | mriedem: which is the correct requirements change? | 15:50 |
| sdague | dims: there are 2 | 15:51 |
| mriedem | https://review.openstack.org/#/c/229956/ is liberty | 15:51 |
| sdague | because branches | 15:51 |
| jroll | ttx: dhellmann: feel free to ignore me until the current fires are out... but thinking we want to do a 1.0.0 release of ironic-python-agent, and make that stable/liberty. we've never released IPA before, so starting the process with https://review.openstack.org/#/c/229983/ . is that all good with y'all? | 15:51 |
| mriedem | https://review.openstack.org/#/c/229962/ is master | 15:51 |
| jroll | ttx: dhellmann: nothing depends on IPA, we wouldn't need a g-r change or whatever | 15:51 |
| dims | thanks mriedem | 15:52 |
| dims | sdague: ack | 15:52 |
| ttx | jroll: let's wait for dhellmann, he knows the implications better than I do | 15:52 |
| ttx | jroll: I'm fine with it if he is fine with it | 15:53 |
| jroll | ttx: cool, thanks | 15:53 |
| openstackgerrit | Jim Rollenhagen proposed openstack/releases: Release ironic-python-agent 1.0.0 https://review.openstack.org/229986 | 15:59 |
| jroll | ^ not ready yet, waiting on one more patch | 15:59 |
| dims | jroll: is there a companion review for project-config/gerrit/acls/openstack/ironic-python-agent.config to fix pushSignedTag? (see http://docs.openstack.org/infra/manual/creators.html) | 16:01 |
| jroll | dims: oops, I missed that | 16:02 |
| jroll | dims: I guess I'll need a bunch of changes there, to add stable access etc, cool with just copying that from ironic? | 16:03 |
| dims | jroll: ++ :) | 16:03 |
| jroll | dims: not sure what pushSignedTag should be though; that section isn't in ironic.config, I assume there's some default and I should just leave it out? | 16:04 |
| dims | [access "refs/tags/*"] | 16:05 |
| dims | pushSignedTag = group library-release | 16:05 |
| jroll | it isn't a library | 16:05 |
| dims | ah sorry, let's check with dhellmann then | 16:06 |
| jroll | hehe | 16:06 |
| * dims trying to help and learn at the same time :) | 16:06 | |
| jroll | ditto! | 16:07 |
| *** thingee has joined #openstack-relmgr-office | 16:08 | |
| jroll | dims: https://review.openstack.org/#/c/229993/ | 16:09 |
| jroll | hopefully dhellmann can verify my assumptions there | 16:09 |
| *** mriedem is now known as mriedem_lunch | 16:22 | |
| *** thingee has quit IRC | 16:23 | |
| *** stevemar_ has joined #openstack-relmgr-office | 16:25 | |
| *** vipul has quit IRC | 16:27 | |
| *** stevemar has quit IRC | 16:27 | |
| *** vipul has joined #openstack-relmgr-office | 16:28 | |
| *** dtantsur is now known as dtantsur|afk | 16:29 | |
| *** mriedem_lunch is now known as mriedem | 16:30 | |
| *** bdemers has quit IRC | 16:31 | |
| *** bdemers has joined #openstack-relmgr-office | 16:31 | |
| dhellmann | jroll, dims : "Release Managers" should be right | 16:32 |
| jroll | dhellmann: for refs/tags/*? is that the default or does it need to be explicit (I notice ironic doesn't have that section) | 16:33 |
| dhellmann | jroll, dims : I believe removing the override for pushSignedTag leaves it to the default, but it would be good to get infra to confirm that | 16:33 |
| jroll | fungi: ^ can you confirm this assumption? (see also my comment here) https://review.openstack.org/#/c/229993/1/gerrit/acls/openstack/ironic-python-agent.config | 16:34 |
| fungi | you can look at the global gerrit acl from which the per-project acls inherit https://review.openstack.org/gitweb?p=All-Projects.git;a=blob_plain;f=project.config | 16:36 |
| fungi | note that in the [access "refs/*"] section there's pushSignedTag = group Release Managers | 16:36 |
| jroll | fungi: perfect, thank you | 16:37 |
| fungi | so by default, anyone in the Release Managers group can push tags for any repo in gerrit _unless_ there's an exclusive group permission in the project-specific acl for that | 16:37 |
| jroll | yep, makes sense | 16:37 |
| fungi | adding a pushSignedTag permission for some other group in your per-project acl is simply additive | 16:37 |
| fungi | (unless it's set as an exclusive group permission that is) | 16:38 |
| dims | ah fungi good to know | 16:38 |
| fungi | would have to say something like exclusiveGroupPermissions = pushSignedTag to prevent inheriting from the global acl | 16:39 |
| fungi | jroll: i'm glad it makes sense to you. gerrit acl inheritance is complex enough it trips me up pretty often | 16:39 |
| jroll | fungi: I'm sure if I didn't focus on a single piece and instead looked at the whole picture, my head would spin | 16:40 |
| dhellmann | fungi: thanks for explaining that, I need to get that written down somewhere since it comes up so often | 16:41 |
| dhellmann | dims, ttx, sdague, mriedem : I need to step out for a bit, but it seems like the novaclient thing is handled? | 16:41 |
| sdague | dhellmann: yes | 16:42 |
| sdague | we're good on that | 16:42 |
| dims | dhellmann: the liberty 2.30.1 - yes | 16:42 |
| dhellmann | ack, good, thanks tims | 16:42 |
| dhellmann | er, dims | 16:42 |
| dims | :) | 16:42 |
| dhellmann | do we need the 2.31.0 release, too? | 16:43 |
| mriedem | yeah i think so | 16:43 |
| dhellmann | dims: have a minute to do that, or should I do it when I get back from this errand? | 16:44 |
| fungi | dhellmann: https://review.openstack.org/Documentation/access-control.html is a good read | 16:44 |
| dims | dhellmann: i did not see any mitaka/*.yaml files...hence was not sure | 16:44 |
| dhellmann | fungi: aha, thanks for that | 16:44 |
| dims | dhellmann: yes, i can take care of it if you can confirm mitaka libraries are ok | 16:45 |
| dims | to release | 16:45 |
| dhellmann | dims: it should be ok to go ahead, as long as the commit is not on the stable branch | 16:45 |
| dhellmann | dims: we need the constraints update, too | 16:45 |
| dims | dhellmann: thanks | 16:45 |
| dhellmann | dims: thanks for diving in, ill timed appointment on my end | 16:46 |
| dims | no worries dhellmann | 16:46 |
| dims | sdague: mriedem: one of you please fix this nit? https://review.openstack.org/#/c/229959/1/deliverables/mitaka/python-novaclient.yaml,cm | 16:51 |
| mriedem | i can do that quick | 16:51 |
| dims | thanks matt | 16:51 |
| openstackgerrit | Matt Riedemann proposed openstack/releases: mitaka: release python-novaclient 2.31.0 https://review.openstack.org/229959 | 16:52 |
| openstackgerrit | Merged openstack/releases: mitaka: release python-novaclient 2.31.0 https://review.openstack.org/229959 | 17:00 |
| *** mriedem has quit IRC | 17:02 | |
| *** dims_ has joined #openstack-relmgr-office | 17:03 | |
| *** dims has quit IRC | 17:04 | |
| *** mriedem has joined #openstack-relmgr-office | 17:07 | |
| *** stevemar_ has quit IRC | 17:18 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/releases: Release ironic-lib 0.2.0 https://review.openstack.org/230036 | 17:41 |
| * dhellmann returns | 17:57 | |
| *** openstackgerrit has quit IRC | 18:31 | |
| *** openstackgerrit has joined #openstack-relmgr-office | 18:32 | |
| dims_ | dhellmann: sdague pointed to a problem with grenade update from liberty->mater in (see gate-grenade-dsvm logs in https://review.openstack.org/#/c/229363/) | 18:49 |
| dims_ | dhellmann: pulling in a couple of oslo.service fixes from master to stable/liberty to see if that fixes the issue | 18:49 |
| dhellmann | dims_: ack | 18:50 |
| dims_ | dhellmann: we'll need a 0.9.1 of oslo.service i think | 18:50 |
| dhellmann | that's easy enough | 18:50 |
| dims_ | dhellmann: will ping you when the changes merge | 18:50 |
| dims_ | hopefully it fixes it! :) | 18:50 |
| dhellmann | dims_: great, thanks for handling the debugging | 18:50 |
| openstackgerrit | Doug Hellmann proposed openstack-infra/release-tools: do not update constraints list on each new release https://review.openstack.org/230080 | 19:01 |
| *** gordc has quit IRC | 19:04 | |
| openstackgerrit | Doug Hellmann proposed openstack/releases: change the master/default branch to mitaka https://review.openstack.org/230083 | 19:04 |
| openstackgerrit | Doug Hellmann proposed openstack/releases: Release python-ironicclient 0.9.0 https://review.openstack.org/227423 | 19:04 |
| *** stevemar has joined #openstack-relmgr-office | 19:04 | |
| openstackgerrit | Doug Hellmann proposed openstack/releases: Release ironic-lib 0.2.0 https://review.openstack.org/230036 | 19:04 |
| dhellmann | dims_: it turns out you were right to think we should wait with mitaka releases, we should take https://review.openstack.org/230083 before we accept too many more | 19:06 |
| dims_ | :) | 19:06 |
| dhellmann | I don't think it caused any harm, but it would be better to have everything lined up | 19:06 |
| dims_ | absolutely ++ | 19:07 |
| dhellmann | dims_: were you going to handle the osc release in https://review.openstack.org/#/c/229694/1 or shall I? | 19:08 |
| openstackgerrit | Monty Taylor proposed openstack/releases: Release 1.7.6 of os-client-config https://review.openstack.org/230085 | 19:08 |
| dims_ | dhellmann: please go ahead | 19:08 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/release-tools: Rework translation-cleanup https://review.openstack.org/227454 | 19:11 |
| openstackgerrit | Merged openstack/releases: OpenStackClient 1.7.1 https://review.openstack.org/229694 | 19:13 |
| *** david-lyle has quit IRC | 19:18 | |
| openstackgerrit | Doug Hellmann proposed openstack/releases: Release 1.7.6 of os-client-config https://review.openstack.org/230085 | 19:18 |
| dhellmann | jroll: did you mean for the ironicclient release to leave out some changes from master? http://logs.openstack.org/23/227423/2/check/gate-releases-tox-list-changes/4bdfabc/console.html | 19:19 |
| jroll | dhellmann: no, but I had put that up like a week ago, I assume that's why | 19:21 |
| jroll | dhellmann: I can update it, or you can, either way | 19:21 |
| dhellmann | jroll: or we could leave it alone, it's up to you | 19:22 |
| dhellmann | I mean, I can release it as originally requested | 19:22 |
| jroll | dhellmann: nah, no reason not to pick them up | 19:22 |
| jroll | I'll get to it in a few | 19:22 |
| dhellmann | jroll: sounds good, I'll look at the ironic-lib release in the mean time | 19:22 |
| jroll | thanks | 19:22 |
| *** david-lyle has joined #openstack-relmgr-office | 19:27 | |
| openstackgerrit | Merged openstack/releases: change the master/default branch to mitaka https://review.openstack.org/230083 | 19:28 |
| openstackgerrit | Jim Rollenhagen proposed openstack/releases: Release python-ironicclient 0.9.0 https://review.openstack.org/227423 | 19:29 |
| jroll | dhellmann: ^ there you are sir | 19:30 |
| dhellmann | jroll: ty | 19:32 |
| jroll | np | 19:33 |
| openstackgerrit | Monty Taylor proposed openstack/releases: Release 1.8.0 of os-client-config https://review.openstack.org/230085 | 19:37 |
| openstackgerrit | Doug Hellmann proposed openstack/releases: show more verbose git log output https://review.openstack.org/230103 | 19:41 |
| openstackgerrit | Merged openstack/releases: Release ironic-lib 0.2.0 https://review.openstack.org/230036 | 19:46 |
| openstackgerrit | Merged openstack/releases: Release 1.8.0 of os-client-config https://review.openstack.org/230085 | 19:53 |
| *** gordc has joined #openstack-relmgr-office | 19:55 | |
| ttx | stevebaker: around? | 20:09 |
| stevebaker | ttx: hi | 20:09 |
| ttx | stevebaker: did you have time to look at the blueprints I listed the other day ? | 20:10 |
| stevebaker | ttx: umm, I think I missed that | 20:10 |
| ttx | you can let me know which milestone to target them to, as some of the milestones are closed and reopening them is a bit painful | 20:10 |
| ttx | https://blueprints.launchpad.net/heat/+spec/functional-tests | 20:11 |
| ttx | https://blueprints.launchpad.net/heat/+spec/reorg-asg-code | 20:11 |
| ttx | https://blueprints.launchpad.net/heat/+spec/add-cinder-backup-constraint | 20:11 |
| ttx | https://blueprints.launchpad.net/heat/+spec/oslo-i18n | 20:11 |
| ttx | https://blueprints.launchpad.net/heat/+spec/template-unicode-support | 20:11 |
| ttx | https://blueprints.launchpad.net/heat/+spec/support-cinder-volume-type-access-operations | 20:11 |
| ttx | https://blueprints.launchpad.net/heat/+spec/cinder-volume-encryption | 20:11 |
| ttx | https://blueprints.launchpad.net/heat/+spec/enhance-property-constraints | 20:11 |
| ttx | https://blueprints.launchpad.net/heat/+spec/resource-for-volume-backup-job | 20:11 |
| ttx | https://blueprints.launchpad.net/heat/+spec/run-heat-api-under-apache2 | 20:11 |
| ttx | stevebaker: here is the list ^ | 20:11 |
| ttx | stevebaker: in other news, any urgency in opening a RC2 ? Or can it wait onday so that we catch translations ? | 20:11 |
| ttx | Monday* | 20:12 |
| stevebaker | ttx: Monday will be fine | 20:12 |
| stevebaker | ttx: so should I go through each bp and comment on the whiteboard what milestone it landed in? | 20:13 |
| ttx | stevebaker: or tell me here | 20:13 |
| stevebaker | ttx: ok, I'll work through them and comment here throughout the day | 20:14 |
| ttx | sounds good! thanks. | 20:14 |
| ttx | SlickNik, SergeyLukjanov : ping me if around | 20:14 |
| openstackgerrit | Merged openstack/releases: Release python-ironicclient 0.9.0 https://review.openstack.org/227423 | 20:17 |
| openstackgerrit | Doug Hellmann proposed openstack/releases: show more verbose git log output https://review.openstack.org/230103 | 20:32 |
| openstackgerrit | Doug Hellmann proposed openstack/reno: add better usage instructions https://review.openstack.org/226572 | 20:36 |
| openstackgerrit | Doug Hellmann proposed openstack/reno: add documentation for the sphinx extension https://review.openstack.org/226584 | 20:36 |
| openstackgerrit | Doug Hellmann proposed openstack/reno: Handle errors scanning a branch https://review.openstack.org/229424 | 20:36 |
| openstackgerrit | Merged openstack-infra/release-tools: Sync with global requirements https://review.openstack.org/227086 | 20:36 |
| openstackgerrit | Doug Hellmann proposed openstack/reno: add better usage instructions https://review.openstack.org/226572 | 20:36 |
| openstackgerrit | Doug Hellmann proposed openstack/reno: add documentation for the sphinx extension https://review.openstack.org/226584 | 20:36 |
| openstackgerrit | Doug Hellmann proposed openstack/reno: Handle errors scanning a branch https://review.openstack.org/229424 | 20:36 |
| openstackgerrit | Doug Hellmann proposed openstack/reno: add better usage instructions https://review.openstack.org/226572 | 20:59 |
| openstackgerrit | Doug Hellmann proposed openstack/reno: add documentation for the sphinx extension https://review.openstack.org/226584 | 20:59 |
| openstackgerrit | Doug Hellmann proposed openstack/reno: Handle errors scanning a branch https://review.openstack.org/229424 | 20:59 |
| SergeyLukjanov | ttx, pong | 21:35 |
| SergeyLukjanov | sorry, was on a lunch | 21:36 |
| *** mriedem has quit IRC | 21:43 | |
| *** david-lyle has quit IRC | 21:43 | |
| *** gordc has quit IRC | 21:46 | |
| openstackgerrit | Doug Hellmann proposed openstack/reno: support multi-line notes and notes with complex formatting https://review.openstack.org/230162 | 21:50 |
| openstackgerrit | Doug Hellmann proposed openstack/reno: reverse slug and uuid order in filenames https://review.openstack.org/230163 | 21:50 |
| *** spzala has quit IRC | 21:54 | |
| *** spzala has joined #openstack-relmgr-office | 21:56 | |
| *** david-lyle has joined #openstack-relmgr-office | 21:56 | |
| *** johnsom has joined #openstack-relmgr-office | 21:59 | |
| stevebaker | ttx: https://blueprints.launchpad.net/heat/+spec/functional-tests liberty-1 | 22:09 |
| stevebaker | ttx: actually, https://blueprints.launchpad.net/heat/+spec/functional-tests kilo-1 | 22:10 |
| stevebaker | ttx: https://blueprints.launchpad.net/heat/+spec/reorg-asg-code kilo-3 | 22:11 |
| stevebaker | ttx: https://blueprints.launchpad.net/heat/+spec/add-cinder-backup-constraint liberty-3 | 22:12 |
| stevebaker | ttx: https://blueprints.launchpad.net/heat/+spec/oslo-i18n kilo-1 | 22:14 |
| stevebaker | ttx: https://blueprints.launchpad.net/heat/+spec/template-unicode-support kilo-1 | 22:16 |
| stevebaker | ttx: https://blueprints.launchpad.net/heat/+spec/support-cinder-volume-type-access-operations liberty-3 | 22:17 |
| stevebaker | ttx: https://blueprints.launchpad.net/heat/+spec/cinder-volume-encryption liberty-3 | 22:18 |
| stevebaker | ttx: https://blueprints.launchpad.net/heat/+spec/enhance-property-constraints liberty-1 | 22:19 |
| stevebaker | ttx: thats it, I've dealt with the others | 22:21 |
| *** SergeyLukjanov has quit IRC | 22:51 | |
| *** mriedem has joined #openstack-relmgr-office | 22:51 | |
| *** SergeyLukjanov has joined #openstack-relmgr-office | 22:51 | |
| *** sigmavirus24 is now known as sigmavirus24_awa | 22:59 | |
| *** stevemar has quit IRC | 23:02 | |
| *** mriedem has quit IRC | 23:10 | |
| *** mriedem has joined #openstack-relmgr-office | 23:10 | |
| *** mriedem has quit IRC | 23:15 | |
| *** mriedem has joined #openstack-relmgr-office | 23:25 | |
| *** johnsom has quit IRC | 23:32 | |
| *** johnsom has joined #openstack-relmgr-office | 23:32 | |
| *** TravT has quit IRC | 23:37 | |
| *** dims has joined #openstack-relmgr-office | 23:56 | |
| *** bnemec has joined #openstack-relmgr-office | 23:57 | |
| *** dims has quit IRC | 23:58 | |
| *** dims_ has quit IRC | 23:58 | |
| SlickNik | ttx: around now. Now a good time to talk? | 23:59 |
| *** dims has joined #openstack-relmgr-office | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!