Skip to content
This repository has been archived by the owner on Apr 4, 2023. It is now read-only.

Merge master into Cassandra feature branch #101

Closed
wants to merge 10 commits into from
Closed

Conversation

wallrj
Copy link
Member

@wallrj wallrj commented Nov 7, 2017

Merge latest master, including:

  • RBAC policy to allow navigator to create / update events
  • ElasticSearch pilot RBAC administration.

Release note:

NONE

wallrj and others added 10 commits October 31, 2017 15:22
* Modify RBAC templates to allow navigator controller to create events for
  leader election and for sync messages.
* Add a SuccessSync message to the ElasticSearch control.
* Test for those events.
Automatic merge from submit-queue.

Permit logging of events when RBAC is enabled

* Modify RBAC templates to allow navigator controller to create events for
  leader election and for sync messages.
* Add a SuccessSync message to the ElasticSearch control.
* Test for those events.

Fixes: #89 

**Release note**:
```release-note
NONE
```
Automatic merge from submit-queue.

Only run Travis-CI tests for PRs and when master branch changes

Fixes: #76 

This PR should only run the Travis-CI tests once, for the PR not for the branch.

Should speed up the time it takes to approve and merge small PRs

**Release note**:
```release-note
NONE
```
Automatic merge from submit-queue.

Allow controller to patch events

Fixes #95

```release-note
NONE
```
Automatic merge from submit-queue.

Elasticsearch Pilot RBAC policies

**What this PR does / why we need it**:

Creates RBAC policies for created Pilot resources

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #

Fixes #92 

**Release note**:
```release-note
Automatically create RBAC policies for Pilots
```
@wallrj
Copy link
Member Author

wallrj commented Nov 7, 2017

/lgtm
/approve

@jetstack-bot
Copy link
Collaborator

@wallrj: you cannot LGTM your own PR.

In response to this:

/lgtm
/approve

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@jetstack-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: wallrj

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these OWNERS Files:

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@munnerz
Copy link
Contributor

munnerz commented Nov 7, 2017

Ah - it appears the submit queue is blocked because the continuous-integration/travis-ci/pr job isn't being run. I'm not 100% sure, but I think this is because of the change to the travis file. I'm going to create a PR that reverts that and we'll see if this PR suddenly goes green :)

@jetstack-bot
Copy link
Collaborator

@wallrj: The following test failed, say /retest to rerun them all:

Test name Commit Details Rerun command
navigator-quick-verify d34a530 link /test verify

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@wallrj
Copy link
Member Author

wallrj commented Nov 7, 2017

Trying a PR from my fork instead: #103

@wallrj wallrj closed this Nov 7, 2017
jetstack-bot added a commit that referenced this pull request Nov 7, 2017
…and-PRs

Automatic merge from submit-queue.

Revert "Only run Travis-CI tests for PRs and when master branch changes"

Reverts #77

It appears we're not always running the `/pr` test any more (see #101). I'm reverting this change as I've not had this problem before we added this section.

```release-note
NONE
```
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants