Wednesday, 2017-08-02

* tonyb fails 00:00
tonybtl;dr: +2 on oslo.policy, questions on barbicanclient00:09
dimstonyb : cool, let me ship oslo.policy00:11
tonybclarkb: The UI in polygerrit looked worrying to me but everythgign I need it theer just moved ;P  review-dev LGTM ;P00:12
* tonyb really wants 2.14 for other reasons so wont do anythign to slow that down ;P00:12
tonybdims: cool.00:12
clarkbtonyb: its liek the old change screen which I much prefer :)00:16
openstackgerritMerged openstack/releases master: Release oslo.policy 1.25.1
tonybclarkb: I don't recall the old one but I'm used to what we have now.  for me the information I need it visible and the keybindinds still work so I'm good00:17
*** rajathagasthya has quit IRC00:20
*** thorst_afk has joined #openstack-release00:40
*** thorst_afk has quit IRC00:43
*** thorst_afk has joined #openstack-release00:43
*** rajathagasthya has joined #openstack-release00:43
*** thorst_afk has quit IRC00:47
*** hongbin has joined #openstack-release01:01
*** bobh has joined #openstack-release01:05
*** yamahata has quit IRC01:20
*** liujiong has joined #openstack-release01:23
*** mriedem has quit IRC01:30
*** https_GK1wmSU has joined #openstack-release01:32
*** https_GK1wmSU has left #openstack-release01:34
*** thorst_afk has joined #openstack-release01:44
*** rajathagasthya has quit IRC01:55
*** thorst_afk has quit IRC02:00
*** bobh has quit IRC02:23
*** yamamoto has quit IRC02:24
*** yamamoto has joined #openstack-release02:30
*** yamamoto has quit IRC02:35
*** yamamoto has joined #openstack-release02:36
*** dave-mcc_ has joined #openstack-release02:40
*** yamamoto has quit IRC02:41
*** dave-mccowan has quit IRC02:42
*** yamamoto has joined #openstack-release02:45
*** bobh has joined #openstack-release03:04
*** dave-mccowan has joined #openstack-release03:12
*** dave-mcc_ has quit IRC03:14
*** bobh has quit IRC03:18
*** gtrxcb has joined #openstack-release03:21
*** rajathagasthya has joined #openstack-release03:22
*** thorst_afk has joined #openstack-release03:23
*** thorst_afk has quit IRC03:27
*** hongbin has quit IRC03:30
*** yamamoto has quit IRC03:33
*** dave-mccowan has quit IRC03:33
*** udesale has joined #openstack-release03:33
openstackgerritSteve Baker proposed openstack/releases master: Release paunch 1.5.0
*** yamamoto has joined #openstack-release03:49
*** rajathagasthya has quit IRC04:03
*** yamamoto has quit IRC04:05
*** yamamoto has joined #openstack-release04:10
*** udesale__ has joined #openstack-release04:29
*** kornicameister has quit IRC04:31
*** udesale has quit IRC04:31
*** kornicameister has joined #openstack-release04:32
*** yamamoto has quit IRC04:50
*** yamamoto has joined #openstack-release05:07
*** jtomasek has joined #openstack-release05:16
*** thorst_afk has joined #openstack-release05:23
*** thorst_afk has quit IRC05:28
*** pcaruana has joined #openstack-release06:34
*** thorst_afk has joined #openstack-release07:24
*** thorst_afk has quit IRC07:29
*** udesale has joined #openstack-release07:33
*** electrofelix has joined #openstack-release07:35
*** udesale__ has quit IRC07:35
*** amoralej|off is now known as amoralej07:44
*** gtrxcb has quit IRC07:44
*** gcb_ has quit IRC07:48
*** alexchadin has joined #openstack-release07:53
*** pbandark has joined #openstack-release08:00
*** udesale__ has joined #openstack-release08:20
*** udesale has quit IRC08:22
*** https_GK1wmSU has joined #openstack-release08:24
*** https_GK1wmSU has left #openstack-release08:25
openstackgerritMerged openstack/releases master: do not classify tempest as a library
openstackgerritMerged openstack/releases master: Release Tricircle 3.4.0
*** lucas-afk is now known as lucasagomes08:32
openstackgerritMerged openstack/releases master: Release paunch 1.5.0
openstackgerritMerged openstack/releases master: make openstack_requirements cycle-with-intermediary
*** openstackgerrit has quit IRC08:33
*** openstackgerrit has joined #openstack-release08:40
openstackgerritMerged openstack/releases master: teach edit-deliverable to add stable branch data
openstackgerritMerged openstack/releases master: add deliverable argument to
openstackgerritMerged openstack/releases master: update to use edit-deliverable
*** wolverineav has joined #openstack-release08:42
*** wolverineav has quit IRC08:42
*** wolverineav has joined #openstack-release08:43
openstackgerritMerged openstack/releases master: Release tooz 1.48.2 for Ocata
openstackgerritMerged openstack/releases master: Release OpenStack-Ansible Newton/14.2.7
*** dtantsur|afk is now known as dtantsur09:17
*** thorst_afk has joined #openstack-release09:25
*** gcb has joined #openstack-release09:28
*** thorst_afk has quit IRC09:30
*** sdague has joined #openstack-release09:32
*** slashme has joined #openstack-release09:52
*** liujiong has quit IRC10:16
*** alexchadin has quit IRC10:43
*** zhurong has quit IRC10:43
amotokidhellmann: is there a plan for a new reno release? I would like to see ingnore-notes feature as horizon has a similar situation on nova.10:58
openstackgerritAkihiro Motoki proposed openstack/releases master: Add horizon release note link for Pike
dimsdhellmann : ttx : smcginnis : o/11:04
*** udesale__ has quit IRC11:09
openstackgerritMerged openstack/releases master: update the release process steps related to documentation
*** thorst_afk has joined #openstack-release11:21
openstackgerritMerged openstack/releases master: update new-release to let it add b3 tags
openstackgerritMerged openstack/releases master: Release OpenStack-Ansible Ocata/15.1.7
*** pbandark has quit IRC11:31
*** pbandark has joined #openstack-release11:34
*** lucasagomes is now known as lucas-hungry11:37
ttxdims: o/11:46
dhellmannamotoki : let me look, I thought that was released12:05
dhellmannttx, dims: o/ -- did you mean to both +2 that patch series without approving it?12:05
ttxdhellmann: no, I was first12:06
dimsdhellmann : wanted to check if we should +A or not. was not sure12:06
dimsbeing conservative :)12:06
dhellmannI think the changes are safe. they're in our commands for wrapping up the release, so it will be easy enough to deal with if they aren't12:07
dhellmannshouldn't affect validation or actual releases12:07
dimscool. +A'ing this series
openstackgerritDoug Hellmann proposed openstack/releases master: reno 2.5.0
dhellmanndims : thanks12:09
amotokidhellmann: thanks for reno12:11
openstackgerritMerged openstack/releases master: fix typos in yamlutils
openstackgerritMerged openstack/releases master: do not let yamlutils emit alias references
openstackgerritMerged openstack/releases master: have propose-final-releases use yamlutils instead of appending to the file
openstackgerritAndreas Jaeger proposed openstack/releases master: Release openstack-doc-tools 1.6.0
openstackgerritMerged openstack/releases master: remove unused data from new-release
openstackgerritMerged openstack/releases master: use yamlutils.loads() everywhere instead of yaml.safe_load()
*** alexchadin has joined #openstack-release12:15
*** amoralej is now known as amoralej|lunch12:18
*** amoralej|lunch is now known as amoralej12:29
dhellmanndims , ttx, smcginnis : I don't know if we need a FFE thread for the reno release ^^ -- it's "independent" but does show up in constraints12:31
ttxdhellmann: would be good to discuss why it's needed now, given its potential for disruption12:32
ttxhappy to have the discussion here, or on a ML thread12:33
dhellmannthe new "ignore" feature is needed to fix up some release note builds where folks modifed stable branch notes on master12:34
dhellmannI don't think it's critical, but the current notes give an incorrect impression of the content of the release12:34
dhellmannI'll go ahead and start a ML thread, since that's what we've asked others to do12:35
dims++ dhellmann12:38
*** lucas-hungry is now known as lucasagomes12:39
*** sdague_ has joined #openstack-release12:43
*** amoralej is now known as amoralej|lunch12:47
dhellmannfungi, dims , ttx, smcginnis : we have a small issue with the validation logic in the releases repo. The check it does to see if a patch has merged looks at cgit, which now (maybe always?) knows about the commit even when it has not merged. I think the issue only affects independent projects, since they don't do the ancestry check to verify that the patch has merged13:01
dhellmannfor example is from the review which has not been approved13:01
dhellmannI don't think this is a pants-on-fire situation, but be careful with reviews of independent releases until we sort it out13:02
*** yamamoto has quit IRC13:02
*** bobh has joined #openstack-release13:14
dimsack dhellmann13:17
dimsdhellmann : we need to check SHA by hand?13:18
*** yamamoto has joined #openstack-release13:18
dhellmanndims : yeah, until I can figure out a way to do the check more properly13:28
dhellmanndims : and only for independent releases, so it shouldn't be many13:28
dhellmannit's possible the "relationship to head" section of the list-changes job will be useful there13:29
dimsack dhellmann, i typically use the github compare to do that13:32
dhellmannthis is likely just a bad assumption on my part that cgit would only know about merged commits13:33
*** amoralej|lunch is now known as amoralej13:37
fungiright, all commits in gerrit get replicated to mirrors (cgit, github) even if they're still under review13:38
fungiyou can use that to your advantage when you want to use a git.o.o url in a script but need to test it against a proposed change13:39
fungithe patch sets in review are named refs, like "refs/changes/43/488543/4"13:40
openstackgerritDoug Hellmann proposed openstack/releases master: update exists check to look for a commit to be on a named branch
fungiso you can refer to them that way on the git servers and in gh too13:41
*** mriedem has joined #openstack-release13:41
dhellmannI think ^^ should fix it. It checks that there is a named branch containing the commit.13:42
openstackgerritDoug Hellmann proposed openstack/releases master: update exists check to look for a commit to be on a named branch
*** sdague has quit IRC13:43
*** alexchadin has quit IRC13:45
dhellmanndims, ttx, smcginnis, tonyb : I'm going to go ahead with the openstack-doc-tools release, since we think it's low impact and not part of projects that sync requirements13:47
dhellmanntonyb : to your point, maybe we can remove the tool from the g-r list? I don't know if it was added recently as part of the migration work or if it's a hold-over from when the API builds were syncing requirements13:47
dhellmanndims, ttx, smcginnis: I'll leave the reno release for one of you to approve now that tonyb has +2ed it13:48
* dhellmann drops offline for a few errands13:48
dimson it dhellmann13:49
openstackgerritMerged openstack/releases master: Release openstack-doc-tools 1.6.0
openstackgerritMerged openstack/releases master: reno 2.5.0
*** yamamoto has quit IRC13:55
*** sdague has joined #openstack-release13:56
*** zhurong has joined #openstack-release13:57
-openstackstatus- NOTICE: We have disable infracloud-vanilla due to the compute host running mirror.regionone.infracloud-vanilla.o.o being offline. Please recheck your failed jobs to schedule them to another cloud.13:58
ttxdhellmann: you good with me approving (late python-solumclient) ?14:02
ttxwas wondering why it did not have stable/pike yet14:02
dimsttx : definitely low risk14:03
ttxyes, let's just wait for confirmation from dhellmann, was wondering if it did not have stable/pike yet for a reason :)14:04
*** udesale__ has joined #openstack-release14:06
*** hongbin has joined #openstack-release14:10
*** mriedem1 has joined #openstack-release14:14
* smcginnis is coming in late today14:16
*** mriedem has quit IRC14:16
mtreinishdhellmann: is this a reno bug in the reno release?
smcginnisIt's for emphasis. :D14:18
smcginnisI've seen that with incorrectly formatted release notes before.14:19
*** mriedem1 is now known as mriedem14:20
smcginnisIronically it's from the release note named "flexible-formatting".14:20
smcginnisI believe that's missing a | or >14:20
smcginnisOh, it's missing the leading -14:24
openstackgerritSean McGinnis proposed openstack/reno master: Fix formatting for "Flexible Formatting" release note
smcginnismtreinish: I think that should fix it. ^14:26
amoralejhi, i've seen that in openstackdocstheme commits have not been included in stable/pike although they were merged on 17jul14:30
amoraleji guess it's because they didn't do a release14:30
amoralejis that an issue from releases projects perspective?14:30
*** bobh has quit IRC14:32
smcginnisamoralej: The last commit for python-cloudkittyclient in the stable/pike release was from way back in March.14:32
amoralejyeah, there are 4 more commits in master from july14:33
amoralejwith the docs update14:33
smcginnisamoralej: Looks like those will need to be backported to stable/pike.14:33
amoralejand, can they create a new release before GA?14:34
*** udesale has joined #openstack-release14:34
amoralejit's frozen for clients, right?14:34
smcginnisamoralej: Correct, we are past the client lib freeze.14:34
ttxneeds to be worth the risk/hassle14:34
ttxso we need to discuss the exception on a public forum14:34
ttxlike the benefit of doing it must outweigh the drawbacls14:35
smcginnisamoralej: Since these are just docs changes, I think you're probably OK just waiting until after the freeze is over, then do a stable release.14:35
*** udesale__ has quit IRC14:35
amoraleji was thinking if that could be an issue for publishing docs on GA14:35
ttx"what can wait should wait" is the general principle we apply14:35
amoraleji can wait, no problem14:35
smcginnisamoralej: There will be a brief window where unfortunately there will be missing docs, but that can be fixed the day after GA.14:36
amoralejok, i'll make cloudkitty team aware of it, thanks!14:36
*** zhurong has quit IRC14:38
openstackgerritMerged openstack/releases master: Release python-monascaclient 1.5.1
*** bauzas has quit IRC14:50
*** armax_ has joined #openstack-release14:53
*** armax has quit IRC14:53
*** armax_ is now known as armax14:53
*** yamamoto has joined #openstack-release14:55
*** yamamoto has quit IRC15:00
*** thorst_afk has quit IRC15:14
*** armax has quit IRC15:19
*** electrofelix has quit IRC15:23
*** armax has joined #openstack-release15:28
*** armax has quit IRC15:33
*** udesale has quit IRC15:34
*** sdague_ has quit IRC15:43
dhellmannttx: solum client was missed for branching because the type was wrong15:47
dhellmannthat patch fixes it15:47
dhellmannI can +2a if you approve15:48
dhellmannwhich it seems you do?15:48
dhellmannwell, there's a bad version #, ,so -1 for now, but I will approve when that's fixed15:48
*** bauzas has joined #openstack-release15:55
dhellmannsmcginnis : I think the thing you found with that release note is a bug in the text formatter, not in the content of the note15:57
dhellmannthe html output looks fine15:57
smcginnisdhellmann: Ah, OK. Should I abandon that one?15:58
dhellmannsmcginnis : yeah. let me know if you want to take a stab at fixing the real issue and I can try to get you started15:59
smcginnisdhellmann: Sure, I'll see if I can get caught up on some things today.15:59
*** sdague has quit IRC16:03
*** Rockyg has joined #openstack-release16:09
*** pcaruana has quit IRC16:14
*** armax has joined #openstack-release16:18
*** pbandark has quit IRC16:19
*** lucasagomes is now known as lucas-afk16:28
*** sdague has joined #openstack-release16:40
*** thorst_afk has joined #openstack-release16:55
*** rajathagasthya has joined #openstack-release16:58
*** dtantsur is now known as dtantsur|afk17:00
openstackgerritRico Lin proposed openstack/releases master: release heat `8.0.3` for ocata
openstackgerritRico Lin proposed openstack/releases master: release heat `7.0.5` for newton
*** harlowja has joined #openstack-release17:13
*** ricolin has quit IRC17:17
dhellmannttx, dims, smcginnis : we should plan to spend some time talking with the infra team at the ptg about what the zuul v3 rollout means for our jobs using scripts baked into the images:
dhellmannmaybe one of you already has17:26
dimsdhellmann : no i have not17:26
dimsdhellmann : sounds like a good idea. but would it be too late?17:26
dhellmanngood point, maybe we should have a chat before then17:27
dimssince they want to throw the switch "First thing Monday morning of the PTG"17:27
dhellmannI mean, it sounds like they have a plan for converting the existing jobs, and I interpreted that as "no down time" but perhaps I shouldn't assume that17:28
clarkbdhellmann: dims I think it will in many cases be run this shell script with some zuulified ansible shell task17:29
clarkbwhich should still have access to scripts on disk. The larges changes will likely be around secrets management17:29
dhellmannclarkb : will the shell script need to move to a new home? or will it stay in project-config?17:29
smcginnisdhellmann, dims: I agree, we should probably discuss ahead of time to see if there is any impact.17:30
* dhellmann envisions moving everything back into the release-tools repo :-)17:30
*** rajathagasthya has quit IRC17:30
clarkbMy understanding is to start we will likely continue to vendor a lot of stuff but then you'll be able to move it into your own repos oever time (this way we avoid needing to update 1k repos all at once)17:30
clarkbmordred is the best person to answer that though17:31
smcginnisclarkb: So as far as you know, no immediate action on anything should be required?17:33
clarkbthat is my understanding. Plan is to do the initial migration for people. Which should cover the vast majority of cases. Then sort out the corner cases that we didn't think about as they come up17:34
clarkbthen you'll be able to on your own time update things to be more on the project side instead of the infra side17:34
smcginnisclarkb: Great, thanks!17:35
*** rajathagasthya has joined #openstack-release17:48
*** yamahata has joined #openstack-release17:57
*** ekcs has joined #openstack-release18:01
*** amoralej is now known as amoralej|off18:08
tellesnobregadhellmann, we merged but it is on master, does this need to be backported once stable/pike is cut?18:10
mordredsmcginnis, clarkb: reading scrollback18:11
mordreddhellmann, dims, smcginnis: yes - no immediate action - things should continue to work as they do now - the jenkins/scripts will still be in place and for jobs that havne't been converted in any way they'll just look like the auto-converted playbooks do today18:12
mordreddhellmann, smcginnis: eventual goal being that we move the content that's currently in bash scripts baked in to the images themselves into ansible playbooks so that updates don't need image rebuilds18:13
dhellmanntellesnobrega : have you already cut a stable/pike branch?18:14
dhellmanntellesnobrega : oh, nevermind, that's the thing to publish the notes18:14
dhellmannno, you don't need to backport that. we always publish release notes from master.18:14
mordred*some* of those will still need to live in project-config - if they are things that need to be careful curated for security reasons - but needing to be in project-config should be the exception18:14
dhellmannreno does the work of scanning the branches18:14
tellesnobregadhellmann, ok, thanks18:14
tellesnobregadhellmann, about stable/pike, should I cut a branch for all sahara projects at this moment or just ad R+0 week?18:16
dhellmannmordred, smcginnis: security was one of the reasons given for having the release tagging scripts in project-config, so we'll probably want to review the risk there and decide if that's still the case18:16
dhellmanntellesnobrega : libraries should be branched now, other projects should be branched around the RC1 week (which is not R+0)18:16
tellesnobregadhellmann, awesome. thanks18:17
dhellmanntellesnobrega : are you following ttx's "countdown" emails on the -dev list?18:17
tellesnobregajust got confused, probably misread some stuff18:17
tellesnobregathanks again18:17
dhellmannok. He'll be including instructions for what to do each week between now and release, so just follow what he says to do. And of course, do ask questions if you have them. :-)18:17
mordreddhellmann: yah - we actually just had a conf-call with fungi on tarball signing - which is in a similar bucket - there are definitely portions of those flows that will need to stay in project-config18:18
tellesnobregadhellmann, will do18:18
dhellmanntellesnobrega : ++18:18
*** kfox1111_ is now known as kfox111118:19
dhellmannmordred : yeah, I think we're one of the edge cases that's going to take some special thought (not much, maybe). as long as leaving things alone will allow them to continue to work until we actually want to move something, I think we're good for now and we can work out the details as we go.18:19
*** bobh has joined #openstack-release18:20
mordredyes exactly - I mostly want to make sure there are plenty of similar examples you can draw from, but I'd rather you be involved in making changes to the release scripts18:21
*** Rockyg has quit IRC18:25
openstackgerritDoug Hellmann proposed openstack/releases master: update exists check to look for a commit to be on a named branch
prometheanfiredhellmann: you are seeking just a UC bump for reno, correct?18:53
dhellmannprometheanfire : yeah18:53
dhellmannhmm, well18:53
dhellmannprometheanfire : projects using that new option will want to update the min version18:53
dhellmannso I guess I need to update that, too18:54
prometheanfiredhellmann: did you see how many projects that'd impact?18:54
dhellmannI expect all of them18:54
prometheanfireya :|18:54
dhellmannI guess we could wing it, since it's just for publishing docs, and use the u-c list, but that feels incomplete18:55
dhellmannlet me go ahead and propose raising the minimum, and we can decide if we want to wait to land that18:55
prometheanfireit is incomplete, but the cost of updating GR here is high18:55
dhellmannyeah, I should have been more on the ball and released this a couple of weeks ago18:57
dhellmannI think I'm ok with leaving the min where it is, since I don't expect most distributors package release notes18:58
*** coolsvap has quit IRC19:01
dimsdhellmann : mordred : so no-op for now then we'll see after switch over?19:07
dhellmanndims : sounds like it19:08
dims++ dhellmann19:10
mordreddims: yah - no action needed on your part currently19:15
dimsthanks mordred19:16
*** jtomasek has quit IRC19:19
*** rajathagasthya has quit IRC19:35
*** rajathagasthya has joined #openstack-release19:38
*** wolverineav has quit IRC19:39
*** wolverineav has joined #openstack-release19:40
*** wolverineav has quit IRC19:44
*** sileht has quit IRC19:55
*** rajathagasthya has quit IRC20:00
*** rajathagasthya has joined #openstack-release20:02
*** bobh has quit IRC20:27
*** https_GK1wmSU has joined #openstack-release20:36
*** https_GK1wmSU has left #openstack-release20:37
prometheanfiredhellmann: we actually package release notes (behind the docs use flag), but I'll make sure 2.5.0 is in tree and stable with the rest of openstack20:46
*** jrist has quit IRC20:54
*** thorst_afk has quit IRC21:06
*** yamamoto_ has joined #openstack-release21:08
*** thorst_afk has joined #openstack-release21:11
*** thorst_afk has quit IRC21:15
*** yamamoto_ has quit IRC21:15
*** yamamoto_ has joined #openstack-release21:17
*** jrist has joined #openstack-release21:26
tonybdhellmann: True we could probably remove it.  I'll look at that today.  It would be a good solution all round :)21:32
dims++ tonyb21:34
*** thorst_afk has joined #openstack-release21:46
*** thorst_afk has quit IRC21:50
*** armax has quit IRC22:01
*** https_GK1wmSU has joined #openstack-release22:19
*** https_GK1wmSU has left #openstack-release22:21
*** yamamoto_ has quit IRC22:29
*** yamamoto has joined #openstack-release22:32
*** efried has quit IRC22:32
*** yamamoto has quit IRC22:36
*** yamamoto has joined #openstack-release22:41
*** yamamoto has quit IRC22:44
*** sdague has quit IRC22:50
*** jrist has quit IRC22:55
*** yamamoto has joined #openstack-release22:56
*** gtrxcb has joined #openstack-release23:06
*** yamamoto has quit IRC23:11
*** yamamoto has joined #openstack-release23:14
*** hongbin has quit IRC23:21
*** rajathagasthya has quit IRC23:29
*** rajathagasthya has joined #openstack-release23:54
*** thorst_afk has joined #openstack-release23:55

Generated by 2.15.3 by Marius Gedminas - find it at!