Skip to content
This repository was archived by the owner on May 1, 2020. It is now read-only.

Set SecurityContexts and ServiceAccounts for Alert and for a new version of BlackDuck#1120

Merged
1 commit merged intoblackducksoftware:masterfrom
mphammer:set-security-contexts
Dec 9, 2019
Merged

Set SecurityContexts and ServiceAccounts for Alert and for a new version of BlackDuck#1120
1 commit merged intoblackducksoftware:masterfrom
mphammer:set-security-contexts

Conversation

@mphammer
Copy link
Contributor

@mphammer mphammer commented Nov 25, 2019 •
edited by msenmurugan
Loading

@mphammer mphammer requested a review from msenmurugan November 25, 2019 20:18
@mphammer mphammer force-pushed the set-security-contexts branch from 05edc9e to 10ce39a Compare November 26, 2019 18:48
@ghost ghost added size/XXL needs-rebase and removed size/XL labels Dec 5, 2019
@mphammer mphammer force-pushed the set-security-contexts branch 3 times, most recently from ef5102e to 814655b Compare December 9, 2019 13:33
@ghost ghost removed the needs-rebase label Dec 9, 2019
@mphammer mphammer force-pushed the set-security-contexts branch 2 times, most recently from 19592f9 to 94b7d0d Compare December 9, 2019 13:44
@mphammer mphammer changed the title [WIP] enable synopsysctl to set security contexts for black duck Set SecurityContexts and ServiceAccounts for Alert and for a new version of BlackDuck Dec 9, 2019
@mphammer mphammer force-pushed the set-security-contexts branch 2 times, most recently from d48e98f to 6289c8d Compare December 9, 2019 19:29
@mphammer mphammer force-pushed the set-security-contexts branch from 6289c8d to 7b52ef7 Compare December 9, 2019 19:39
msenmurugan
msenmurugan approved these changes Dec 9, 2019
View reviewed changes
Copy link
Contributor

@msenmurugan msenmurugan left a comment

Choose a reason for hiding this comment

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

/approve

@msenmurugan
Copy link
Contributor

msenmurugan commented Dec 9, 2019

/lgtm

@ghost ghost assigned msenmurugan Dec 9, 2019
@ghost ghost added the lgtm label Dec 9, 2019
@ghost
Copy link

ghost commented Dec 9, 2019

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mphammer, msenmurugan, yashbhutwala

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

The pull request process is described here

Details Needs approval from an approver in each of these files:
  • OWNERS [mphammer,msenmurugan,yashbhutwala]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ghost ghost merged commit c13ec65 into blackducksoftware:master Dec 9, 2019
@yashbhutwala yashbhutwala mentioned this pull request Jan 10, 2020
Closed
12 tasks
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

2 more reviewers

@yashbhutwala yashbhutwala yashbhutwala approved these changes

@msenmurugan msenmurugan msenmurugan approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@msenmurugan msenmurugan

Labels

approved lgtm size/XXL

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Support arbitrary Security Contexts for GID, UID, and fsGroup for Kubernetes

3 participants

@mphammer @msenmurugan @yashbhutwala