Skip to content

Commit 940df8d

Browse files
committed
Update templated files to rev 718281e (#96)
Automatically created PR based on commit 718281e1f18effe44d05ff211474833f207c0e5b in stackabletech/operator-templating repo. Original commit message: Updated PR Template to include a check of the helm chart (#59) * Updated PR Template to include a check of the helm chart * `make compile-chart` always rebuilds the chart. Co-authored-by: Razvan-Daniel Mihai <84674+razvan@users.noreply.github.com> Co-authored-by: Stacky McStackface <stackable-bot@users.noreply.github.com>
1 parent 14e0964 commit 940df8d

File tree

3 files changed

+124
-8
lines changed

3 files changed

+124
-8
lines changed

.github/pull_request_template.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,5 +10,6 @@
1010
- [ ] Documentation added (or not applicable)
1111
- [ ] Changelog updated (or not applicable)
1212
- [ ] Cargo.toml only contains references to git tags (not specific commits or branches)
13+
- [ ] Helm chart can be installed and deployed operator works (or not applicable)
1314

1415
Once the review is done, comment `bors r+` (or `bors merge`) to merge. [Further information](https://bors.tech/documentation/getting-started/#reviewing-pull-requests)

Makefile

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -34,21 +34,16 @@ compile-chart: version crds config
3434
chart-clean:
3535
rm -rf deploy/helm/hive-operator/configs
3636
rm -rf deploy/helm/hive-operator/crds
37-
rm -rf deploy/helm/hive-operator/templates/crds.yaml
3837

3938
version:
4039
yq eval -i '.version = ${VERSION} | .appVersion = ${VERSION}' deploy/helm/hive-operator/Chart.yaml
4140

42-
config: deploy/helm/hive-operator/configs
43-
44-
deploy/helm/hive-operator/configs:
41+
config:
4542
cp -r deploy/config-spec deploy/helm/hive-operator/configs
4643

47-
crds: deploy/helm/hive-operator/crds/crds.yaml
48-
49-
deploy/helm/hive-operator/crds/crds.yaml:
44+
crds:
5045
mkdir -p deploy/helm/hive-operator/crds
51-
cat deploy/crd/*.yaml | yq eval '.metadata.annotations["helm.sh/resource-policy"]="keep"' - > ${@}
46+
cat deploy/crd/*.yaml | yq eval '.metadata.annotations["helm.sh/resource-policy"]="keep"' - > deploy/helm/hive-operator/crds/crds.yaml
5247

5348
chart-lint: compile-chart
5449
docker run -it -v $(shell pwd):/build/helm-charts -w /build/helm-charts quay.io/helmpack/chart-testing:v3.5.0 ct lint --config deploy/helm/ct.yaml
Lines changed: 120 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,120 @@
1+
version: 0.1.0
2+
spec:
3+
units:
4+
- unit: &unitUri
5+
name: "uri"
6+
regex: "^(([^:/?#]+):)?(//([^/?#]*))?([^?#]*)(\\?([^#]*))?(#(.*))?"
7+
examples:
8+
- "ldap://[2001:db8::7]/c=GB?objectClass?one"
9+
comment: "Specified in https://tools.ietf.org/html/rfc3986#appendix-B"
10+
- unit: &unitDirectory
11+
name: "directory"
12+
regex: "^/|(/[\\w-]+)+$"
13+
examples:
14+
- "/tmp/xyz"
15+
- unit: &unitClassName
16+
name: "classname"
17+
regex: "([a-zA-Z_$][a-zA-Z\\d_$]*\\.)*[a-zA-Z_$][a-zA-Z\\d_$]*"
18+
examples:
19+
- "org.apache.derby.jdbc.EmbeddedDriver"
20+
21+
properties:
22+
- property:
23+
propertyNames:
24+
- name: "javax.jdo.option.ConnectionURL"
25+
kind:
26+
type: "file"
27+
file: "hive-site.xml"
28+
datatype:
29+
type: "string"
30+
# unit: *unitUri
31+
roles:
32+
- name: "metastore"
33+
required: true
34+
asOfVersion: "0.0.0"
35+
description: "JDBC connect string for a JDBC metastore.
36+
To use SSL to encrypt/authenticate the connection, provide database-specific SSL flag in the connection URL.
37+
For example, jdbc:postgresql://myhost/db?ssl=true for postgres database."
38+
- property:
39+
propertyNames:
40+
- name: "javax.jdo.option.ConnectionDriverName"
41+
kind:
42+
type: "file"
43+
file: "hive-site.xml"
44+
datatype:
45+
type: "string"
46+
unit: *unitClassName
47+
roles:
48+
- name: "metastore"
49+
required: true
50+
asOfVersion: "0.0.0"
51+
- property:
52+
propertyNames:
53+
- name: "javax.jdo.option.ConnectionUserName"
54+
kind:
55+
type: "file"
56+
file: "hive-site.xml"
57+
datatype:
58+
type: "string"
59+
roles:
60+
- name: "metastore"
61+
required: true
62+
asOfVersion: "0.0.0"
63+
- property:
64+
propertyNames:
65+
- name: "javax.jdo.option.ConnectionPassword"
66+
kind:
67+
type: "file"
68+
file: "hive-site.xml"
69+
datatype:
70+
type: "string"
71+
roles:
72+
- name: "metastore"
73+
required: true
74+
asOfVersion: "0.0.0"
75+
- property:
76+
propertyNames:
77+
- name: "hive.metastore.port"
78+
kind:
79+
type: "file"
80+
file: "hive-site.xml"
81+
datatype:
82+
type: "integer"
83+
min: "1024"
84+
max: "65535"
85+
defaultValues:
86+
- value: "9083"
87+
roles:
88+
- name: "metastore"
89+
required: true
90+
asOfVersion: "0.0.0"
91+
- property:
92+
propertyNames:
93+
- name: "hive.metastore.metrics.enabled"
94+
kind:
95+
type: "file"
96+
file: "hive-site.xml"
97+
datatype:
98+
type: "bool"
99+
defaultValues:
100+
- value: "false"
101+
roles:
102+
- name: "metastore"
103+
required: false
104+
asOfVersion: "0.0.0"
105+
- property:
106+
propertyNames:
107+
- name: "hive.metastore.warehouse.dir"
108+
kind:
109+
type: "file"
110+
file: "hive-site.xml"
111+
datatype:
112+
type: "string"
113+
unit: *unitUri
114+
defaultValues:
115+
- value: "/user/hive/warehouse"
116+
roles:
117+
- name: "metastore"
118+
required: false
119+
description: "URI of the default location for native tables."
120+
asOfVersion: "0.0.0"

0 commit comments

Comments
 (0)