+cc Justin I'd appreciate if you could do a pass on this as you uncovered the most issues last time
Thanks, Max On Mon, Aug 12, 2019 at 1:38 PM Maxime Beauchemin < maximebeauche...@gmail.com> wrote: > We should probably develop and evolve guidelines on what constitutes a > release blocker. For this first Apache release I think we're more focussed > on license-related topics and ASF-type requirements along with "P0"-type > blockers that would make significant portions of Superset unusable. > > Bellow are some of the issues raised in a previous release attempt (from > an email that Justin McLean sent). I believe everything in there has been > tackled, but we should probably double check. > > ``` > I checked: > - incubating is missing from name > - signatures and hashes good > - DISCLAIMER exists > - LICENSE is missing a few licenses (see below) > - NOTICE should not include LICENSE information e.g The line about CC 4.0. > This may need to be removed. Also this content may be Category X and this > can’t included in a source release. [11] > - Some 3rd party code have headers with "[LICENSE TBD]” [4][5][6][7][9] > which seems odd > - The ASF headers on a number of files are not correct e.g the file [13] > and other under that directory (search for COPYRIGHT 2016 and and COPYRIGHT > 2017 and COPYRIGHT 2018) Or are these files not actually licensed to the > ASF? > - Source release may contained compiled code e.g [20] my understanding is > that a .mo file is compiled from the .po file. I not sure if it could be > classified as code however, but even if that is so it may be best to remove > them from the source release. > > There is a few bits of missing licensing information: > - How is the map date here licensed? [3] Is it from > http://www.diva-gis.org/Data? I can see there is a credit in the code > that has a link to https://bl.ocks.org/john-guerra. How is that code > licensed? > - This file [7] is missing from LICENSE I assume it’s public domain? [8] > - This file [9] is licensed how? It seems to come from here [10] which > says to contact the author for a license. > - How is this image [14] licensed? (One reference I found said it could > only be used for personal use [21]) > - LICENSE is missing license information for these fonts > [15][16][17][18][19] The license for Glyphicons Halflings is complex and in > this context (outside of bootstrap) a commercial license may be needed [22] > and so it can’t be included. > > Some other improvements that could be made: > - The LICENSE fieldworker is a little odd with headings like BSD 2-Clause > licenses with noting under them, people (or tools) may incorrectly assume > that it does contain BSD 2-Clause licensed code. > - Make vote link link in future emails to lists.apache,org > - Sign release with an apache.org email address > - This [12] has copyright 2018 when it shovel be 2019 > > Thanks, > Justin > > 1. https://incubator.apache.org/policy/incubation.html#releases > 2. > https://cwiki.apache.org/confluence/display/INCUBATOR/Incubator+Release+Checklist > 3. apache-superset > -0.32.0rc2/superset/assets/src/visualizations/CountryMap/* > 4. apache-superset-0.32.0rc2/superset/assets/vendor/cal-heatmap/cal-heatmap.js > / .css > 5 > apache-superset-0.32.0rc2/superset/assets/vendor/parallel_coordinates/d3.parcoords.js > / .css > 6. apache-superset > -0.32.0rc2/superset/assets/vendor/parallel_coordinates/divgrid.js > 7 apache-superset-0.32.0rc2/superset/assets/vendor/pygments.css > 8. https://github.com/richleland/pygments-css/blob/master/UNLICENSE.txt > 9. ./apache-superset > -0.32.0rc2/superset/assets/vendor/parallel_coordinates/divgrid.js > 10. http://bl.ocks.org/syntagmatic/3687826 > 11. http://www.tavfrna.incubator.apache.org/legal/resolved.html#cc-sa > 12. ./apache-superset-0.32.0rc2/docs/conf.py > 13. ./apache-superset-0.32.0rc2/superset/translations/messages.pot > 14. ./apache-superset-0.32.0rc2/superset/assets/images/babytux.jpg > 15. apache-superset > -0.32.0rc2/superset/assets/stylesheets/fonts/Roboto-Regular.woff > 16. apache-superset > -0.32.0rc2/superset/assets/stylesheets/fonts/Roboto-Regular.woff2 > 17. apache-superset > -0.32.0rc2/superset/assets/stylesheets/fonts/glyphicons-halflings-regular.ttf > 18. apache-superset > -0.32.0rc2/superset/assets/stylesheets/fonts/glyphicons-halflings-regular.woff > 19. apache-superset > -0.32.0rc2/superset/assets/stylesheets/fonts/glyphicons-halflings-regular.woff2 > 20 apache-superset > -0.32.0rc2/superset/translations/de/LC_MESSAGES/messages.mo > 21. https://www.pngfly.com/png-79pbyi/ > 22. https://www.glyphicons.com/license/ > ```` > > > > Max > > On Mon, Aug 12, 2019 at 10:55 AM Charles Givre <cgi...@gmail.com> wrote: > >> >> >> >> I built and ran some queries in SQL Lab, created some visualizations. >> >> I did encounter an issue, which is not new, but I'd love to see fixed. >> Specifically, when I attempt to create a line chart, I run the query, but >> in order to actually get the visualization, I have to refresh my browser. >> However, this is not new so I'll open a ticket. >> >> >> +1 (non-binding) from me. >> > >> > >> > >> > >> >> On Aug 12, 2019, at 1:22 PM, Ville Brofeldt < >> ville.v.brofe...@gmail.com <mailto:ville.v.brofe...@gmail.com>> wrote: >> >> >> >> It's my understanding that only PMC members can cast binding votes. >> Hence >> >> my non-binding vote (committer, not PMC member). Ville >> >> >> >> >> >> On Mon, Aug 12, 2019, 20:08 Bolke de Bruin <bdbr...@gmail.com <mailto: >> bdbr...@gmail.com>> wrote: >> >> >> >>> We need binding votes from the committers... 3 being the minimum. Its >> >>> common to make it clear that you are either voting binding or non >> binding. >> >>> Binding votes can only be cast by committers. >> >>> >> >>> This is my +1, NON-binding. >> >>> >> >>> Bolke >> >>> >> >>> Verstuurd vanaf mijn iPad >> >>> >> >>>> Op 10 aug. 2019 om 06:16 heeft Ville Brofeldt < >> >>> ville.v.brofe...@gmail.com <mailto:ville.v.brofe...@gmail.com>> het >> volgende geschreven: >> >>>> >> >>>> Hi all, >> >>>> >> >>>> I believe this is a good point in time to cut the 0.34 release, as >> there >> >>>> have been numerous important bugfixes and features introduced since >> 0.33, >> >>>> with especially important license housekeeping in the form of >> removal of >> >>>> requests (thanks Gianluca!) and FOSSA CI (thanks Max!). So would >> like to >> >>>> extend my non-binding +1 to this. Fingers crossed! >> >>>> >> >>>> Ville >> >>>> >> >>>> On Fri, Aug 9, 2019 at 9:36 PM Maxime Beauchemin < >> >>> maximebeauche...@gmail.com <mailto:maximebeauche...@gmail.com>> >> >>>> wrote: >> >>>> >> >>>>> Dear all, >> >>>>> >> >>>>> The source release 0.34.0 RC1 for Apache Superset is baked and >> available >> >>>>> at: >> >>>>> https://dist.apache.org/repos/dist/dev/incubator/superset/ < >> https://dist.apache.org/repos/dist/dev/incubator/superset/>, public >> >>>>> keys are available >> >>>>> at >> https://dist.apache.org/repos/dist/release/incubator/superset/KEYS < >> https://dist.apache.org/repos/dist/release/incubator/superset/KEYS> >> >>>>> >> >>>>> We're using the 0.34 branch as the base for this first ASF release >> as >> >>>>> opposed to 0.33 in previous attempts. I think all license-related >> issues >> >>>>> have been ironed out in our dependency, here's the FOSSA report >> >>>>> < >> >>>>> >> >>> >> https://app.fossa.com/projects/custom+11342%2f...@github.com:apache%2Fincubator-superset.git/refs/branch/0.34/a04fad858644466219b7ea399aead110cb8ea655 >> < >> https://app.fossa.com/projects/custom+11342%2f...@github.com:apache%2Fincubator-superset.git/refs/branch/0.34/a04fad858644466219b7ea399aead110cb8ea655 >> > >> >>>>>> >> >>>>> . *We're still ironing out our release process, so please bear with >> us >> >>> and >> >>>>> help if you can*. >> >>>>> >> >>>>> As I went along, I documented the process in [yet-to-be-merged] >> >>>>> RELEASING/README.md in the repo, latest edits here # >> >>>>> <https://github.com/apache/incubator-superset/pull/8016>8016 >> >>>>> <https://github.com/apache/incubator-superset/pull/8016>. As part >> of >> >>>>> `RELEASING/`, we ship docker files to help test releases in a >> >>> reproducible >> >>>>> way. >> >>>>> >> >>>>> For context the `0.34` release branch was cut at SHA 9233a63, that >> was >> >>>>> merged on master on Aug 8th 2019. From that common ancestor, the >> >>> following >> >>>>> list of commit was added as cherry-picks. The SHAs in the list >> bellow >> >>>>> reference the cherries on the release branch, PR number are >> available to >> >>>>> get more details. >> >>>>> >> >>>>> <no cherries yet on the 0.34 branch> >> >>>>> >> >>> >> >>> >> > >> >>