Monday, 2015-08-24

openstackgerritBrant Knudson proposed openstack/releases: keystoneauth 0.4.0  https://review.openstack.org/21357301:13
*** sigmavirus24_awa is now known as sigmavirus2401:48
*** bnemec has quit IRC01:50
*** sigmavirus24 is now known as sigmavirus24_awa03:20
*** openstackgerrit has quit IRC03:31
*** openstackgerrit has joined #openstack-relmgr-office03:32
*** sigmavirus24_awa is now known as sigmavirus2404:23
*** sigmavirus24 is now known as sigmavirus24_awa04:34
openstackgerritAndreas Jaeger proposed openstack/releases: Fix invocation of git_log  https://review.openstack.org/21611106:20
openstackgerritAndreas Jaeger proposed openstack/releases: Fix invocation of git_log  https://review.openstack.org/21611106:42
*** flaper87 has quit IRC07:08
*** flaper87 has joined #openstack-relmgr-office07:08
openstackgerritMerged openstack/releases: fix releases where SHA is not HEAD  https://review.openstack.org/21579009:57
dhellmanngood morning10:45
*** isviridov_away has quit IRC10:55
*** jroll has quit IRC10:56
*** jroll has joined #openstack-relmgr-office11:02
*** isviridov_away has joined #openstack-relmgr-office11:04
dhellmannttx; I'm looking at the release tags governance changes this morning. I see a note about renaming cycle-with-milestones to once-per-cycle. How strongly do you feel we need to do that?11:14
*** gordc has joined #openstack-relmgr-office11:43
ttxdhellmann: I think once-per-cycle is clearer, but I'd like to stop gratuitously changing tags as we near the end of the cycle, which means we need to decide soon :)11:46
ttxI guess cycle-with-milestones could do it11:47
dhellmannttx: ok, I'm not sure either is clearer than the other myself11:47
ttxok, then we should keep it as is11:47
ttxoh, I remember now11:48
ttxdhellmann: it's when we said that "milestones" might not match beta tags in the next cycle11:48
dhellmannoh, good point11:49
ttxso to support projects that would not follow milestones (but still issue beta tags leading to a single release)11:49
dhellmannbut I expect most projects to move to cycle-with-intermediary at that point11:49
ttxwhich we don't need to track separatelyt from cycle-with-milestones11:49
dhellmanncycle-with-intermediary projects treat milestones as dates on the schedule, cycle-with-milestones add beta tags on those dates11:50
ttxthinking from a consumer perspective, I think what they want to know is whether they can expect intermediary releases, or if the project won't make any.11:51
ttxhence the "once-per-cycle"11:51
ttxthey don't really care about milestones11:51
ttxthose are dev artifacts11:51
ttxso it's not necessarily clearer, but it's closer to the question it's supposed to answer11:52
dhellmannyeah, and I'm not sure we can always predict when a project will make releases mid-cycle. I'm not sure we want a model that forbids it.11:52
dhellmannother than the milestone model, that is11:52
ttxI don't think you would accidentally switch from a "single release" to a "any number of releases" model by accident though11:53
ttxonce model uses a mostly-flat model, the other uses a bell curve model11:53
ttxone*11:53
dhellmannhmm11:54
ttxif some team decides to switch in the middle, they switch11:54
ttxbut I don't think they would flap between the two11:54
ttxit's a maturity thing11:54
ttxwe could also decide it's not a question anyone but us want answered11:55
ttxand remove those tags altogether11:55
dhellmannremove all of the release model tags?11:57
ttxif we can't figure out the question enough to judge the quality of the answer... that means we aren't sure about the question11:58
ttxfeels like those tags are for us (buckets for work and process) and not to answer a question downstream users have11:58
* dhellmann stops submitting patches to update release tags11:59
dhellmannI'm not sure. I think they're probably more useful to distros than to deployers.11:59
ttxdistros are a "downstream user" to me12:00
ttxLet's just keep using the two tags we have now12:00
dhellmannof course if we get most everyone onto the cycle-with-intermediary model, there will be fewer exceptions12:00
dhellmannthe goal of my original patch was to get us into a state where we could require all new repositories to declare their release model12:01
ttxagreed12:01
ttxrelease-none is imho a useful bit of info for downstream :)12:01
dhellmannbreaking up the patch will take more work to get there, but do you still think that's a useful end state?12:01
dhellmannheh12:02
ttxI think it's a useful end state.12:02
ttxI just have trouble seeing the future enough to see which of those will end up being useful one year from now12:03
dhellmannok, sure12:03
*** dims has joined #openstack-relmgr-office13:12
*** sigmavirus24_awa has quit IRC13:22
*** jroll has quit IRC13:23
*** dims has quit IRC13:23
*** sigmavirus24_awa has joined #openstack-relmgr-office13:28
openstackgerritAndreas Jaeger proposed openstack/releases: openstackdocstheme 1.2.1 release  https://review.openstack.org/21602813:31
*** jroll has joined #openstack-relmgr-office13:37
*** dims_ has joined #openstack-relmgr-office13:52
openstackgerritAndreas Jaeger proposed openstack/releases: openstack-doc-tools 0.30.0 release  https://review.openstack.org/21626414:01
dhellmannttx: I'm going to go through the list of release requests and publish them. Let me know if you want to talk through the end-of-release sequence more, or if you want to talk to lifeless first.14:01
ttxdhellmann: I'll start a specific thread about it, and talk to lifeless tomorrow14:02
dhellmannok14:02
ttxnotmyname: ping me when around14:03
*** bswartz has joined #openstack-relmgr-office14:05
bswartzdhellmann: ping re: http://lists.openstack.org/pipermail/openstack-dev/2015-August/072697.html14:06
bswartzdhellmann: there's no patch in the listed link for manila, however I don't think any changes are required14:06
dhellmannbswartz: pong14:06
dhellmannbswartz: checking...14:06
dhellmannbswartz: yes, I think all your repos had a release tag already. Do you have a specs repo?14:07
bswartzdhellmann: no spcs14:07
bswartzno specs* repo14:08
dhellmannok, that's usually the one that even an up to date project has missing a tag14:08
openstackgerritMerged openstack/releases: openstackdocstheme 1.2.1 release  https://review.openstack.org/21602814:13
ttxdhellmann, lifeless: pushed to ML14:17
*** ativelkov has joined #openstack-relmgr-office14:26
*** jokke_ has joined #openstack-relmgr-office14:26
nikhil_kttx: dhellmann hi14:26
dhellmannhi, nikhil_k14:27
nikhil_kativelkov: has a question regarding python-glanceclient release of a experimental branch for liberty to be used by murano14:27
*** mfedosin has joined #openstack-relmgr-office14:27
nikhil_kdhellmann: we wish to seek a solution that will not break gates and help murano consume the lib14:27
notmynamettx: around14:28
nikhil_kdhellmann: there's a feature branch for this experiemental feature and wondering if there was a way to release client for this purpose14:28
ttxnotmyname: hi! wanted to discuss th need to have "release candidates" for swift's final liberty release. A bit of historical context first14:29
*** sergmelikyan has joined #openstack-relmgr-office14:29
ttxnotmyname: swift releases don't do RCs, we did tag RCs on the "final" one to emulate what was done with the milestone-droven thingies like Nova14:30
ttxnotmyname: now more things move to an "intermediary release" ala-swift model14:30
ttxwhich doesn't really need RCs14:31
ttxsince you can just apply a new .Z tag14:31
ttxThis has several benefits14:31
dhellmannnikhil_k: let me think about that a bit while ttx and notmyname chat14:31
ttxOne is that you don't really make any release "special"14:31
nikhil_kdhellmann: thanks!14:31
ttxintermediary and end-of-cycle swift releases follow same process14:31
ttxthe other is that you don't have to retag the rc1 to final at the end of the cycle14:32
ttxnotmyname: what do you think about not using RCs for the final swift libert release ?14:32
notmynamea few things14:32
ttxISTR I was the one asking for it back then, not you, but thought I'd rather doublecheck14:33
notmyname1) ultimately it doesn't really matter since, for my perspective, it's low-cost either way14:33
notmyname2) it's not the tags that make a release special. it's the stuff around it (eg this is the Integrated Release. all the marketing etc). so if that's still happening, then that tag is still more equal than the others14:34
ttxnotmyname: re:(2) the only difference is that instead of tagging 2.3.4rc1, 2.3.4rc2, 2.3.4 you would tag 2.3.4, 2.3.5 on the stable/liberty branch before the end of the cycle14:35
notmyname3) I'd be happy to drop the rc tag dance and the milestone-proposed branch (more happy about not doing m-p), but I will be the first to bring up that we have used that to find and fix bugs. the "cost" of doing new tags, and potentially having several right around release, is the tradeoff, I think14:35
ttxAgree the "last one" is more equal. But "last one" doesn't necessarily mean "using RCs"14:36
ttxnotmyname: we don't do m-p since kilo14:36
ttxthere we did stable/kilo directly14:36
notmynameright :-)14:37
ttxbasically the proposed process is:14:37
ttx- you come up with something that looks like it will be the final swift release for liberty14:37
ttx- you tag that (2.3.4)14:37
ttx- we create a stable/liberty release branch from there14:37
ttx- you find critical issues in that branch14:38
ttx- you fix in master and backport the fix14:38
ttx- you tag a new release on the release branch (2.3.5)14:38
ttx- it's release day ! fireworks14:38
ttx- stable/liberty becomes a stable branch and only has security fixes and major issues backported.14:38
ttxnext release of swift is done off master and called 2.4.0 or 3.0.014:39
notmynameok. so the only difference is bumping the rev version number instead of doing a bunch of rc tags. otherwise the same as normal14:39
ttxyeah14:39
notmynameah...except14:39
notmynameso everything sounds great until that last thing14:39
notmynamemeans that again the last release is more special, and now not just because of marketing14:40
notmynamemeans that it's the last of the X.Y series14:40
ttxthat's more an artifact of having a stable branch14:40
ttxand it's true already14:41
notmynameit's that you can't follow a Liberty release with a quick release of bugfixes14:41
ttxphone ringing14:41
ttxok, call dismissed14:42
notmynamebut TBH, that's somethign I can probably get over. the practical reality is that there will be (for the foreseeable future) new stuff more than just bugfixes in releases after a cycle release14:42
ttxnotmyname: the alternative is.. not being able to issue a point release on the stable branch at all14:42
ttxI guess that could work too14:42
ttxwe kind of need that for libraries, but probably not require it for services14:43
ttxdhellmann: opinion on that ?14:43
* ttx checks if we did that for swift in the past14:44
* dhellmann reads scrollback14:44
notmynamettx: what do you mean?14:44
ttxissue a point release on a stable branch14:45
notmyname"not doing point releases on stable"? the .Z would always be 0?14:45
ttxnotmyname: it's quite orthogonal to the change I'm discussing though. We have that issue whether we use RCs or not14:45
dhellmannttx: we do want point releases for both services and libs on stable branches14:45
openstackgerritMerged openstack/releases: pycadf 1.1.0  https://review.openstack.org/21523214:46
ttxthe question of bumping Y on the first release of a new dev cycle is not linked to the question of using RCs in the final release process14:46
openstackgerritDavid Lyle proposed openstack/releases: Release django_openstack_auth 1.3.2  https://review.openstack.org/21468914:46
notmynamettx: true. the rc-tag-ectomy is fine with me. the rest is making sure we dont' become slave to our own processes14:46
openstackgerritDavid Lyle proposed openstack/releases: Release django_openstack_auth 1.4.0  https://review.openstack.org/21468914:47
jokke_?wi2314:48
ttxnotmyname: ok, we'll discuss the Y start-of-cycle bump separately then14:48
notmynamettx: if we have our strict x.y.z policy that we must never deviate from (which is something we made up for ourselves), then what you describe is the way to do it: .Y gets increased the first release after a cycle release. if we allow for a x.y.z.Ω for backports, that goes away (but breaks the policy)14:50
notmynamettx: to be clear, it doesn't really matter to me14:50
notmyname(it's near the bottom of my Things I Want to Spend a Lot of Time Thinking About list ;-)14:50
ttxnotmyname: understood14:51
notmynamedhellmann: I got a reply from graham about wsgi. thanks again14:51
ttxnikhil_k: so... having a special release for Murano triggers plenty of "WRONG" signs blinking around me14:52
ttxbeyond the technical madness, it's a slippery slope14:53
ttxexperimentation is fine, but having /some/ projects wanting to consume that experimentation is where the bucket stops imho14:53
ttxit's either released or it's not.14:54
ttxSo the only way to make it work woud be to create a completely separate project called glance-for-murano and release that14:54
nikhil_kttx: I see, this is py-client though.14:55
nikhil_kttx: and yes, I was thinking the same that we might possibly need to releae this feature branch in pip as some py-glanceclient-for-murano14:55
ttxglance-client-for-murano then. Or you manage to merge the feature back in python-glanceclient proper and trigger the code path using --murano14:56
nikhil_kttx: please no, having it in the master is more troubling!14:56
nikhil_kativelkov: jokke_ ^14:56
nikhil_kmfedosin: ^14:56
ttxnikhil_k: maybe more troubling for you, but less troubling for me, the gate, QA guys and the world :)14:56
nikhil_khaha14:57
ativelkovsergmelikyan: ^14:57
nikhil_kttx: it shall be more troubling for all of us in the near future if we don't have it sperate, please trust me on that!14:57
ttxnikhil_k: how about Murano team waits for that experimentation to be merged back ? Or better, contribute resources to finish it asap ?14:58
* ttx has no idea what feature we are tralking about here, so probably conjecturing14:59
nikhil_kttx: it's the artifacts thing14:59
sergmelikyanativelkov: correct me, if I am wrong, but I guess it's not the question of resources for now14:59
nikhil_kthe Glance API is being restructured as per suggestions from the API_WG14:59
ativelkovmurano team is contributing the resources (I am the resource) :)14:59
ttxjust saying I see no way of having parallel release channels for experimental branches that would make sense in a x.y.z versioning and not break everything if done on the same namespace14:59
nikhil_kso we want the client to support minimalistic for now14:59
ttxativelkov: sounds fun :)15:00
sergmelikyanativelkov: ttx is asking if it's possible to add more people in order to finish on time15:00
ativelkovFolks, there is one more option15:02
ttxso yeah, slightly less worse options would be: copy-paste the client code to Murano, publish it temporarily under a glanceclient-for-murano repo. Best solution would be to wait until it's released, but I suspect that was considered and rejected15:02
ativelkovttx - you stole it!15:02
ativelkovI just wanted to propose the same :)15:02
ttxativelkov: it's not a good option, but at least it's not a completely wrong one15:03
ativelkovSo, if sergmelikyan is ok with some copy-paste from glanceclient to temporary land in muranoclient, then we'll have the solved15:03
nikhil_kI am okay with that because I trust ativelkov . We shall have a consistent working code next cycle!15:04
sergmelikyanativelkov: I think we are ok with that, it will allow us to make sure that code is complete and contribute back in beginning of the M15:04
sergmelikyanonce we will be able to release first version of glanceclient15:05
sergmelikyanwith artifacts15:05
nikhil_kawesome15:06
ativelkovOK, so the feature branch will remain in glance for now, but will be published in python-muranoclient releases in L15:06
ativelkovi.e. we will copy-paste the code from python-glanceclient/feature/artifacts to python-muranoclient/master15:07
ativelkov(similar as we do with oslo incubator code)15:07
ativelkovAfter the v3 refactoring is complete and API is stable (in M), wel'll drop the copy from murano and merge the artifact client from feature/artifacts to master of python-glanceclient15:09
ativelkovLooks fine15:09
ativelkovthanks ttx :)15:09
ttxnp15:09
nikhil_kExcellent, thanks all!15:15
nikhil_kttx: what would be our freeze for release of clients?15:15
nikhil_kttx: just for awareness15:15
jokke_ativelkov, nikhil_k, sergmelikyan, mfedosin, ttx: thnx that sounds very sane resolution!15:23
dhellmannnotmyname: great, I hope things work out with graham & wsgi15:24
ativelkovnikhil_k: jokke_: I'll send an email to ML about this solution15:25
nikhil_kativelkov: thanks!15:25
dhellmannnikhil_k, ttx, ativelkov : have you explored the idea of releasing the experimental client under a different name?15:25
ativelkovdhellmann: well, adding15:26
ativelkovdhellmann: well, adding a new project just for one release sounds cumbersome15:26
dhellmannthat is, use the same feature branch in the repo but change the name of the dist that it builds15:26
dhellmannativelkov: all you have to do is change the name in setup.cfg15:27
ativelkovdhellmann: we'll get pythons' package name conflict in this cas15:27
ativelkovcase15:27
dhellmannah, true, so you'd have to rename the root package :-/15:27
ativelkovyup15:27
dhellmannwhich is still not actually that much work, but might not be worth it15:27
ativelkovIt's easier to copy-paste the glanceclient/v3 to muranoclient/artifacts and release with murano15:27
dhellmannativelkov: is this a whole new glance client api?15:28
jokke_Curly haired guy sitting at the end of our row ;)15:28
jokke_darn, sry15:28
ativelkovdhellmann: it will be. However for now it cannot work with the legacy images, just artifacts15:28
dhellmannativelkov: ok, I've caught up on scrollback now -- vendoring it in murano for now seems reasonable15:28
openstackgerritMerged openstack/releases: Release django_openstack_auth 1.4.0  https://review.openstack.org/21468915:33
* SergeyLukjanov doing openstack/django_openstack_auth release 1.4.0 now15:35
SergeyLukjanovdhellmann ^^15:35
*** dims__ has joined #openstack-relmgr-office15:36
dhellmannSergeyLukjanov: ack, thanks -- I assumed as much since you approved it15:36
SergeyLukjanov:)15:36
SergeyLukjanovheh, looks like we have no django_openstack_auth15:37
*** david-ly_ is now known as david-lyle15:38
SergeyLukjanovoh, I it's using - instead of _15:38
*** dims_ has quit IRC15:39
redrobothello relmgrs.  I believe the acl issues in python-barbicanclient have been fixed.  Could someone take a look at https://review.openstack.org/#/c/214280/ again?15:42
SergeyLukjanovredrobot, I'll do it15:43
redrobotSergeyLukjanov thanks!15:44
dhellmannSergeyLukjanov: do you think we can automate a check for release permissions so we don't run into a problem like ^^ again? I would have hated to approve that patch only to discover the permissions issue later15:44
openstackgerritMerged openstack/releases: python-barbicanclient 3.3.0  https://review.openstack.org/21428015:45
SergeyLukjanovdhellmann, I think it's possible, but could be complicated15:47
SergeyLukjanovdhellmann, we'll need to grep the ACL file path from the gerrit's project list file in project-config and then grep the needed rule in this file15:47
ttxSergeyLukjanov: announce approved15:58
SergeyLukjanovttx thx15:59
SergeyLukjanovredrobot, released16:04
*** dims__ has quit IRC16:12
*** dims has joined #openstack-relmgr-office16:13
*** dims is now known as dims__16:13
openstackgerritMerged openstack/releases: openstack-doc-tools 0.30.0 release  https://review.openstack.org/21626416:25
david-lyleSergeyLukjanov: thanks for the d-o-a release16:28
dhellmanndevananda, jroll : I'm looking at the ironic 4.0.0 release request (https://review.openstack.org/#/c/214301/). We need to coordinate this so we land a patch to remove the version from setup.cfg, too.16:36
dhellmanndevananda, jroll : I have to step out for a bit, but I should be able to help you with the release this week if we can find a good time to do it together.16:37
devanandaawesome, let's do it16:37
devanandadhellmann: also, I leave early thursday morning for black rock. will be offline completely for ~ 10 days16:37
devanandamaybe closer to 1416:38
dhellmanndevananda: ok, I have an errand I need to take care of but I should be back in about an hour16:38
devanandacool. i'll be around16:38
dhellmannI also need to think through the steps before we start16:38
dhellmannok, I'll ping you again16:38
redrobotSergeyLukjanov awesome, thank you!16:41
*** dims__ has quit IRC16:43
*** dims__ has joined #openstack-relmgr-office16:45
*** armax has joined #openstack-relmgr-office17:53
dhellmanndevananda: hey, I'm back if you're around17:56
*** bswartz has quit IRC17:57
devanandadhellmann: finishing a meeting, then a few wrap ups. around in 30 ?17:58
dhellmanndevananda: sure17:58
lifelessdims__: we should put that new fix of yours in a point release of pbr18:03
dims__lifeless: ++18:05
lifelessdims__: if I put up a releases patch, will you do it ?18:11
dims__lifeless: ack18:11
lifelessdims__: (I'm in quiet time just before daughter wakes right now ;))18:11
dims__totally hear you :)18:12
openstackgerritlifeless proposed openstack/releases: pbr 1.5.1  https://review.openstack.org/21636518:14
lifelessdims__: ^18:14
openstackgerritDoug Hellmann proposed openstack/releases: Make list-changes smarter  https://review.openstack.org/21637018:20
dhellmannlifeless, dims__ : ^^ should make it easier to look at the ironic release in https://review.openstack.org/#/c/214301/218:21
*** sergmelikyan has quit IRC18:30
openstackgerritDoug Hellmann proposed openstack/releases: keystoneauth 0.4.0  https://review.openstack.org/21357318:31
devanandadhellmann: ok - i think i can give this my undivided attention for a while :)18:37
dhellmanndevananda: ok, first question, I notice you're not releasing HEAD of master. Is that intentional?18:38
devanandalet me see what jroll was doing18:39
devanandaalso, it's unfortunate that he's on vacation this week, and I'm on vacation next week18:39
dhellmannthat change is about 40 patches earlier than head18:40
dhellmannthe sha in the request, that is18:40
dhellmannideally we would release head, land a patch to remove the version, and then you would continue landing regular patches18:40
devanandaright - I suspect that was HEAD when he proposed it18:41
devanandajudging by the proposal date, and merge date of that SHA18:41
dhellmannah, yeah18:41
devanandaI can update it18:41
dhellmannok, good18:41
dhellmannis there anything in the merge queue for ironic right now?18:41
* devananda checks18:41
dhellmannI don't see anything, so that's good18:41
devanandadhellmann: nothing afaict either18:42
dhellmannok, so while you update the release request I'll put up a patch to remove the version from ironic's setup.cfg18:43
devanandadhellmann: side question: my understanding is that we should still have a "stabilization period" on master before tagging a new major release18:43
devanandathis 4.0.0 is going to be a bit of an exception since we're in the ramp-up period right now. I know folks wanted to get this oiut a few weeks ago, when it was slower18:44
devanandabut coordination issues and then learning about the new relesae system stalled us18:44
dhellmannyeah, I think this came in while ttx and I were both out18:44
devanandaand other developers kept going18:44
devanandayea18:44
dhellmannswitching to post-versioning requires some coordination18:45
dhellmannafter that, you can tag whatever you want at any point18:45
devanandac/should we do a 4.0.0-beta as the first tag, then?18:45
dhellmannyou could do that. I'm not sure what it would buy you. What're your thoughts?18:46
devanandahm. or we end up doing, say, 4.1.0 a couple weeks from now18:47
devanandathat's probably also fine18:47
dhellmannoh, yeah, you shouldn't plan on tagging alphas or betas as a regular thing18:47
devanandaright18:47
devanandaso yea, let's just cut 4.0.0 and then I'll ask folks to stabilize it, and we'll do another release real soon18:48
devanandawill be good learning for the team to finally be able to release on demand18:49
dhellmannok, so, next question: 4.0 is right? I can't remember how many cycles ironic has been doing releases. It looks like 2014.1, 2014.2, 2015.1.0 exist so 4 should be right18:49
dhellmann++18:49
devanandayes, we've had 3 releases managed by openstack18:49
devanandaI, J, K18:50
dhellmanncool18:50
openstackgerritDevananda van der Veen proposed openstack/releases: Add Ironic 4.0.0  https://review.openstack.org/21430118:50
dhellmannok, so next I need to figure out whether anyone is going to be confused if ^^ has a different version in setup.cfg18:50
dhellmannpbr is not, as long as the git repository is there18:51
dhellmannpackaging tools that don't use the git repo might be18:51
devanandapbr borks for me if I try to pip install ^18:52
dhellmannso the question is, is it better to have those tools be confused or to land the patch to remove the version from setup.cfg (which would cause the version number to dip to a dev version of 2015.1.1) and then tag that commit as 4.0.0,18:52
lifelessdevananda: what error18:52
devanandadhellmann: I think we already proposed a patch to ironic for that18:52
lifelessdevananda: and had you removed the version= from setup.cfg ?18:52
dhellmanndevananda: possibly, but your setup.cfg still has a version in it18:53
lifelessdhellmann: what tools?18:53
dhellmannlifeless: I don't know, the deb and rpm guys always yell when things change with packaging and I don't know how those work but they seem fragile18:53
lifelessdhellmann: anything in the openstack ecosystem should cope fine since the clients have not had versions in their setup.cfgs for a very long time18:53
dhellmannso I'm worried that even if we tag $SHA as 4.0.0 if the setup.cfg in $SHA has version=x.y.z their tools will be confused18:54
devanandahttps://review.openstack.org/#/c/192404/118:54
dhellmannlifeless: this is ironic server18:54
devanandadhellmann: looks like you tried it a while ago. I'll update to 4.0.0, one sec18:54
lifelessdhellmann: yes, but the packaging tools are the same18:54
dhellmanndevananda: no, don't do that18:54
dhellmanndevananda: the right way to do it is to remove the line entirely, and I have another patch ready for that now18:54
devanandaahhh18:54
devanandaok18:54
dhellmanndevananda: I'm working out what order to do things, tag then land or land then tag18:55
lifelessdevananda: so I bet you get an error saying pbr can't cope,right ?18:55
dhellmannlifeless: I'm sure tools we own will work fine. I'm less sure that zigo is going to figure out that the version number of ironic is 4.0.0 instead of 2015.1.1.dev80+18:55
devanandalifeless: to that ^ patch, yes18:55
lifelessdevananda: so thats because the new version being targeted in the config is less than the most recent tag18:55
lifelessdevananda: its an impossible situation per versioning rules18:56
devanandalifeless: indeed. which is why that line is being removed, IIUC18:56
lifelessdevananda: you have to do the actual tag. The removal of the line makes maintenance easier18:56
lifelessdevananda: so if you do a signed tag locally of 4.0.0 and then either a) change version to 4.1.0 in the cfg or b) remove the line, then pbr should be happy18:57
devanandadhellmann: oh - is this done by signed git tags now (similar to the client release mechanism) ?18:57
dhellmanndevananda: yes, you'll be switching to post-versioning using tags18:57
devanandawhat is the purpose of the CR to openstack/releases then?18:57
dhellmanndevananda: it triggers the tagging, although we don't have the final step automated yet18:57
dhellmannI have a script that reads the file you're editing there18:58
dhellmanndevananda: at some point next cycle (I hope) approval will cause all of the rest of the process to happen, and this won't take up so much of our day-to-day :-)18:58
dhellmannso, lifeless, what do you think about ordering? If we tag, then remove the version in setup.cfg, the tagged version and its setup.cfg don't match. If we change setup.cfg first and tag the results, the version number will dip from 2015.2.0.dev563 to 2015.1.1.dev56419:00
dhellmannand then to 4.0.0 of course19:00
devanandadhellmann: "it triggers the tagging" -- you mean it lands the tag that I signed and proposed?19:01
devanandaI can see how this is confusing then19:01
dhellmanndevananda: yes, there will be a job to do the tagging based on the instructions you're giving in that file -- it will run more or less the same script I'm going to run for you shortly19:01
dhellmannand it will fix launchpad, and send email, and whatever else we need to do19:02
devanandanice19:02
devanandaoh yea, LP19:02
devanandawe have nearly stopped using it, btw19:02
dhellmannoh?19:02
dhellmannbugs go where?19:02
devanandafor features,19:02
dhellmannah19:02
devanandaI hit enter too fast :)19:02
devanandabugs are very much still in LP; features are in specs and a spreadsheet19:05
dhellmannok, that's fine (though I'm curious to hear about how that's working for you)19:05
devanandait saves time. but how effective is it? still to be seen19:05
dhellmannk19:05
devanandaI'm curious how it will affect the release process19:06
dhellmannwell, it's going to mean your release reporting pages don't talk about any features, for one19:06
devanandasince that is/was tied to LP series numbers, release milestones, and the autogeneration of change logs from BPs19:06
dhellmannso that's less than ideal19:06
devanandaright19:06
devanandawe still require specs be linked to a BP, though we don't do anything beyond that19:07
dhellmannoh, so you have the blueprints but their statuses are likely to be wrong?19:07
devanandaI could script updating LP based on completed specs ... or see if ttx' scripts could do what we need19:07
devanandadhellmann: yes19:07
dhellmannok19:07
* dhellmann thinks19:07
dhellmannthe release script will update the status of targeted bugs from "fix committed" to "fix released"19:08
dhellmannI think it depends on the blueprints being done by hand19:08
dhellmannmostly because for libraries we don't tend to have a lot of blueprints for any one lib, so it's not a big problem19:09
dhellmanndevananda: spec2bp.py might be close to what you want19:11
dhellmanndevananda: that or adjust_blueprints.py19:12
devanandahmmm yea19:12
lifelessdhellmann: tag + land updated setup.cfg IMO19:12
lifelessdhellmann: the very next patch to land will be the updated setup.cfg because everything else will error19:12
dhellmannlifeless: ok, so we're not worried about the mismatch for packagers not having the git repo where they're building a package?19:13
dhellmannlifeless: well, it won't, because 2015.2 is higher than 4.019:13
lifelessdhellmann: oh, right. Meh19:13
lifelessdhellmann: it really doesn't matter19:13
dhellmannok19:13
lifelessdhellmann: we've already signed on to 'we are breaking all the rules on versioning here'19:14
lifelesstrying to make it 'correct' is a bit of an oxymoron19:14
dhellmannlifeless: do do you know where the distros get a version number for something like oslo.config, if they don't package from a dir with the git repo?19:14
lifelessdhellmann: from the sdist19:14
lifelessdhellmann: pbr bundles that into it19:14
dhellmannyeah, I'm not trying for "correct" just "less broken19:15
dhellmannah, right, metadata19:15
dhellmannok, I think we're good to do it in that order, then19:15
dhellmannso now we just need to sort out the blueprints19:15
dhellmanndevananda: here's your ironic version fix: https://review.openstack.org/216388  -- that needs to be the very next ironic patch to land after we tag the repo19:16
devanandadhellmann: ok - so - process is: 1) I sign a tag on HEAD now. 2) I push the tag to gerrit 3) ???? 4) I approve that ^ patch 5) profit19:17
patchbotdevananda: https://review.openstack.org/#/c/5/19:17
dhellmanndevananda: no, you don't do any of those things, I do them.19:17
dhellmannthat's why I asked you to submit the patch to the releases repository :-)19:17
devanandaooh19:18
dhellmanndevananda: as a managed project, you will just submit patches there with instructions, we'll review them, and then the release will happen after the change is approved19:18
devanandaand after this is done, you will continue doing those things when I (or jroll) submits a patch to openstack/releases ?19:19
dhellmannthat's right19:19
devanandagotcha19:19
dhellmannuntil such a time as I can automate myself out of this particular job19:19
lifelessfirst rung on the automation ladder19:19
dhellmannright19:19
devanandainteresting. jroll and I both thought we would become responsible for doing that for the point releases19:19
dhellmannoh, no, just for submitting the request19:19
devanandaand you'd only do it for the integrated "Liberty" release19:19
dhellmannI mean, you could go release:independent, but I'm not sure you want to19:20
dhellmannI'm sorry, that was poor communication on our part19:20
devanandainteresting19:21
devanandathe client is also release:managed19:21
devanandathat is NOT what I thought19:22
devanandaalso, I know the release process for the client has been, up to now, thta I sign and push a tag to gerrit19:22
dhellmannyeah, that will also be done with a change to the releases repo19:22
dhellmannwe changed a lot of the libraries early in the cycle so we could get a handle on the use of semver19:22
dhellmanndevananda: so, next step is to figure out what to do with the blueprints in lp19:24
dhellmannI see a bunch in https://blueprints.launchpad.net/ironic/liberty19:24
dhellmannI wonder if any of those have accurate status?19:25
devanandadhellmann: so ecd007e1 looks like an error19:25
dhellmannoh?19:26
devanandadhellmann: in that commit to openstack/governance, you tagged a couple ironic projects as manged which are not19:26
devanandapython-ironicclient and python-ironic-inspector-client19:27
dhellmannwe want to manage those19:27
devanandaoh19:28
dhellmannthere was a whole thread about this months ago, when I changed the acls, did you miss that?19:28
devanandaapparently so19:28
devananda:(19:28
dhellmannok, sorry19:28
devanandaso all clients are now managed?19:28
dhellmannthe tl;dr is that folks were doing semver wrong and confusing everyone, so we pulled in the clients for the integrated release projects and some others, changed the acls, and instituted this releases repo19:28
dhellmannnot all, but most19:29
devanandaheh19:29
devananda++ for consistency19:29
dhellmannobviously we missed some ptls along the way19:29
dhellmannwe discussed it on the ML and in some cross-project meetings, so I thought silence was consent19:29
devanandaI have been, well, distracted .... my fault19:30
dhellmannyeah, no problem, my fault for not getting explicit acks from everyone19:30
devanandaanyway, knowing that, it's fine. I don't have any particular desire to do the tagging myself19:30
dhellmannyeah, it's the same process, with a patch to the releases repo19:30
devanandaif you want to do all the work, well, good for you :)19:30
dhellmanneventually we'll do everything that way and let the bots do the hard work19:30
dhellmannheh19:31
devanandastill need a human to review the incoming request19:31
dhellmannso far you haven't asked me to do any work on that :-)19:31
dhellmannyeah, but that'll probably be one +2a19:31
devanandacause I didnt know you were working on that :)19:31
dhellmann"does their semver make sense? ok"19:31
devanandaI was going to do a client release on my own soon, heh19:31
devanandaso we'll need to do a similar switch in the client, but there it won't break anything because we have been doing semver19:32
devanandaat least as far as i understand it19:32
devananda:)19:32
dhellmannyeah, the other thing I want to do is publish an unreleased log so we can find things we need to be releasing more often19:32
devananda++19:32
dhellmannthe client is already post-versioning, so you just need to submit the patch with the sha and version19:32
dhellmannthe rest is easy, once we've made the shift19:32
devanandacool19:32
dhellmannwell, unless you need to do the launchpad work, too19:33
devanandaok - anything else to get me up to speed?19:33
devanandaoh - we don't track the client on LP. never did. I have manually generated the release notes in the tag message19:33
devananda*track the releases of19:34
dhellmannso right now we need: 1) do not land any more ironic patches until we've tagged 2) clean up LP for ironic server 3) dhellmann  tag ironic 4) devananda land patch 21638819:34
patchbotdhellmann: https://review.openstack.org/#/c/216388/19:34
devanandawe, again, just use it for bugs19:34
dhellmannthat's new, I wonder where patchbot came from19:34
devanandaI've notified cores of (1). I'll take a look at (2) now19:34
dhellmannk19:34
devanandaoh interesting -- it looks like some automation has been keeping https://launchpad.net/python-ironicclient up to date with milestones19:35
dhellmannyes, that's part of what you get when we tag releases for you19:35
devanandaneat. I imagine the same will happen for ironic 4.0.0 then, once the switch is complete?19:37
dhellmannyes, that's right19:38
dhellmannthat's part of why we want to clean up the blueprints now :-)19:38
devananda:)19:38
dhellmannI'll make a next-liberty milestone, and you can put anything there that's been done in liberty19:39
dhellmannthat will be renamed to 4.0.0 when we cut the release19:39
devanandacool19:39
devanandaonce that's created, i'll move things that are currently targeted to l1, l2, l3 to that19:39
dhellmannok, that's done19:40
devanandaactually, simply deleting l1, l2, l3 should free those items up, yes?19:40
devananda"fix committed" bugs will get picked up when we close next-liberty19:40
dhellmannyes, that's right19:41
dhellmannit picks up all of them, whether they are in a milestone or not19:41
dhellmannbugs, but not blueprints19:41
devanandaright19:41
devanandaok - milestones deleted19:43
devanandalets see if I can figure out ttx' scripts again :)19:43
dhellmannthose tend to work best when there are a bunch of blueprints in a certain state19:44
dhellmannin this case, it might be simpler to just update them one by one19:44
dhellmannif you have a list, I can help19:44
devanandahttp://specs.openstack.org/openstack/ironic-specs/19:45
devanandawoops, better link19:45
devanandahttp://specs.openstack.org/openstack/ironic-specs/#implemented-in-liberty19:45
dhellmannthose 4?19:46
dhellmannI'll do the 2nd 219:46
devanandaalso, part of our "new" release process would be to go through those and make sure folks have updated their spec to "done" when the work is done19:46
devanandaI suspect a few of the "approved" are actually "implemented" right now19:46
dhellmannoops, I started with the first one19:46
devanandano worries, easy enough for me to do these19:47
* devananda goes through the rest19:48
dhellmannhttps://launchpad.net/ironic/+milestone/next-liberty looks good19:50
devanandagive me a bit to review the rest of these specs19:50
dhellmannsure19:50
dhellmanndevananda: I'm going to keep double verifying that you're ready for me to do steps before I do them, don't worry :-)19:51
openstackgerritDoug Hellmann proposed openstack-infra/release-tools: fix ms2version.py use of print  https://review.openstack.org/21571920:15
notmynamedevananda: patchbot is mine20:17
dhellmannnotmyname: nice work20:19
openstackgerritDavanum Srinivas (dims) proposed openstack/releases: pbr 1.5.1  https://review.openstack.org/21636520:27
openstackgerritDavanum Srinivas (dims) proposed openstack/releases: Add oslo.versionedobjects 0.8.0 deliverable  https://review.openstack.org/21266520:27
openstackgerritDavanum Srinivas (dims) proposed openstack/releases: Oslo Releases for Aug 24, 2015  https://review.openstack.org/21641520:27
dims__dhellmann: 3 reviews to cover oslo releases this week - https://etherpad.openstack.org/p/library-releases20:29
dhellmanndims__: ok, I'll take a look shortly20:31
devanandadhellmann: I think https://launchpad.net/ironic/+milestone/next-liberty is accurate now, as far as BPs go20:32
dhellmanndevananda: great, I think that means we're ready to tag your release!20:37
dhellmanndevananda: I'll approve the release request now if you agree20:37
devanandadhellmann: do it20:45
devanandaat least we have a few days to fix something if it breaks :)20:46
dhellmannright20:48
* dhellmann finishes his expense report while waiting for the patch to land20:48
openstackgerritMerged openstack/releases: Add Ironic 4.0.0  https://review.openstack.org/21430120:50
dhellmanndevananda: the tag has been pushed. Release note generation failed to parse some info out of your readme, so I'll submit a patch to fix that21:02
dhellmannit's updating bugs now, and that may take it a while21:02
dhellmanndevananda: you should +2a https://review.openstack.org/#/c/216388 in the mean time21:07
dhellmannthat will then free up the ironic team to continue working21:07
devanandaack21:07
dhellmanndevananda: launchpad is updated https://launchpad.net/ironic/+milestone/4.0.021:08
devanandaapproved21:08
dhellmannok, the release job is still running but from my end everything looks ok21:09
dhellmanndevananda: I'll work on the release notes script and send a release announcement tomorrow morning21:10
dhellmannttx: I assume we still want to send release announcements for post-versioned server projects that aren't published to pypi?21:10
devanandadhellmann: so in the new world, the openstack liberty/stable branch of ironic will be based on the most current released version at that point21:11
devanandadhellmann: yes?21:11
dhellmannthat's right21:11
devanandaperfect21:11
dhellmannwe won't create that branch yet, but it will be created from some tagged version we pick together21:11
devananda++21:11
dims__dhellmann: all the jobs for the 3 reviews are green. fyi21:13
dhellmanndims__: do you want me to approve, and you do the releasing?21:14
dims__dhellmann: sounds good21:14
dims__yes please21:14
*** dougwig has joined #openstack-relmgr-office21:14
dhellmanndims__: is it easier for you to merge them one at a time, or should I go ahead and +a them? (otherwise I can +2 and let you +a)21:14
* dougwig waves21:14
dhellmanndougwig: o/21:14
dims__dhellmann: please +2 and i can +A21:15
dhellmanndims__: ack21:15
dhellmannlifeless, dims__ : for pbr 1.5.1 I see "Expose a 'rpm_version' extra command" -- is that a feature?21:15
dhellmanndims__: for oslo.versionedobjects, did you work out the nova issues? https://review.openstack.org/#/c/212665/21:16
dims__dhellmann: yep, done21:16
dims__checking the pbr one21:16
lifelessdhellmann: bah humbug, ssssh :)21:18
dims__LOL21:18
* dhellmann smiles sweetly at lifeless 21:18
dhellmanndims__: +2 on those others, they look good21:19
dhellmannlifeless: if you have time for a quick review, https://review.openstack.org/216370 would be nice21:20
lifelessdhellmann: did you see my one on the i18n patch?21:20
dhellmannno?21:20
dhellmannlifeless: those are good points21:21
lifelessdhellmann: you were mid-ironic release discussion at the time21:21
dhellmannit came about 10 min after I approved, so we'll have to either revert or clean it up in another patch21:22
dhellmanndims__: you might want to hold off on the i18n release ^^21:22
dims__ack21:22
dhellmannlifeless: do you think it's worth reverting? I'm about to go start a fire to make dinner on :-/21:22
dhellmannif we fix it in the next day or two, we could treat the update as a bug fix21:22
dims__dhellmann: lifeless: holding off on i18n and pbr21:23
lifelessdims__: change the pbr thing to 1.6.0 ?21:23
lifelessdhellmann: update as bug fix would be fine I think21:24
lifelessdhellmann: your call really - you filed the bug :)21:24
dims__lifeless: +1 if you are ok with that21:24
dhellmannlifeless: heh, if you don't think this is going to be worse than where we were, I'm fine with continuing to iterate on it21:24
lifelessdhellmann: I don't think its worse21:25
lifelessdhellmann: I don't think its necessarily better - and since I don't know what prompted the report I'm not qualified to know :)21:25
lifelessdhellmann: so I'm just giving things that occur to me when I think about the code in front of me21:25
dhellmannoh, I think someone pointed out that since we don't test translated strings a bad one could cause the service to fall over in unexpected ways21:25
dhellmannyeah, it has been so long since I thought about that problem I'm in the same mode, I just wasn't as thorough21:26
*** gordc has quit IRC21:26
dims__so go for pbr as 1.6.0 - lifeless please update review21:26
dims__i18n, release what we have21:27
dims__ok dhellmann and lifeless21:27
dhellmanndims__: ++21:27
openstackgerritlifeless proposed openstack/releases: pbr 1.5.1  https://review.openstack.org/21636521:28
dims__lifeless: subject too :)21:28
openstackgerritlifeless proposed openstack/releases: pbr 1.6.0  https://review.openstack.org/21636521:28
openstackgerritMerged openstack/releases: Make list-changes smarter  https://review.openstack.org/21637021:28
openstackgerritDoug Hellmann proposed openstack-infra/release-tools: Fix formatting in process_bugs.py output  https://review.openstack.org/21643421:36
openstackgerritDavanum Srinivas (dims) proposed openstack/releases: pbr 1.6.0  https://review.openstack.org/21636521:44
openstackgerritMerged openstack/releases: Add oslo.versionedobjects 0.8.0 deliverable  https://review.openstack.org/21266521:45
openstackgerritDavanum Srinivas (dims) proposed openstack/releases: pbr 1.6.0  https://review.openstack.org/21636521:48
openstackgerritDavanum Srinivas (dims) proposed openstack/releases: Oslo Releases for Aug 24, 2015  https://review.openstack.org/21641521:48
openstackgerritMerged openstack/releases: pbr 1.6.0  https://review.openstack.org/21636521:52
*** bnemec has joined #openstack-relmgr-office22:20
*** dims_ has joined #openstack-relmgr-office23:16
*** dims__ has quit IRC23:18
*** dims__ has joined #openstack-relmgr-office23:19
*** dims_ has quit IRC23:21
*** morganfainberg is now known as morgan23:42

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!