Skip to content

Commit de31383

Browse files
Simon Emmsroboquat
Simon Emms
authored andcommitted
docker-registry - dependent upon external PR
1 parent cc669b9 commit de31383

File tree

4 files changed

+54
-1
lines changed

4 files changed

+54
-1
lines changed

installer/pkg/components/docker-registry/helm.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@ var Helm = common.CompositeHelmFunc(
3131
helm.KeyValue("docker-registry.service.port", strconv.Itoa(common.ProxyContainerHTTPSPort)),
3232
helm.KeyValue("docker-registry.tlsSecretName", BuiltInRegistryCerts),
3333
helm.KeyValue("docker-registry.image.repository", repository),
34+
helm.KeyValue("docker-registry.serviceAccount.name", Component),
3435
}
3536

3637
if len(cfg.Config.ImagePullSecrets) > 0 {

installer/pkg/components/docker-registry/objects.go

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,19 @@ package dockerregistry
66

77
import (
88
"github.com/gitpod-io/gitpod/installer/pkg/common"
9+
"k8s.io/apimachinery/pkg/runtime"
10+
"k8s.io/utils/pointer"
911
)
1012

1113
var Objects = common.CompositeRenderFunc(
1214
certificate,
15+
rolebinding,
1316
secret,
17+
func(ctx *common.RenderContext) ([]runtime.Object, error) {
18+
if !pointer.BoolDeref(ctx.Config.ContainerRegistry.InCluster, false) {
19+
return nil, nil
20+
}
21+
22+
return common.DefaultServiceAccount(Component)(ctx)
23+
},
1424
)
Lines changed: 42 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,42 @@
1+
// Copyright (c) 2021 Gitpod GmbH. All rights reserved.
2+
// Licensed under the GNU Affero General Public License (AGPL).
3+
// See License-AGPL.txt in the project root for license information.
4+
5+
package dockerregistry
6+
7+
import (
8+
"fmt"
9+
"github.com/gitpod-io/gitpod/installer/pkg/common"
10+
rbacv1 "k8s.io/api/rbac/v1"
11+
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
12+
"k8s.io/apimachinery/pkg/runtime"
13+
"k8s.io/utils/pointer"
14+
)
15+
16+
func rolebinding(ctx *common.RenderContext) ([]runtime.Object, error) {
17+
if !pointer.BoolDeref(ctx.Config.ContainerRegistry.InCluster, false) {
18+
return nil, nil
19+
}
20+
21+
return []runtime.Object{
22+
&rbacv1.RoleBinding{
23+
TypeMeta: common.TypeMetaRoleBinding,
24+
ObjectMeta: metav1.ObjectMeta{
25+
Name: Component,
26+
Namespace: ctx.Namespace,
27+
Labels: common.DefaultLabels(Component),
28+
},
29+
RoleRef: rbacv1.RoleRef{
30+
Kind: "ClusterRole",
31+
Name: fmt.Sprintf("%s-ns-psp:restricted-root-user", ctx.Namespace),
32+
APIGroup: "rbac.authorization.k8s.io",
33+
},
34+
Subjects: []rbacv1.Subject{
35+
{
36+
Kind: "ServiceAccount",
37+
Name: Component,
38+
},
39+
},
40+
},
41+
}, nil
42+
}

installer/third_party/charts/docker-registry/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,5 +8,5 @@ name: docker-registry
88
version: 1.0.0
99
dependencies:
1010
- name: docker-registry
11-
version: 1.14.0
11+
version: 1.16.0
1212
repository: https://helm.twun.io

0 commit comments

Comments
 (0)