Skip to content
This repository has been archived by the owner on Feb 22, 2022. It is now read-only.

[incubator/drone] Makes agent + server affinities configurable #3455

Merged
merged 2 commits into from
Jan 30, 2018
Merged

[incubator/drone] Makes agent + server affinities configurable #3455

merged 2 commits into from
Jan 30, 2018

Conversation

fedenusy
Copy link
Contributor

  • Useful to specify anti-affinity between agents.
  • Useful to restrict server to specific nodes for persistent storage.
  • This PR also bumps default agent + server image tags to 0.8.4.

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Jan 26, 2018
@fedenusy
Copy link
Contributor Author

/assign @viglesiasce

@viglesiasce
Copy link
Contributor

/ok-to-test

@k8s-ci-robot k8s-ci-robot removed the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label Jan 30, 2018
@viglesiasce
Copy link
Contributor

/ok-to-test

@viglesiasce
Copy link
Contributor

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jan 30, 2018
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: fedenusy, viglesiasce

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

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 30, 2018
@k8s-ci-robot k8s-ci-robot merged commit 3eb0836 into helm:master Jan 30, 2018
@fedenusy fedenusy deleted the drone-affinities branch January 30, 2018 17:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@viglesiasce viglesiasce

Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@fedenusy @viglesiasce @k8s-ci-robot