Yes. Alan.
On Fri, Aug 16, 2019 at 12:45 PM Maxime Beauchemin < maximebeauche...@gmail.com> wrote: > @mentors, so should I kick off a vote with general@ now? > > Max > > On Fri, Aug 16, 2019 at 12:00 PM Jakob Homan <jgho...@gmail.com> wrote: > > > +1 (binding) > > * Some LICENSEs appear to be missing; can be fixed next release > > * Artifact contains rc1 suffix; usually this isn't the case. Not sure > > if the IPMC will be concerned > > * Sig looks good > > * DISCLAIMER/LICENSE (other than above)/NOTICE look good > > * Spot check on headers look good > > > > -Jakob > > > > On Thu, Aug 15, 2019 at 4:11 PM Beto Dealmeida <robe...@dealmeida.net> > > wrote: > > > > > > I can take a look at the release today and vote. > > > > > > On 8/15/19 3:51 PM, Alan Gates wrote: > > > > We should have a 3rd PPMC vote. If we can get Thejas or Jakob to > vote > > as > > > > mentors that will give us another PPMC vote and one more IPMC vote. > > > > > > > > Alan. > > > > > > > > On Thu, Aug 15, 2019 at 2:26 PM Maxime Beauchemin < > > > > maximebeauche...@gmail.com> wrote: > > > > > > > >> Vote passes! > > > >> > > > >> +2 binding votes (Alan & Max) > > > >> +3 non-binding vote (Ville, Charles, Bolke) > > > >> > > > >> No neutral or negative votes. > > > >> > > > >> Now (assuming 2 binding votes are enough) I have to figure out what > my > > > >> steps are. I think it consist of starting a vote thread similar to > > this one > > > >> on gene...@incubator.apache.org where I [think I] need 3 or more > IPMC > > > >> binding votes. > > > >> > > > >> Max > > > >> > > > >> On Mon, Aug 12, 2019 at 2:48 PM Maxime Beauchemin < > > > >> maximebeauche...@gmail.com> wrote: > > > >> > > > >>> +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> > > > >>>>>>>>>> > > > >>>>>>>> > > > >>>>> > > >