Skip to content

Commit f104368

Browse files
author
Derek Wasinger
committed
secret and config cleanup
1 parent c022195 commit f104368

File tree

3 files changed

+8
-15
lines changed

3 files changed

+8
-15
lines changed

deployment/README.md

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -31,13 +31,12 @@ helm template . \
3131
--values values-dev.yaml \
3232
--set git.uri=https://github.com/rht-labs/open-management-portal-git-api.git \
3333
--set git.ref=master \
34+
--set configFile=<your-config-file> \
3435
--set configRepositoryId=<your-config-repository-id> \
3536
--set deployKey=<your-gitlab-deploy-key-id> \
3637
--set gitLabApiUrl=<your-gitlab-base-url> \
3738
--set gitLabPersonalAccessToken=<your-gitlab-personal-access-token> \
38-
--set jwtVerifyPublicKeyLocation=<your-openid-connect-cert-url> \
39-
--set residenciesParentRepositoryId=<your-gitlab-group-id> \
40-
--set templateRepositoryId=<your-template-repository-id> \
39+
--set engagementsRepositoryId=<your-gitlab-group-id> \
4140
--set trustedClientKey=<your-trusted-client-key> \
4241
| oc apply -f -
4342
```
@@ -48,13 +47,12 @@ It accepts the following variables
4847
|---|---|
4948
| `git.uri` | The HTTPS reference to the repo (your fork!) to build |
5049
| `git.ref` | The branch name to build |
50+
| `configFile` | The path including name of the config file |
5151
| `configRepositoryId` | The GitLab ID of the config repository |
5252
| `deployKey` | The ID of the GitLab deploy key to enable on newly-created repositories |
5353
| `gitLabApiUrl` | The base URL of the GitLab instance to use |
5454
| `gitLabPersonalAccessToken` | The access token to use to auth against GitLab |
55-
| `jwtVerifyPublicKeyLocation` | The URL at which your OpenID Connect (SSO) provider exposes its public key |
56-
| `residenciesParentRepositoryId` | The ID of the GitLab group under which to create new projects |
57-
| `templateRepositoryId` | The ID of the GitLab repository which defines a template to use for creating new repos |
55+
| `engagementsRepositoryId` | The ID of the GitLab group under which to create new projects |
5856
| `trustedClientKey` | [Temporary] Used as a placeholder to authenticate client requests, being replaced by validating JWT tokens against the `jwtVerifyPublicKeyLocation` |
5957

6058
This will spin up all of the usual resources that this service needs in production, plus a `BuildConfig` configured to build it from source from the Git repository specified. To trigger this build, use `oc start-build omp-git-api`.

deployment/templates/gitlab-secret.yaml

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -5,15 +5,11 @@ metadata:
55
name: omp-gitlab-configuration
66
type: Opaque
77
stringData:
8-
CACHE_SERVICE: "{{ .Values.cacheService }}"
9-
CACHE_USE_AUTH: "{{ .Values.cacheUseAuth }}"
8+
CONFIG_FILE: "{{ .Values.configFile }}"
109
CONFIG_REPOSITORY_ID: "{{ .Values.configRepositoryId }}"
1110
DEPLOY_KEY: "{{ .Values.deployKey }}"
1211
GITLAB_API_URL: "{{ .Values.gitLabApiUrl }}"
1312
GITLAB_PERSONAL_ACCESS_TOKEN: "{{ .Values.gitLabPersonalAccessToken }}"
14-
MP_JWT_VERIFY_PUBLICKEY_LOCATION: >-
15-
{{ .Values.jwtVerifyPublicKeyLocation }}
16-
RESIDENCIES_PARENT_REPOSITORIES_ID: "{{ .Values.residenciesParentRepositoryId }}"
17-
TEMPLATE_REPOSITORY_ID: "{{ .Values.templateRepositoryId }}"
13+
ENGAGEMENTS_REPOSITORY_ID: "{{ .Values.engagementsRepositoryId }}"
1814
TRUSTED_CLIENT_KEY: "{{ .Values.trustedClientKey }}"
1915
{{- end }}

deployment/values-dev.yaml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,11 +13,10 @@ git:
1313
uri: https://github.com/rht-labs/open-management-portal-git-api.git
1414
ref: master
1515

16+
configFile: false
1617
configRepositoryId: false
1718
deployKey: false
1819
gitLabApiUrl: false
1920
gitLabPersonalAccessToken: false
20-
jwtVerifyPublicKeyLocation: false
21-
residenciesParentRepositoryId: false
22-
templateRepositoryId: false
21+
engagementsRepositoryId: false
2322
trustedClientKey: false

0 commit comments

Comments
 (0)