openservicebrokerapi / servicebroker

Open Service Broker API Specification
https://openservicebrokerapi.org/
Apache License 2.0
1.19k stars 436 forks source link

Release Proposal: v2.17 #751

Closed rsampaio closed 1 year ago

rsampaio commented 2 years ago

Announcing a new release candidate as described in the Release Process.

Release version: v2.17 Release RC SHA: c619d7165a469b333442bc2d20cc934dc0f7ea69 Target release date: 2022-07-26

The one-week Review Process will be triggered on the next weekly call.

rsampaio commented 2 years ago

@Samze @jberkhahn @gberche-orange We need a couple more approvals here to cut this release, are you guys able to approve? Thanks!

gberche-orange commented 2 years ago

this release LGTM (however I'm not an approved reviewer).

zoharyehezkely commented 1 year ago

Hi, Any update regarding the release?

Thanks, Zohar.

rsampaio commented 1 year ago

@tinygrasshopper or @jberkhahn we still need a +1 to complete this release, if any of you are available I would appreciate a review.

Thanks @gberche-orange, for some reason I thought you joined the PMC team and we should also fix that :)

rsampaio commented 1 year ago

@zoharyehezkely it is in progress, we need one more approval from the PMC members to complete the release, it should happen soon.

tinygrasshopper commented 1 year ago

@pivotal-marcela-campo any thoughts on the release?

Samze commented 1 year ago

LGTM, would like @pivotal-marcela-campo to give it a thumbs up from the CF side before approval

pivotal-marcela-campo commented 1 year ago

@Samze @tinygrasshopper I am okay with this release.

zoharyehezkely commented 1 year ago

Perfect, Now that we have 3 approvers what is the next step? Will it be automatically merged?

Thanks, Zohar.

rsampaio commented 1 year ago

Thanks for the approvals @Samze and @tinygrasshopper ! I will move forward with the process today 👍

@zoharyehezkely the next steps are outlined here, I will merge and complete the steps sometime today.

rsampaio commented 1 year ago

An update on this release: looks like TravisCI is not running the validation step required, I couldn't find a reason why this is failing or not triggering and I am investigating, as soon as we get the CI to pass I will merge this and complete the release.

@pivotal-marcela-campo maybe you have the permission to check the travisCI account for cloudfoundry, I suspect this is related to a plan migration that happened in Travis and they now require a plan to build projects.

pivotal-marcela-campo commented 1 year ago

@rsampaio Looking into this

zoharyehezkely commented 1 year ago

Hi,

Any updates?

Thanks, Zohar.

rsampaio commented 1 year ago

@avilupu @Samze @tinygrasshopper CI caught a lower case reference and I just fixed, I may need your approvals again, sorry about that!

zoharyehezkely commented 1 year ago

Hi @rsampaio,

Are we good for merging?

Thanks, Zohar

rsampaio commented 1 year ago

@pivotal-marcela-campo Can you make me admin of this repo? Apparently this travis problem is a known issue and a fix is documented here but requires admin to enable protected branches: https://travis-ci.community/t/known-issue-travis-ci-reports-expected-waiting-for-status-to-be-reported-on-the-github-status-api-but-the-status-never-arrives/1154

pivotal-marcela-campo commented 1 year ago

@rsampaio Thanks for the instructions, was under the impression that Travis was fixed after assigning a plan and importing the repos. I had now configured the check in this repo, working now.

zoharyehezkely commented 1 year ago

Thank you all very much