Skip to content

Latest commit

 

History

History
178 lines (149 loc) · 9.06 KB

CONTRIBUTING.md

File metadata and controls

178 lines (149 loc) · 9.06 KB
Raw

Contributing

All contributors sending Pull Requests (PRs) must have a Contributor License Agreement on file as either an individual or via their employer.

Minor Changes

Minor change proposals to the specification, changes such as editorial bugs or enhancements that do not modify the semantics of the specification or syntax of the API, can be suggested via a Github Issue or Pull Request (PR).

If there is a need for some discussion around how best to address the concern then opening an Issue, prior to doing the work to develop a PR, would be best. These minor issues do not need a formal review, per the process described below, rather the issue should be used to come to a consensus around a PR that will eventually be submitted.

If the proposed change is non-controversial (e.g. a typo) then a PR can be submitted directly without opening an issue.

Either way, once a PR is submitted it will be reviewed per the process described below.

Design Change Proposals

New design proposals to the API spec should be submitted by opening a Github issue with a link to a Google Doc containing the proposal. Proposals should focus primarily on motivation, problem statement, and use cases before suggesting a solution or change. Collaboration on the design, fleshing out of use cases, etc can occur as comment discussions in the Google Doc, as well as on our weekly calls.

For changes largely impacting the SB API actors (platforms, service authors, etc...), it is recommended to solicit feedback from these actors and leave enough time (say 2 weeks) for feedback to be provided, and for the potentially received objections/suggestions to be handled.

Once the design has been finalized in the Google Doc, the proposed set of changes to the specification should be made available for review. This could be done by pointing to a branch in a Github repo with the proposed edits. Reviewers, or implementers of the feature, can then easily see the exact changes being proposed and comment on them within that branch.

Before a formal PR for the changes is created, the feature needs to be validated through a full implementation by at least one platform that is easily accessible to reviewers. Note that in many cases this would include support for the proposed changes by one or more service brokers (potentially samples). How each platform makes this new feature available will vary between platforms. It is strongly recommended that as part of the design review process platforms that plan on implementing the feature get confirmation from the reviewers that their planned mechanism for providing access to the implementation of the feature is acceptable.

It is expected that during this implementation phase there will be changes made to the design and proposed specification edits to accurately represent the current status of the proposal.

Once support for the change has been implemented, a PR should be sent to this repo for the change to the Open Service Broker API specification. By this point, the API interactions should be well understood and there should be no technical surprises; we expect the only discussion necessary on PRs to be for wordsmithing and formatting.

GitHub Labels

When new design proposals for the API specification are submitted, the working group will use a set of pre-defined GitHub labels to highlight the current stage of the proposal. Note that these labels will not be used for minor changes.

The labels that will be used are as follows:

  • 1 - reviewing proposal: The API working group is actively reviewing a proposal that has been submitted as a GitHub Issue with the aim of validating both the problem statements and any proposed solutions.
  • 2 - validating through implementation: One of more platforms are actively working on the proposal with the aim of providing feedback on the implementation of the proposed solution.
  • 3 - reviewing PR: Feedback has been received on the implementation of the proposed solution and a Pull Request has been created containing the validated specification changes.

Note that not all issues will require a validation through implementation stage, and proposals can move back to a previous label at any time. When a PR is merged, the associated issue should be closed and any labels removed from the issue.

PR/Issue Review Process

All proposals (either Pull Requests or Issues) will follow the process described below:

  • All proposals must be available for review for no less than one week before they are approved. This will provide each dedicated committer enough time to review the proposal without unnecessarily delaying forward progress. Any non-trivial edit to the proposal (e.g. edits larger than typos) will reset the clock.
  • Any dedicated committer can veto (via a "NOT LGTM" comment in the proposal). The comment must include the reasoning behind the veto. It is then expected that the dedicated committers will discuss the concerns and determine the next step for proposal - either close/reject the proposal or address the concerns raised such that the "NOT LGTM" can be rescinded.
  • A proposal requires at least 4 "LGTM" comments from at least 4 different organizations to be approved.
  • Once a "design change" issue is approved, it will be tagged with an "proposal finalized" label. This indicates that it is ready to be implemented by a platform developer, see the process above.
  • Once a Pull Request is approved, it will be merged into the 'master' branch.

Release Process

Any member of the PMC can request a specific SHA on master (the Release SHA) is ready to be released into a new version of the spec. They will do this by creating a new PR with the title of the proposed release. For example, "Release Proposal: v$major.$minor".

Prepare a PR

  1. In a fork, create a new branch called "v$major.$minor-rc" from the Release SHA.
  2. Create a new commit titled prepare release with the following changes:
  • Update release-notes.md detailing the changes that are to be released in this version.
  • Update README.md with an updated Latest Release subheading and links to the latest version of the documents (spec.md, profile.md, etc).
  • Update spec.md with an updated Changes Since v... section (and link from table of contents) containing a copy of the relevant release notes, and with any references to the previous version of the specification (i.e. the X-Broker-API-Version headers) updated. Do not update the header Open Service Broker API (master - might contain changes that are not yet released) (this will be done if and when the release proposal is approved).
  • Update diagram.md to show the new version number in the header and link to the correct version of the Google Drawing from the OSBAPI Google Drive Folder.
  1. Open a new Pull Request titled Release Proposal: v$major.$minor from the branch of the fork to the master branch of the repository.
  2. Announce the release proposal on the next weekly call and notify the mailing list of the proposal, triggering the start of the Review Process as outlined below.

Review Process

  • All release proposals must be available for review for no less than one week before they are approved. This will provide each dedicated committer enough time to review the release proposal without unnecessarily delaying forward progress.
  • Any dedicated committer can veto (via a "NOT LGTM" comment in the proposal). The comment must include the reasoning behind the veto. It is then expected that the dedicated committers will discuss the concerns and determine the next steps for release proposal. The submitter should either close/reject the proposal or address the concerns raised such that the "NOT LGTM" can be rescinded.
  • A release proposal requires at least 4 "LGTM" comments from at least 4 different organizations to be approved.

Once Approved

Once the release is approved, the following actions should be taken by any PMC member:

  1. Merge the release proposal PR into the master branch of the repository. There should not be any conflicts as the text in the files that have changed should only be changed during this release process.
  2. Create a new branch called "v$major.$minor" from the Release SHA.
  3. Cherry pick the commit in which the release proposal PR was merged, to pick up the file changes.
  4. Create a new commit updating the spec.md and profile headers to include the version of the release (Open Service Broker API (v$major.$minor)).
  5. Push the branch to the repository (v$major.$minor).
  6. Notify the mailing list of the new release.
  7. Update the Roadmap & Release Planning project.
  8. The PMC will create a blog post for the new release.