Thursday, 2016-09-29

onovyzigo: ad .git suffix. git URL without .git suffix is not correct. Correct URL is with git. Without git it's working only because github "hacks" it and checks request type.05:23
onovyyou can check correct url by 'clone or download' button05:24
onovyworking on python-os-api-ref06:21
lennybzigo, is it normal to take more then 19h to be merged? Or something wrong/missing in the patch https://review.openstack.org/#/c/372527/  ?06:44
onovyzigo: how to create dfsg tags correctly?06:52
onovyi created one here: https://github.com/openstack/deb-python-os-api-ref/commit/71a597131bb93175c85b774cd1156c44343b89c506:53
onovybut can't merge them06:53
openstackgerritOndřej Nový proposed openstack/deb-swift: Merge tag '2.10.0' into debian/newton  https://review.openstack.org/37924307:50
openstackgerritMerged openstack/deb-swift: Merge tag '2.10.0' into debian/newton  https://review.openstack.org/37924308:09
*** strigazi_AFK is now known as strigazi08:29
zigocoreycb: ddellav: FYI, I'm done with absolutely everything but the Horizon plugins, though you may want to *not* sync these because they wont work with the Ubuntu Horizon package.08:30
zigolennyb: It's not normal that it takes that much time. I'm not sure what went wrong with infra.08:31
zigolennyb: I typed "recheck" to re-trigger the build, let's see if it goes through.08:33
zigoonovy: About adding the .git sufix, I don't mind so much, really ...08:33
zigoonovy: But I think it's a loss of time.08:33
onovymy time :) np.08:34
zigoonovy: Moving from github.com -> git.openstack.org is a way more useful though.08:34
zigoonovy: Also, if you want your swift-related project-config patch to go through, I'd suggest you to ping some people in #openstack-infra.08:34
onovyalready have +208:35
zigoonovy: I'd prefer if *you* were doing it, so that you get to know them better, and they know I'm not the only one working on packaging-deb.08:35
zigoBut I can ping guys for you if you ask me! :)08:35
zigoonovy: I'd suggest ajaeger, he's up early in the morning.08:35
zigoAnd he's nice.08:35
zigoWell, they all are ! :)08:35
onovyi know ajaeger already08:35
onovyand he already done +208:35
zigoVery early in the morning, you can ask ianw, he's in Australia. But then, your only hope is to wait the begining of the afternoon, as everyone else in in north america.08:36
onovyok08:38
zigofungi, pabelanger, and mordered (when he's in the channel) have all been super helpful so far.08:43
lennybzigo: in the worst case I can abandon and make a new one08:50
onovyzigo: what about os-api-ref?08:58
zigoonovy: What about it? :)09:00
onovy^^09:00
zigoWell, we need the latest version from upper-constraints.txt if possible.09:01
zigono?09:01
onovyehm09:01
onovy08:52:51           onovy | zigo: how to create dfsg tags correctly?09:01
onovy08:53:30           onovy | i created one here:https://github.com/openstack/deb-python-os-api-ref/commit/71a597131bb93175c85b774cd1156c44343b89c509:01
onovy08:53:35           onovy | but can't merge them09:01
zigoOh...09:01
zigoWell, I'm not sure about it, really.09:02
zigoI wrote a message to the -infra list, hang on.09:02
zigoYeah, I remember now.09:03
zigoThe issue is that we need a project-config patch to allow having a master-dfsg branch in the Gerrit repos which isn't building packages.09:03
zigoMy message didn't go through because I wasn't registered to the list.09:04
onovyyep, we need master-dfsg09:08
onovyso i will git merge into master-dfsg from tag, then remove dfsg, git review, git tag?09:08
onovy"remove dfsg" => "remove dfsg files"09:09
zigoonovy: Something like that indeed, though we need to be able to git review without building the package in that branch, which isn't currently possible without a patch to project-config.09:12
onovycool09:13
onovyso no new os-api-ref :]09:13
zigolennyb: The issue is that your patch was stacked on top of the .gitreview patch, which was abandonned because it was done in another patch.09:13
zigolennyb: Best way is to abandon your current patch and make a new one indeed.09:13
zigoonovy: Well, we got to find out the way to do it, because I need it for ryu as well (the upstream project has a debian folder we need to get rid of).09:14
zigoonovy: I registered to the openstack-infra@ list so I can send my message...09:14
onovyso let's fix master-dfsg branch in infra :)09:14
zigoYup ! :)09:14
zigoHopefully, we'll have an answer from them about how to do it.09:14
onovy+there is problem with py3, i can't build py3 version of os-api-ref09:15
onovythis patch https://review.openstack.org/#/c/379186/ will fix one of bug. but i can't figure out second09:15
zigoonovy: It's pretty bad that they don't have Py3 gating.09:16
onovyyep09:16
openstackgerritThomas Goirand proposed openstack/deb-horizon: Allow plugin to define SECRET_KEY before import  https://review.openstack.org/37931009:28
openstackgerritThomas Goirand proposed openstack/deb-horizon: Use /tmp for SECRET_KEY  https://review.openstack.org/37932509:47
openstackgerritMerged openstack/deb-horizon: Use /tmp for SECRET_KEY  https://review.openstack.org/37932510:20
onovyzigo: how we will do backports in infra? without branch, just tag?10:46
openstackgerritLenny Verkhovsky proposed openstack/deb-networking-mlnx: Service name fixed to networking-mlnx-eswitchd  https://review.openstack.org/37936810:58
openstackgerritThomas Goirand proposed openstack/deb-ironic-ui: Merge tag '2.0.0' into debian/newton  https://review.openstack.org/37869411:00
openstackgerritThomas Goirand proposed openstack/deb-manila-ui: Merge tag '2.5.0' into debian/newton  https://review.openstack.org/37871511:12
openstackgerritThomas Goirand proposed openstack/deb-trove-dashboard: Merge tag '7.0.0_rc2' into debian/newton  https://review.openstack.org/37875211:14
*** openstackgerrit has quit IRC11:19
*** openstackgerrit has joined #openstack-pkg11:19
openstackgerritThomas Goirand proposed openstack/deb-ironic-ui: Merge tag '2.0.0' into debian/newton  https://review.openstack.org/37869411:32
openstackgerritMerged openstack/deb-sahara-dashboard: Merge tag '5.0.0_rc2' into debian/newton  https://review.openstack.org/37901011:42
openstackgerritMerged openstack/deb-trove-dashboard: Merge tag '7.0.0_rc2' into debian/newton  https://review.openstack.org/37875211:43
openstackgerritMerged openstack/deb-ironic-ui: Merge tag '2.0.0' into debian/newton  https://review.openstack.org/37869411:47
openstackgerritOndřej Nový proposed openstack/deb-swauth: Mass changes  https://review.openstack.org/37939111:52
openstackgerritOndřej Nový proposed openstack/deb-swift-bench: Mass changes  https://review.openstack.org/37939311:52
openstackgerritOndřej Nový proposed openstack/deb-swift-plugin-s3: Mass changes  https://review.openstack.org/37939411:52
openstackgerritThomas Goirand proposed openstack/deb-senlin-dashboard: Merge tag '0.4.0' into debian/newton  https://review.openstack.org/37939511:52
openstackgerritOndřej Nový proposed openstack/deb-liberasurecode: Mass changes  https://review.openstack.org/37939611:52
openstackgerritOndřej Nový proposed openstack/deb-libisal: Mass changes  https://review.openstack.org/37939711:52
onovyzigo: i can set track-upstream now, right? https://review.openstack.org/#/c/379400/111:59
zigoonovy: If you've checked that these repositories already have the content from Alioth, then yes.11:59
onovyit have content11:59
zigoOtherwise you got to wait for manage-projects to run.11:59
zigoOk, good, go ahead then.12:00
onovycool12:00
onovyi added you as reviewer12:00
zigoThough someone may just approve the proposal bot to remove the upstream: fields, so it may merge conflict if that happens before, but you should at least try.,12:00
zigoonovy: https://git.openstack.org/openstack/liberasurecode <--- I'm surprised by this...12:01
onovyhttps://git.openstack.org/cgit/openstack/liberasurecode12:01
zigoProbably because of the ongoing justice case for liberasurecode ...12:01
onovyit's not liberasurecode12:01
onovyit's.... mmnt12:01
onovylibjerasure212:01
onovyyou mean this lib?12:01
onovycgit on os infra is not too smart12:02
onovyif you open "git" url, it will give you blank page12:02
onovyyou need to add /cgit/ to url12:02
openstackgerritMerged openstack/deb-senlin-dashboard: Merge tag '0.4.0' into debian/newton  https://review.openstack.org/37939512:19
openstackgerritThomas Goirand proposed openstack/deb-horizon: Releasing to experimental.  https://review.openstack.org/37944012:54
fungionovy: without cgit/ it's the git protocol interface (for cloning, et cetera). with cgit/ it's a browseable web interface13:47
onovyfungi: yep, i know13:47
onovybut there is simple hacks which allows to use git url for browsing13:47
onovyfor example github sends redirect if you open git url in browser13:48
fungiyeah, so far we've not been comfortable that it wouldn't require a lot of tweaking and tuning for minimal benefit13:48
onovyi don't say you should do it13:48
onovyi'm just saying: it's not too smart :)13:49
*** themill has quit IRC14:12
*** themill has joined #openstack-pkg14:15
openstackgerritThomas Goirand proposed openstack/deb-manila-ui: Merge tag '2.5.0' into debian/newton  https://review.openstack.org/37871514:53
zigoonovy: Infra's log disk is full, we shouldn't attempt to merge anything right now.15:09
onovyok15:17
*** openstackgerrit has quit IRC15:49
*** openstackgerrit has joined #openstack-pkg15:50
openstackgerritMerged openstack/deb-manila-ui: Merge tag '2.5.0' into debian/newton  https://review.openstack.org/37871518:40

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