Friday, 2018-07-13

openstackgerritDariusz Lisiak proposed openstack-infra/jenkins-job-builder master: Add support for filtering by name in git source  https://review.openstack.org/58245206:06
*** electrofelix has joined #openstack-jjb07:37
*** SotK has joined #openstack-jjb09:21
ssbarneaelectrofelix: zxiiro hi! are you here today?09:54
electrofelixssbarnea: I'm around, just been snowed under with work09:58
openstackgerritMerged openstack-infra/jenkins-job-builder master: adopt pre-commit hooks  https://review.openstack.org/57659810:08
*** abelur has quit IRC11:15
*** abelur has joined #openstack-jjb11:16
*** abelur has quit IRC11:16
*** abelur has joined #openstack-jjb11:17
*** yashagrawal3 has joined #openstack-jjb12:14
*** yashagrawal3 has quit IRC12:27
*** yashagrawal3 has joined #openstack-jjb13:24
*** yashagrawal3 has quit IRC13:51
*** tanhengyeow has joined #openstack-jjb14:12
tanhengyeowAnyone able to approve this patch? :) https://review.openstack.org/#/c/580839/14:12
zxiirossbarnea: electrofelix good morning, it's release day, any release items we want to get in for release?14:13
zxiirossbarnea: do you want to release python-jenkins today?14:13
zxiiroI think I'm going to take a new position with releases. I will bump the middle number, and we'll reserve the last number in case we need hotfixes14:14
ssbarneano personal need14:14
zxiirossbarnea: ok so I will release python-jenkins 1.1.0 today and jjb 2.1.0 today.14:14
zxiiroWe keep adding new features to releases so bumping the last number isn't semver.14:15
ssbarneaok, no problem with that approach. is a good idea for avoiding the need to create x.y.z.w for a hotfix.14:15
openstackgerritTan Heng Yeow proposed openstack-infra/jenkins-job-builder master: Refactor clone_workspace function in publishers.py to utilize convert_mapping_to_xml  https://review.openstack.org/58226914:17
electrofelixzxiiro: that sounds like a good idea, new features/plugins supported = middle number14:19
zxiirotanhengyeow: fyi I don't know if your mentor shared this with you but I'd highly recommend following this best practice https://chris.beams.io/posts/git-commit/14:34
zxiirotanhengyeow: it's a little long but it's worth reading once in awhile. I go over it myself every few months to make sure I remember :)14:35
tanhengyeowzxiiro: Thanks for the resource :) I briefly looked through it. Was there a bad habit I'm adopting in the recent patches I am submitting that can be improved?14:39
zxiirotanhengyeow: your summary lines are too long14:40
zxiirotanhengyeow: basically tools like github and git shortlog will trim the summary line if it's over 50 characters long.14:40
zxiirotanhengyeow: but in general I'd read the whole thing it's a really good resource. (I know folks don't like to read but... :)14:41
zxiirotanhengyeow: in general though your commit messages have been pretty good. I'm not complaining, just want to make sure you're learning :)14:41
tanhengyeowzxiiro: Ah right, I always end up typing alot more than necessary >< I'll keep that in mind and read this resource every now and then to let it sink :p14:41
tanhengyeowzxiiro: Np! I'm always open to constructive feedback so that I can improve :)14:42
zxiirotanhengyeow: I also run weekly office hours around this time for the community where they can join and ask me random questions on a Google Hangouts. You're welcome to jump in anytime if you have questions for me.14:47
zxiirotanhengyeow: it's every Friday starting from 1 hr ago and runs for 5 hrs. give or take, I do it very casually so cannot guarentee I'll always be available in it :)14:47
zxiirohttps://plus.google.com/hangouts/_/linuxfoundation.org/zxiiro14:47
zxiirossbarnea: can you give me admin access to the jjb google group?14:50
zxiirossbarnea: I think some of my emails are getting moderated14:51
zxiiroI don't see my last few release emails to the list available in archives hmm...14:51
openstackgerritMerged openstack-infra/jenkins-job-builder master: Refactor ivy function in triggers.py to utilize convert_mapping_to_xml  https://review.openstack.org/58083914:51
zxiirooh wait i do see them, they are just threaded for some reason14:52
zxiirohuh14:52
zxiiroalright nevermind :)14:52
zxiirotanhengyeow: this is the reason commit message summary's are so important. We use them for releases so if you have a bad summary line, you can't tell what is being added to a release from reading the summary line https://groups.google.com/d/msg/jenkins-job-builder/bomkpJBLHBM/P33W0JNPBwAJ14:53
tanhengyeowzxiiro: I see, thanks for the heads up :)14:55
tanhengyeowzxiiro: I don't think I have any burning questions to ask or clarify but is the weekly hangouts link available somewhere if I need to access it someday?14:56
zxiirotanhengyeow: I can add you to the calendar invite if you'd like14:56
openstackgerritTan Heng Yeow proposed openstack-infra/jenkins-job-builder master: Refactor clone_workspace function to use convert_mapping_to_xml  https://review.openstack.org/58226914:56
zxiirotanhengyeow: otherwise just know that you can jump on, on Fridays14:56
tanhengyeowAlright, I'll probably look out for it on Fridays, unless it becomes a frequent thing then I'll hope to get the calendar invite :)14:58
zxiirono worries, it's purely community service I'm doing here. hah14:59
tanhengyeowAppreciate that :))14:59
zxiiroelectrofelix: ssbarnea there seems to be someone squatting the jenkins-python name in pypi https://github.com/armata/jenkins-python15:40
zxiirothe code doesn't even have anythign to do with jenkins as far as i can tell15:40
zxiirooh the stackoverflow link seems to say it's a jenkins hash imlpementation in python15:41
zxiiroI wonder if we should contact them to rename their project more appropriately.15:41
ssbarneai think this was done on purpose15:44
ssbarneahave a look at https://github.com/lgastako/jenkins/issues15:45
ssbarneathis is about "jenkins" package on pypi, i asked to get it but didn't get an answer.15:46
ssbarneafeel free to comment on it, he is clearly active on github but not on that ticket.15:47
ssbarneai could try to ask pypi to takeover the project without him but I am not sure is the best approach.15:48
ssbarneai had success in the past doing this with "jira" package and now the project has a nice and simple package name.15:48
ssbarneathus is why i made myself a rule to always assure that module name = package name and to get hold of it as soon as possible, to avoid more or less "accidents".15:49
zxiirooh wow another one15:58
zxiirook yeah this is confusing to users15:58
ssbarneazxiiro: i am going to raise a takeover request for both now, just to avoid accidents. I will ask to have them assigned to openstack foundation account not personal, right?16:00
zxiirossbarnea: I don't think openstack foundation will care about it. Just ask for personal and you can add me to it too.16:00
ssbarneayour github account is the same as username here right?16:01
zxiirossbarnea: yes16:04
ssbarneazxiiro: https://github.com/pypa/warehouse/issues/429416:16
ssbarneawe may need some support on this.16:16
zxiirossbarnea: thanks16:17
zxiiroI subscribed I'll leave a comment later. busy on some work stuff right now.16:18
ssbarneai liked issues on trackers of both culprit packages.16:21
ssbarneai read a bit more that was not made on purpose, there is a real "jenkins hash" thing.16:21
*** yashagrawal3 has joined #openstack-jjb16:28
*** electrofelix has quit IRC16:57
*** tanhengyeow has quit IRC18:12
*** yashagrawal3_ has joined #openstack-jjb19:13
*** yashagrawal3_ has quit IRC19:40
*** yashagrawal3 has quit IRC19:40
*** yashagrawal3_ has joined #openstack-jjb19:40
*** yashagrawal3 has joined #openstack-jjb19:40
*** yashagrawal3_ has quit IRC22:21
*** yashagrawal3 has quit IRC22:21

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