Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New Repo: eventing-natss #236

Closed
10 tasks done
Harwayne opened this issue Aug 25, 2020 · 5 comments
Closed
10 tasks done

New Repo: eventing-natss #236

Harwayne opened this issue Aug 25, 2020 · 5 comments
Assignees

Comments

@Harwayne
Copy link
Contributor

Harwayne commented Aug 25, 2020

Repo information

Org: knative-sandbox

Repo: eventing-natss

Purpose (Description): NATS streaming integration with Knative Eventing.

Sponsoring WG: Event Delivery WG

Actions to fulfill

This area is used for the TOC to track the repo creation
process

  • Add this issue to the TOC project board for review.

  • Send a PR adding entries for this repo in /peribolos/knative-sandbox.yaml

    • Add the repository and a description.
    • Grant Knative Admin the admin privilege.
    • Grant the sponsoring WG the write privilege.

Once the TOC has approved the above, it will merge and Peribolos will create an empty repository.

  • (golang) Send a PR to add aliases for knative.dev/$REPONAME import paths (sample).

  • Have a lead from the sponsoring WG bootstrap the Git repository by pushing an
    appropriate "template" repository (basic,
    sample-controller,
    sample-source) to the new repository as
    a git remote. For example:

      git clone https://github.com/knative-sandbox/sample-controller.git
      cd sample-controller
      git remote add newrepo https://github.com/knative-sandbox/$REPONAME.git
      git push newrepo master
  • Set up test-infra following the
    docs linked at the beginning.

  • Create a sample PR to verify Prow (e.g. edit the boilerplate README)

Once Prow has been verified.

  • Have a member of the TOC enable branch protections so that tide is a required presubmit check.

Helps with knative/eventing-contrib#1482
FYI @devguyio.

@mattmoor
Copy link
Member

mattmoor commented Sep 9, 2020

@grantr was it your intention to still own this given the new self-serve process? I've updated the issue above to reflect the new process.

@evankanderson
Copy link
Member

/assign @n3wscott

@mattmoor
Copy link
Member

Needs branch protection and done

@mattmoor
Copy link
Member

/assign @grantr

@grantr
Copy link
Contributor

grantr commented Sep 24, 2020

Branch protection rule created. I'm closing this as I think everything is done, but feel free to reopen if there's more.

@grantr grantr closed this as completed Sep 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants