Skip to content

Conversation

pleshakov
Copy link
Contributor

@pleshakov pleshakov commented Aug 3, 2018

  • minions are only allowed to inherent certain annotations from their
    master

Additional changes:

  • nginx.org/grpc-services is not allowed in the master anymore
  • nginx.org/server-snippets is not allowed in the minion anymore

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that all unit tests pass after adding my changes
  • I have updated necessary documentation
  • I have rebased my branch onto master
  • I will ensure my PR is targeting the master branch and pulling from my branch from my own fork

@pleshakov pleshakov added the enhancement Pull requests for new features/feature enhancements label Aug 3, 2018
Copy link
Contributor

@isaachawley isaachawley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A whitelist for minion inheritance is a better approach 👍

- minions are only allowed to inherent certain annotations from their
master

Additional changes:
- nginx.org/grpc-services is not allowed in the master anymore
- nginx.org/server-snippets is not allowed in the minion anymore
@pleshakov pleshakov force-pushed the mergeable-annotations-inheritance branch from ed83c2a to 3651bdf Compare August 7, 2018 10:50
@pleshakov pleshakov merged commit 42e53c9 into master Aug 7, 2018
@pleshakov pleshakov deleted the mergeable-annotations-inheritance branch August 8, 2018 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Pull requests for new features/feature enhancements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants