Re: [openssl-project] Issues review
I have now also reviewed the PRs. We currently have 144 PRs open: PRs against 1.0.2: 6 PRs against 1.1.0: 7 PRs against 1.1.1: 55 PRs against 1.2.0: 9 PRs against Post 1.1.1: 63 PRs against Other: 2 I was quite strict about allocating PRs to a milestone. This might mean that some of those decisions are controversial. Basically if a PR was about adding a feature, and it wasn't directly TLSv1.3 related (which is the stated primary objective for the release) then I put it against the Post 1.1.1 milestone. This does *not* mean that I think that a PR flagged as "Post 1.1.1" shouldn't go into 1.1.1. It just means that it is not relevant as far as the release criteria go - if it makes it into the release then great; if it doesn't - well we're not going to hold up the release schedule for it. If there are individual cases where you think a PR absolutely *MUST* go into 1.1.1 for whatever reason (to the point that we should hold up the release schedule for it) then we can argue that out on a case-by-case basis and amend the milestones accordingly. Alternatively just make sure you get it reviewed and committed before feature freeze. Matt On 23/01/18 17:49, Matt Caswell wrote: > I completed my first pass review of all issues. I still need to look at > PRs. I have put all PRs against a milestone using the following criteria: > > If it only applies to 1.0.2 or below: 1.0.2 milestone > If it only applies to 1.1.0 or below: 1.1.0 milesone > If it's API/ABI breaking to fix: 1.2.0 milestone > If it's a feature request that we aren't already planning to do for > 1.1.1: Post 1.1.1 milestone > If it's something which is independent of a release (e.g. web issues, > policy/procedure etc): Other milestone > If it applies to master and doesn't fit into one of the categories > above: 1.1.1 milestone > > > Some stats: > > We now have 249 issues open. > Since I started this exercise we have closed 134 issues. > Issues against 1.0.2: 17 > Issues against 1.1.0: 3 > Issues against 1.1.1: 141 > Issues against 1.2.0: 15 > Issues against Post 1.1.1: 58 > Issues against Other: 15 > > > Matt > ___ > openssl-project mailing list > openssl-project@openssl.org > https://mta.openssl.org/mailman/listinfo/openssl-project > ___ openssl-project mailing list openssl-project@openssl.org https://mta.openssl.org/mailman/listinfo/openssl-project
Re: [openssl-project] Issues review
On Tue, Jan 23, 2018 at 11:20:20PM +, Matt Caswell wrote: > > > On 23/01/18 20:55, Benjamin Kaduk wrote: > > On Tue, Jan 23, 2018 at 06:11:50PM +, Matt Caswell wrote: > >> > >> > >> 1.1.0 and 1.0.2 are still supported so issues against those milestones > >> are still relevant. They are *not* relevant to the 1.1.1 release > >> timetable though (which is why I started this exercise). Consider an > >> issues against the 1.1.0 milestone to mean, relevant to the next 1.1.0 > >> letter release. > > > > That's great if that's the intent, but I don't think that the > > current application of those tags is consistent with the above > > description. For example, #1418 is a somewhat abstract question of > > what it means for acertificate to be self-signed, yet has the 1.0.2 > > milestone, when (to me) 1.2.0 would seem more appropriate. > > That *is* the intent. What I've done here is *triage* - spending a few > minutes on each one to assess the correct milestone. It would not > surprise me to learn that, having done that for over 380 issues, we > might have come to a different assessment on a few of them. :-) Understood. Hopefully the bits I did last week helped with the triage. I mostly was just not sure if this has always been the policy, or if there was a period of time when things were much more haphazard as we first started using github. Going forward I expect that we'll be in better shape. > To be absolutely sure though I just re-reviewed all of those issues > against the 1.0.2 and 1.1.0 milestones (there weren't that many of > them), to make sure I got them right. I made 2 or 3 changes including to > the issue you highlighted (moving it to the "Post 1.1.1" milestone). Thank you; I do appreciate it! > Feel free to make any other adjustments you think might be necessary as > you come across them. Okay. I may try to look at the 1.0.2 and 1.1.0 issues, but really ought to finish up the draft-23 support PR first :) > Although there may be some disagreements on a few of the issues. I am > confident that the milestones as they are currently set are broadly > correct - and a good basis for planning. We should probably revive the release timeline/planning thread now, yes. -Ben ___ openssl-project mailing list openssl-project@openssl.org https://mta.openssl.org/mailman/listinfo/openssl-project
Re: [openssl-project] Issues review
On 23/01/18 20:55, Benjamin Kaduk wrote: > On Tue, Jan 23, 2018 at 06:11:50PM +, Matt Caswell wrote: >> >> >> On 23/01/18 18:05, Benjamin Kaduk wrote: >>> On Tue, Jan 23, 2018 at 05:51:41PM +, Matt Caswell wrote: On 23/01/18 17:49, Matt Caswell wrote: > I completed my first pass review of all issues. I still need to look at > PRs. I have put all PRs against a milestone using the following criteria: I have put all *issues* against a milestone not PR!! >>> >>> Do we still need to review the issues assigned to milestones that >>> have already happened (e.g., 1.1.0, post-1.1.0)? >> >> There no issues against post-1.1.0 (there are PRs - but that will be >> fixed when I do the PR review). >> >> 1.1.0 and 1.0.2 are still supported so issues against those milestones >> are still relevant. They are *not* relevant to the 1.1.1 release >> timetable though (which is why I started this exercise). Consider an >> issues against the 1.1.0 milestone to mean, relevant to the next 1.1.0 >> letter release. > > That's great if that's the intent, but I don't think that the > current application of those tags is consistent with the above > description. For example, #1418 is a somewhat abstract question of > what it means for acertificate to be self-signed, yet has the 1.0.2 > milestone, when (to me) 1.2.0 would seem more appropriate. That *is* the intent. What I've done here is *triage* - spending a few minutes on each one to assess the correct milestone. It would not surprise me to learn that, having done that for over 380 issues, we might have come to a different assessment on a few of them. :-) To be absolutely sure though I just re-reviewed all of those issues against the 1.0.2 and 1.1.0 milestones (there weren't that many of them), to make sure I got them right. I made 2 or 3 changes including to the issue you highlighted (moving it to the "Post 1.1.1" milestone). Feel free to make any other adjustments you think might be necessary as you come across them. The primary objective here is to inform the debate on the release criteria for 1.1.1 and the associated timeframes. You will recall that one of the proposed criteria was: "- All open github issues/PRs older than 2 weeks at the time of release to be assessed for relevance to 1.1.1. Any flagged with the 1.1.1 milestone to be closed (see below) ... Valid reasons for closing an issue/PR with a 1.1.1 milestone might be: - We have just now or sometime in the past fixed the issue - Unable to reproduce (following discussion with original reporter if possible) - Working as intended - Deliberate decision not to fix until a later release - Not enough information and unable to contact reporter - etc" We then got into discussions about timeframes - which was difficult without knowing the scope of the work. Although there may be some disagreements on a few of the issues. I am confident that the milestones as they are currently set are broadly correct - and a good basis for planning. Matt ___ openssl-project mailing list openssl-project@openssl.org https://mta.openssl.org/mailman/listinfo/openssl-project
Re: [openssl-project] Issues review
On Tue, Jan 23, 2018 at 06:11:50PM +, Matt Caswell wrote: > > > On 23/01/18 18:05, Benjamin Kaduk wrote: > > On Tue, Jan 23, 2018 at 05:51:41PM +, Matt Caswell wrote: > >> > >> > >> On 23/01/18 17:49, Matt Caswell wrote: > >>> I completed my first pass review of all issues. I still need to look at > >>> PRs. I have put all PRs against a milestone using the following criteria: > >> > >> I have put all *issues* against a milestone not PR!! > > > > Do we still need to review the issues assigned to milestones that > > have already happened (e.g., 1.1.0, post-1.1.0)? > > There no issues against post-1.1.0 (there are PRs - but that will be > fixed when I do the PR review). > > 1.1.0 and 1.0.2 are still supported so issues against those milestones > are still relevant. They are *not* relevant to the 1.1.1 release > timetable though (which is why I started this exercise). Consider an > issues against the 1.1.0 milestone to mean, relevant to the next 1.1.0 > letter release. That's great if that's the intent, but I don't think that the current application of those tags is consistent with the above description. For example, #1418 is a somewhat abstract question of what it means for acertificate to be self-signed, yet has the 1.0.2 milestone, when (to me) 1.2.0 would seem more appropriate. -Ben ___ openssl-project mailing list openssl-project@openssl.org https://mta.openssl.org/mailman/listinfo/openssl-project
Re: [openssl-project] Issues review
On 23/01/18 18:05, Benjamin Kaduk wrote: > On Tue, Jan 23, 2018 at 05:51:41PM +, Matt Caswell wrote: >> >> >> On 23/01/18 17:49, Matt Caswell wrote: >>> I completed my first pass review of all issues. I still need to look at >>> PRs. I have put all PRs against a milestone using the following criteria: >> >> I have put all *issues* against a milestone not PR!! > > Do we still need to review the issues assigned to milestones that > have already happened (e.g., 1.1.0, post-1.1.0)? There no issues against post-1.1.0 (there are PRs - but that will be fixed when I do the PR review). 1.1.0 and 1.0.2 are still supported so issues against those milestones are still relevant. They are *not* relevant to the 1.1.1 release timetable though (which is why I started this exercise). Consider an issues against the 1.1.0 milestone to mean, relevant to the next 1.1.0 letter release. Matt ___ openssl-project mailing list openssl-project@openssl.org https://mta.openssl.org/mailman/listinfo/openssl-project
Re: [openssl-project] Issues review
On Tue, Jan 23, 2018 at 05:51:41PM +, Matt Caswell wrote: > > > On 23/01/18 17:49, Matt Caswell wrote: > > I completed my first pass review of all issues. I still need to look at > > PRs. I have put all PRs against a milestone using the following criteria: > > I have put all *issues* against a milestone not PR!! Do we still need to review the issues assigned to milestones that have already happened (e.g., 1.1.0, post-1.1.0)? -Ben ___ openssl-project mailing list openssl-project@openssl.org https://mta.openssl.org/mailman/listinfo/openssl-project
Re: [openssl-project] Issues review
I reviewed all PR’s and issues that have FIPS in the text and closed most of them as not relevant. On 1/23/18, 12:51 PM, "Matt Caswell" wrote: On 23/01/18 17:49, Matt Caswell wrote: > I completed my first pass review of all issues. I still need to look at > PRs. I have put all PRs against a milestone using the following criteria: I have put all *issues* against a milestone not PR!! > > If it only applies to 1.0.2 or below: 1.0.2 milestone > If it only applies to 1.1.0 or below: 1.1.0 milesone > If it's API/ABI breaking to fix: 1.2.0 milestone > If it's a feature request that we aren't already planning to do for > 1.1.1: Post 1.1.1 milestone > If it's something which is independent of a release (e.g. web issues, > policy/procedure etc): Other milestone > If it applies to master and doesn't fit into one of the categories > above: 1.1.1 milestone > > > Some stats: > > We now have 249 issues open. > Since I started this exercise we have closed 134 issues. > Issues against 1.0.2: 17 > Issues against 1.1.0: 3 > Issues against 1.1.1: 141 > Issues against 1.2.0: 15 > Issues against Post 1.1.1: 58 > Issues against Other: 15 > > > Matt > ___ > openssl-project mailing list > openssl-project@openssl.org > https://mta.openssl.org/mailman/listinfo/openssl-project > ___ openssl-project mailing list openssl-project@openssl.org https://mta.openssl.org/mailman/listinfo/openssl-project ___ openssl-project mailing list openssl-project@openssl.org https://mta.openssl.org/mailman/listinfo/openssl-project
Re: [openssl-project] Issues review
On 23/01/18 17:49, Matt Caswell wrote: > I completed my first pass review of all issues. I still need to look at > PRs. I have put all PRs against a milestone using the following criteria: I have put all *issues* against a milestone not PR!! > > If it only applies to 1.0.2 or below: 1.0.2 milestone > If it only applies to 1.1.0 or below: 1.1.0 milesone > If it's API/ABI breaking to fix: 1.2.0 milestone > If it's a feature request that we aren't already planning to do for > 1.1.1: Post 1.1.1 milestone > If it's something which is independent of a release (e.g. web issues, > policy/procedure etc): Other milestone > If it applies to master and doesn't fit into one of the categories > above: 1.1.1 milestone > > > Some stats: > > We now have 249 issues open. > Since I started this exercise we have closed 134 issues. > Issues against 1.0.2: 17 > Issues against 1.1.0: 3 > Issues against 1.1.1: 141 > Issues against 1.2.0: 15 > Issues against Post 1.1.1: 58 > Issues against Other: 15 > > > Matt > ___ > openssl-project mailing list > openssl-project@openssl.org > https://mta.openssl.org/mailman/listinfo/openssl-project > ___ openssl-project mailing list openssl-project@openssl.org https://mta.openssl.org/mailman/listinfo/openssl-project
[openssl-project] Issues review
I completed my first pass review of all issues. I still need to look at PRs. I have put all PRs against a milestone using the following criteria: If it only applies to 1.0.2 or below: 1.0.2 milestone If it only applies to 1.1.0 or below: 1.1.0 milesone If it's API/ABI breaking to fix: 1.2.0 milestone If it's a feature request that we aren't already planning to do for 1.1.1: Post 1.1.1 milestone If it's something which is independent of a release (e.g. web issues, policy/procedure etc): Other milestone If it applies to master and doesn't fit into one of the categories above: 1.1.1 milestone Some stats: We now have 249 issues open. Since I started this exercise we have closed 134 issues. Issues against 1.0.2: 17 Issues against 1.1.0: 3 Issues against 1.1.1: 141 Issues against 1.2.0: 15 Issues against Post 1.1.1: 58 Issues against Other: 15 Matt ___ openssl-project mailing list openssl-project@openssl.org https://mta.openssl.org/mailman/listinfo/openssl-project