| prometheanfire | tonyb: swift is really the only one I might wait on, I'll jude depend on it if that's alright | 00:16 |
|---|---|---|
| tonyb | prometheanfire: Sure works for me | 00:17 |
| tonyb | prometheanfire: then I *think* we can make the grenade/gate patches depeing on yours but I'm not 100% there | 00:17 |
| * prometheanfire shrugs | 00:18 | |
| prometheanfire | not to big of a deal to rebase | 00:18 |
| tonyb | sure | 00:18 |
| openstackgerrit | Matthew Thode proposed openstack/releases master: branch openstack/requirements - stable/stein https://review.openstack.org/647255 | 00:19 |
| *** armax has joined #openstack-release | 00:36 | |
| *** ekcs has quit IRC | 00:48 | |
| *** hongbin has joined #openstack-release | 01:05 | |
| openstackgerrit | Tony Breeds proposed openstack/releases master: Remove per release setting of tarball-base https://review.openstack.org/647258 | 01:07 |
| openstackgerrit | Tony Breeds proposed openstack/releases master: release 1.4.1 of release-test https://review.openstack.org/647259 | 01:07 |
| openstackgerrit | Merged openstack/releases master: Release stable branches for heat repos https://review.openstack.org/644486 | 01:08 |
| *** armax has quit IRC | 01:54 | |
| openstackgerrit | Tony Breeds proposed openstack/releases master: Remove per release setting of tarball-base https://review.openstack.org/647258 | 02:07 |
| *** ricolin has joined #openstack-release | 02:26 | |
| *** whoami-rajat has joined #openstack-release | 02:43 | |
| *** binh_ has quit IRC | 02:51 | |
| *** binh_ has joined #openstack-release | 02:52 | |
| *** udesale has joined #openstack-release | 03:16 | |
| *** hongbin_ has joined #openstack-release | 03:21 | |
| *** hongbin has quit IRC | 03:23 | |
| *** elbragstad has joined #openstack-release | 03:25 | |
| openstackgerrit | Merged openstack/releases master: Manila stable/queens 6.2.0 release https://review.openstack.org/645149 | 03:58 |
| *** hongbin_ has quit IRC | 03:59 | |
| *** ekcs has joined #openstack-release | 04:01 | |
| *** binh_ has quit IRC | 04:26 | |
| *** binh_ has joined #openstack-release | 04:26 | |
| *** ykarel has joined #openstack-release | 05:06 | |
| *** ykarel has quit IRC | 05:20 | |
| *** ykarel has joined #openstack-release | 05:21 | |
| *** ykarel has quit IRC | 05:24 | |
| *** ykarel has joined #openstack-release | 05:24 | |
| *** ykarel has quit IRC | 05:27 | |
| *** ykarel has joined #openstack-release | 05:28 | |
| *** ykarel has quit IRC | 05:30 | |
| *** ykarel has joined #openstack-release | 05:30 | |
| openstackgerrit | Matthew Thode proposed openstack/releases master: branch openstack/requirements - stable/stein https://review.openstack.org/647255 | 05:53 |
| *** ykarel has quit IRC | 05:53 | |
| *** jaosorior has joined #openstack-release | 06:02 | |
| *** ykarel has joined #openstack-release | 06:06 | |
| *** udesale has quit IRC | 06:15 | |
| *** udesale has joined #openstack-release | 06:16 | |
| *** udesale has quit IRC | 06:20 | |
| *** udesale has joined #openstack-release | 06:20 | |
| *** ykarel has quit IRC | 06:21 | |
| *** lennyb has quit IRC | 06:24 | |
| *** ykarel has joined #openstack-release | 06:27 | |
| *** binh_ has quit IRC | 06:36 | |
| *** binh_ has joined #openstack-release | 06:36 | |
| *** ykarel_ has joined #openstack-release | 06:42 | |
| *** ykarel has quit IRC | 06:42 | |
| *** ykarel_ is now known as ykarel | 06:43 | |
| *** elbragstad has quit IRC | 06:45 | |
| *** ykarel is now known as ykarel|meetup | 06:53 | |
| *** ykarel|meetup has quit IRC | 06:57 | |
| *** pcaruana has joined #openstack-release | 07:07 | |
| *** ykarel|meetup has joined #openstack-release | 07:31 | |
| *** oanson has joined #openstack-release | 07:37 | |
| *** ykarel|meetup has quit IRC | 07:48 | |
| *** ifat_afek has joined #openstack-release | 07:55 | |
| *** tesseract has joined #openstack-release | 08:10 | |
| *** udesale has quit IRC | 08:14 | |
| *** udesale has joined #openstack-release | 08:15 | |
| *** udesale has quit IRC | 08:17 | |
| *** udesale has joined #openstack-release | 08:17 | |
| *** tosky has joined #openstack-release | 08:28 | |
| *** hberaud has joined #openstack-release | 08:32 | |
| *** zbr has quit IRC | 08:53 | |
| *** jpich has joined #openstack-release | 09:07 | |
| *** zbr has joined #openstack-release | 09:20 | |
| *** jonher has quit IRC | 09:26 | |
| *** dtantsur|afk is now known as dtantsur | 09:32 | |
| ttx | smcginnis, dhellmann re qinling-dashboard I would just do 1.0.0.0rc1 -- we might have to remove that 0.1.0 mention from the file for it to work | 09:53 |
| *** shardy has joined #openstack-release | 09:54 | |
| ttx | Probably safer to remove stable/stein and recreate it at 1.0.0.0rc1 too | 09:54 |
| ttx | hmm difficult now | 09:55 |
| * ttx tests | 09:56 | |
| openstackgerrit | Thierry Carrez proposed openstack/releases master: Fix numbering for qinling-dashboard https://review.openstack.org/647376 | 09:59 |
| tbarron | tonyb: if your'e still around I have a reply in https://review.openstack.org/#/c/645147 for your consideration | 10:11 |
| *** smrcascao has joined #openstack-release | 10:26 | |
| openstackgerrit | Daniel Mellado proposed openstack/releases master: Release kuryr-tempest-plugin for Stein https://review.openstack.org/647385 | 10:36 |
| *** jonher has joined #openstack-release | 10:37 | |
| openstackgerrit | sahid proposed openstack/reno master: add python 3.7 unit test job https://review.openstack.org/647387 | 10:39 |
| *** trident has quit IRC | 10:43 | |
| *** ifat_afek has quit IRC | 10:59 | |
| openstackgerrit | Daniel Mellado proposed openstack/releases master: Release kuryr-tempest-plugin for Stein https://review.openstack.org/647385 | 11:00 |
| openstackgerrit | Daniel Mellado proposed openstack/releases master: Release Kuryr-Kubernetes 0.6.2 https://review.openstack.org/647400 | 11:01 |
| *** trident has joined #openstack-release | 11:08 | |
| *** ifat_afek has joined #openstack-release | 11:08 | |
| *** jpich has quit IRC | 11:17 | |
| *** jpich has joined #openstack-release | 11:21 | |
| *** ifat_afek has quit IRC | 11:29 | |
| *** jroll has quit IRC | 12:14 | |
| *** jroll has joined #openstack-release | 12:15 | |
| *** whoami-rajat has quit IRC | 12:24 | |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: Replace openstack.org git:// URLs with https:// https://review.openstack.org/647108 | 12:28 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: release-test 1.4.1 https://review.openstack.org/647409 | 12:28 |
| *** ricolin has quit IRC | 12:37 | |
| *** cdent has joined #openstack-release | 12:38 | |
| openstackgerrit | Merged openstack/releases master: Move tarball-base settings to repo settings https://review.openstack.org/645865 | 12:43 |
| openstackgerrit | Akihiro Motoki proposed openstack/releases master: Release neutron-tempest-plugin (stein) https://review.openstack.org/647418 | 12:59 |
| *** mriedem has joined #openstack-release | 12:59 | |
| *** zul has joined #openstack-release | 13:09 | |
| smcginnis | ttx: I think we would need infra's help removing the current stable/stein, but I believe we've done that before. | 13:10 |
| ttx | smcginnis: I don't think we need to remove it after all | 13:11 |
| ttx | I think the branch creation will be skipped as it is already there | 13:12 |
| ttx | and since the new tag would point to the same SHA, branch point is essentially the same | 13:12 |
| ttx | Looking at test results for https://review.openstack.org/647376 it does not look like i'ts panicking | 13:13 |
| smcginnis | ttx: Oh, true. We aren't branching at a different point, just tagging it as something else. | 13:14 |
| smcginnis | ttx: That patch looks fine, and I _think_ that's all we should have to do then. | 13:15 |
| *** elbragstad has joined #openstack-release | 13:24 | |
| *** elbragstad is now known as lbragstad | 13:25 | |
| *** trident has quit IRC | 13:27 | |
| *** irclogbot_3 has joined #openstack-release | 13:28 | |
| *** trident has joined #openstack-release | 13:29 | |
| *** cdent has quit IRC | 13:31 | |
| *** altlogbot_2 has quit IRC | 13:31 | |
| *** altlogbot_2 has joined #openstack-release | 13:32 | |
| *** cdent has joined #openstack-release | 13:37 | |
| *** tosky has quit IRC | 13:38 | |
| *** irclogbot_3 has quit IRC | 13:38 | |
| *** irclogbot_1 has joined #openstack-release | 13:40 | |
| openstackgerrit | Akihiro Motoki proposed openstack/releases master: Release neutron-tempest-plugin (stein) https://review.openstack.org/647418 | 13:42 |
| openstackgerrit | Jiao Pengju proposed openstack/releases master: Release karbor 1.2.1 for stable/stein https://review.openstack.org/647473 | 13:52 |
| *** mlavalle has joined #openstack-release | 14:12 | |
| *** e0ne has joined #openstack-release | 14:23 | |
| *** ykarel|meetup has joined #openstack-release | 14:28 | |
| *** ykarel|meetup is now known as ykarel | 14:28 | |
| *** armax has joined #openstack-release | 14:46 | |
| *** ykarel is now known as ykarel|away | 14:50 | |
| *** N3l1x has joined #openstack-release | 14:54 | |
| *** ykarel|away has quit IRC | 14:56 | |
| *** beekneemech is now known as bnemec | 15:00 | |
| *** whoami-rajat has joined #openstack-release | 15:01 | |
| openstackgerrit | Merged openstack/reno master: add python 3.7 unit test job https://review.openstack.org/647387 | 15:03 |
| openstackgerrit | Brian Rosmaita proposed openstack/releases master: Release os-brick 2.3.6 https://review.openstack.org/647503 | 15:11 |
| ttx | smcginnis: Ok I'll remove the W-1 | 15:11 |
| ttx | Maybe dhellmann can do the craziness safety check | 15:12 |
| smcginnis | ++ | 15:12 |
| *** binh_ has quit IRC | 15:12 | |
| *** binh_ has joined #openstack-release | 15:12 | |
| dhellmann | ttx, smcginnis : on https://review.openstack.org/#/c/647376/ ? | 15:13 |
| smcginnis | Yeah | 15:13 |
| dhellmann | I think your logic on that is sound. Shall I approve? | 15:14 |
| ttx | ++ | 15:14 |
| dhellmann | approved | 15:14 |
| smcginnis | Still not sure how the script ended up with that, but at least this gets things "more correct". | 15:14 |
| openstackgerrit | Brian Rosmaita proposed openstack/releases master: Release os-brick 2.5.6 https://review.openstack.org/647506 | 15:15 |
| smcginnis | ttx, dhellmann: Either of you know where get-repo-owner used to exist? | 15:17 |
| smcginnis | Looking at annouce.sh fixes and we call that at one point, but I don't see it defined anywhere. | 15:17 |
| smcginnis | Reference: http://git.openstack.org/cgit/openstack/releases/tree/tools/announce.sh#n163 | 15:17 |
| smcginnis | PROJECT_OWNER is always empty | 15:18 |
| dhellmann | I wonder if that used to be a cli that was removed? | 15:18 |
| smcginnis | Time for some spelunking I guess. | 15:18 |
| dhellmann | it was added in 4f07da7e158ff9dea0857528af8faee877e3058d | 15:19 |
| dhellmann | and the script is still there | 15:20 |
| dhellmann | I wonder if we lost the setup.cfg entry in a merge conflict | 15:20 |
| openstackgerrit | Brian Rosmaita proposed openstack/releases master: Release cinder 11.2.1 https://review.openstack.org/647507 | 15:20 |
| dhellmann | these tools need unit tests :-/ | 15:20 |
| smcginnis | dhellmann: I'm not seeing it in that commit. | 15:21 |
| smcginnis | Looks like it was there since the script moved from release-tools. I wonder if it was a function that got missed migrating over. | 15:21 |
| dhellmann | oh, that's deliverable owner | 15:22 |
| dhellmann | duh | 15:22 |
| smcginnis | Doesn't look like we kept a copy of release-tools around anywhere. | 15:22 |
| smcginnis | Huh, doesn't look like we would do anything with PROJECT_OWNER, even if it was set. I'll just drop that bit. | 15:23 |
| openstackgerrit | Brian Rosmaita proposed openstack/releases master: Release cinder 12.0.6 https://review.openstack.org/647508 | 15:24 |
| dhellmann | oh, good | 15:24 |
| smcginnis | Not sure if we care, but I see even though we are publishing services to pypi, the tag metadata still has "meta:pypi no" | 15:26 |
| smcginnis | Oh wait, I may be looking at a bad example. | 15:26 |
| dhellmann | that depends on the flag in the deliverable file | 15:26 |
| openstackgerrit | Brian Rosmaita proposed openstack/releases master: Release cinder 13.0.4 https://review.openstack.org/647509 | 15:28 |
| smcginnis | OK, Cinder is publishing, but 2012.2 shows up as the latest. | 15:28 |
| openstackgerrit | Merged openstack/releases master: Fix numbering for qinling-dashboard https://review.openstack.org/647376 | 15:29 |
| dhellmann | yeah, we probably want to go delete the year-based releases | 15:29 |
| smcginnis | I suppose if we care about including that, makes more sense to be explicit with the deliverable flag than figuring out if it's publishing or not. | 15:29 |
| dhellmann | or at least hide them | 15:30 |
| smcginnis | I've asked in infra if someone can delete that. | 15:30 |
| smcginnis | I don't see any reason we would want to keep those around. | 15:30 |
| dhellmann | not everything we release is a python program, too, so we could potentially change the logic that sets that flag to just check the project type | 15:31 |
| dhellmann | or add pypy_url as optional metadata or something like that | 15:31 |
| smcginnis | I'm going to leave that as an exercise for the future reader. :) | 15:31 |
| *** ykarel|away has joined #openstack-release | 15:40 | |
| *** udesale has quit IRC | 15:55 | |
| *** cdent has quit IRC | 16:00 | |
| *** ifat_afek has joined #openstack-release | 16:10 | |
| *** ricolin has joined #openstack-release | 16:18 | |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Fix announce email paths https://review.openstack.org/647528 | 16:22 |
| *** ricolin has quit IRC | 16:24 | |
| openstackgerrit | Ben Nemec proposed openstack/releases master: oslo.db 4.45.0 https://review.openstack.org/647530 | 16:33 |
| *** e0ne has quit IRC | 16:35 | |
| *** cdent has joined #openstack-release | 16:37 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/releases master: Release Patrole 0.5.0 for Stein https://review.openstack.org/645868 | 16:42 |
| *** ekcs has quit IRC | 16:46 | |
| smcginnis | dhellmann: Any reason to hold off on the requirement branching? | 16:50 |
| *** diablo_rojo has joined #openstack-release | 16:51 | |
| openstackgerrit | Merged openstack/releases master: Release keystone-tempest-plugin 0.2.0 https://review.openstack.org/646042 | 16:56 |
| openstackgerrit | Merged openstack/releases master: Swift 2.21.0 and stable/stein release https://review.openstack.org/645972 | 17:01 |
| dhellmann | smcginnis : I don't think so. | 17:03 |
| dhellmann | smcginnis : oh, maybe wait for that oslo.db release? | 17:04 |
| dhellmann | https://review.openstack.org/647530 | 17:04 |
| dhellmann | and then the uc change, too, so we don't have to backport it | 17:05 |
| * dhellmann steps out for lunch | 17:05 | |
| *** hberaud has quit IRC | 17:06 | |
| *** dustinc has joined #openstack-release | 17:15 | |
| *** hberaud has joined #openstack-release | 17:18 | |
| openstackgerrit | Graham Hayes proposed openstack/releases master: Release designate-tempest-plugin 0.6.0 https://review.openstack.org/647547 | 17:27 |
| *** jpich has quit IRC | 17:28 | |
| *** dtantsur is now known as dtantsur|afk | 17:30 | |
| ttx | smcginnis: only two forced? | 17:33 |
| ttx | That's not bad at all | 17:33 |
| *** diablo_rojo has quit IRC | 17:39 | |
| openstackgerrit | Merged openstack/releases master: Mark unacknowledged RC releases as forced https://review.openstack.org/645860 | 17:43 |
| *** ykarel|away has quit IRC | 17:44 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/releases master: Release neutron-tempest-plugin (stein) https://review.openstack.org/647418 | 17:46 |
| gmann | smcginnis: updated neutron-tempest-plugin release. these tarball-base name started giving error now only? It was ok with Rocky release | 17:46 |
| smcginnis | gmann: The location does not cause an error. I just wanted that moved since there were other things to fix too. Having it under the repo-settings makes sure it carries forward to the next cycle, so hopefully that reduces the chance of errors from someone forgetting to include it in an individual release. | 17:51 |
| smcginnis | ttx: I was surprised by that too. I did not include the ones where the current PTL didn't ack but the incoming one did though. | 17:52 |
| smcginnis | dhellmann: Good call. We can wait on that. | 17:52 |
| gmann | smcginnis: +1 on location things. its name thing started giving error now than previous release. I am not sure we should keep tarball-base name to neutron_temepst_plugin and update setup.cfg or vice versa. | 17:53 |
| openstackgerrit | Eric Kao proposed openstack/releases master: congress tempest plugin 0.2.0 https://review.openstack.org/647563 | 17:53 |
| lbragstad | smcginnis should PTLs have stable/stein +2/+W even if they don't have stable core for a project? | 17:55 |
| smcginnis | gmann: This changed it for some reason: https://review.openstack.org/#/c/602375/5/setup.cfg | 17:55 |
| lbragstad | I thought I remember being able to approve stable/pike patches even though i wasn't keystone stable core, before we officially released stable/pike (strictly for RC period) | 17:55 |
| smcginnis | lbragstad: No. There used to be an interum group, but now it's just the stable group. | 17:55 |
| lbragstad | ok - so i'm not going crazy | 17:56 |
| gmann | smcginnis: humm, got it, leaving it to neutron team then. | 17:57 |
| *** cdent has quit IRC | 17:57 | |
| smcginnis | At least not in this case lbragstad ;) | 17:57 |
| lbragstad | cc cmurphy ^ | 17:57 |
| lbragstad | ack - thanks smcginnis! | 17:57 |
| cmurphy | o7 | 17:57 |
| gmann | smcginnis: it passed now - https://review.openstack.org/#/c/647418/ | 17:57 |
| *** ifat_afek has quit IRC | 18:03 | |
| lbragstad | smcginnis just fyi - we might need some help +W'ing some stable/stein patches since I proposed them and kmalloc is the only other keystone stable core | 18:04 |
| *** e0ne has joined #openstack-release | 18:05 | |
| openstackgerrit | Merged openstack/releases master: oslo.db 4.45.0 https://review.openstack.org/647530 | 18:14 |
| openstackgerrit | Ghanshyam Mann proposed openstack/releases master: Release Patrole 0.5.0 for Stein https://review.openstack.org/645868 | 18:21 |
| *** diablo_rojo has joined #openstack-release | 18:28 | |
| *** hberaud is now known as hberaud|gone | 18:34 | |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release neutron-tempest-plugin (stein) https://review.openstack.org/647418 | 18:35 |
| *** tesseract has quit IRC | 18:44 | |
| *** dustinc is now known as dustinc|away | 18:44 | |
| openstackgerrit | Merged openstack/releases master: Remove per release setting of tarball-base https://review.openstack.org/647258 | 18:46 |
| openstackgerrit | Merged openstack/releases master: Release kuryr-tempest-plugin for Stein https://review.openstack.org/647385 | 18:46 |
| openstackgerrit | Merged openstack/releases master: Release designate-tempest-plugin 0.6.0 https://review.openstack.org/647547 | 18:49 |
| openstackgerrit | Merged openstack/releases master: congress tempest plugin 0.2.0 https://review.openstack.org/647563 | 18:49 |
| *** diablo_rojo_ has joined #openstack-release | 18:57 | |
| *** diablo_rojo has quit IRC | 19:01 | |
| *** cdent has joined #openstack-release | 19:03 | |
| openstackgerrit | Tom Barron proposed openstack/releases master: Release manila-tempest-plugin for Stein https://review.openstack.org/647585 | 19:35 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Extract announce info from deliverable file https://review.openstack.org/647592 | 19:57 |
| *** diablo_rojo_ has quit IRC | 19:57 | |
| *** diablo_rojo_ has joined #openstack-release | 20:08 | |
| *** whoami-rajat has quit IRC | 20:11 | |
| openstackgerrit | Tom Barron proposed openstack/releases master: Release manila-tempest-plugin for Stein https://review.openstack.org/647585 | 20:12 |
| *** cdent has quit IRC | 20:18 | |
| *** zbr has quit IRC | 20:22 | |
| *** zbr has joined #openstack-release | 20:22 | |
| *** pcaruana has quit IRC | 20:23 | |
| *** tosky has joined #openstack-release | 20:23 | |
| prometheanfire | heat-11.0.1 doesn't have a tarball | 20:25 |
| prometheanfire | https://tarballs.openstack.org/heat/ | 20:25 |
| openstackgerrit | Merged openstack/releases master: Release neutron-tempest-plugin (stein) https://review.openstack.org/647418 | 20:29 |
| dhellmann | prometheanfire : some of the heat releases failed to upload to pypi, and I think the tarball step still comes after that step in the release jobs | 20:30 |
| prometheanfire | ah | 20:30 |
| *** ekcs has joined #openstack-release | 20:32 | |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Extract announce info from deliverable file https://review.openstack.org/647592 | 20:36 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release manila-tempest-plugin for Stein https://review.openstack.org/647585 | 20:42 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Add release note links https://review.openstack.org/647601 | 20:47 |
| *** e0ne has quit IRC | 20:48 | |
| openstackgerrit | Tom Barron proposed openstack/releases master: Release manila-tempest-plugin for Stein https://review.openstack.org/647585 | 20:49 |
| tbarron | smcginnis: thanks, didn't see that you had fixed it, clumsy afternoon ... | 20:53 |
| openstackgerrit | Merged openstack/releases master: Release Patrole 0.5.0 for Stein https://review.openstack.org/645868 | 20:54 |
| smcginnis | tbarron: ;) | 20:55 |
| openstackgerrit | Merged openstack/releases master: Release puppet-ceph 2.6.0 and branch stable/mimic https://review.openstack.org/644599 | 21:11 |
| openstackgerrit | Merged openstack/releases master: Release manila-tempest-plugin for Stein https://review.openstack.org/647585 | 21:55 |
| *** mlavalle has quit IRC | 22:08 | |
| *** jtomasek has joined #openstack-release | 22:11 | |
| mwhahaha | so question about tarbals, it looks like the setup.cfg is being converted to use tabs instead of the spaces in the release tarball so this is a difference from the actual code itself. any idea where this converstion might be occuring? | 22:16 |
| dhellmann | mwhahaha : I think setuptools does that when it rewrites the file to add some metadata about the package it builds | 22:56 |
| dhellmann | it seems like we ran into that somewhere once before | 22:56 |
| mwhahaha | yea i would be shocked if this is the first time someone has hit this | 22:56 |
| dhellmann | where are you seeing it? | 22:56 |
| mwhahaha | in a downstream patch on top of a tarbal | 22:56 |
| mwhahaha | the patch has spaces, the output of the tarball has tabs | 22:57 |
| dhellmann | figures | 22:57 |
| mwhahaha | so the rpm package errors when it goes to patch | 22:57 |
| dhellmann | yeah | 22:57 |
| dhellmann | in fact, I think that's where we saw it before | 22:57 |
| mwhahaha | i'm sure there's a spec hack that has to be done | 22:57 |
| mwhahaha | just trying to figure out what it is :D | 22:57 |
| mwhahaha | this can't be the first time someone tried to patch setup.cfg :D | 22:57 |
| dhellmann | I wonder if it's as easy as converting the tabs to spaces before patching | 22:57 |
| mwhahaha | it's all in the %autosetup macro, so i'm hoping it's not something too complex to work around | 22:58 |
| mwhahaha | i'll poke the packaging experts tomorrow | 22:58 |
| dhellmann | yeah, I don't know how that stuff works. the #rdo folks may remember the previous case | 22:58 |
| smcginnis | Seems like a really odd thing for setuptools to be doing. | 22:59 |
| mwhahaha | i just found it weird that the code was being rewritten like that | 22:59 |
| dhellmann | smcginnis : I think it uses a configparser to read and then write the file, and the writer is just configured differently | 22:59 |
| smcginnis | It would be nice if it was smart enough to use whatever was in the file for its whitespacing. | 23:00 |
| smcginnis | Or just use spaces like any logical and sane person would do. :) | 23:00 |
| mwhahaha | yea it's the hanging indentation that's switched | 23:01 |
| mwhahaha | so classifier is converted asn scripts/data_files | 23:01 |
| *** mriedem has quit IRC | 23:03 | |
| *** ekcs has quit IRC | 23:06 | |
| *** jtomasek has quit IRC | 23:11 | |
| *** diablo_rojo_ has quit IRC | 23:36 | |
| *** diablo_rojo has joined #openstack-release | 23:36 | |
| *** jtomasek has joined #openstack-release | 23:53 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!