Skip to content

Commit 833ced7

Browse files
committed
Fixed logging type
1 parent 16e66a2 commit 833ced7

File tree

11 files changed

+220
-0
lines changed

11 files changed

+220
-0
lines changed
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
---
2+
apiVersion: v1
3+
kind: Namespace
4+
metadata:
5+
name: test-log-app-interface
6+
spec:
7+
finalizers:
8+
- kubernetes
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
---
2+
apiVersion: v1
3+
kind: Secret
4+
metadata:
5+
name: puptoo
6+
namespace: test-log-app-interface
7+
labels:
8+
app: puptoo
9+
ownerReferences:
10+
- apiVersion: cloud.redhat.com/v1alpha1
11+
kind: ClowdApp
12+
name: puptoo
13+
type: Opaque
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,66 @@
1+
---
2+
apiVersion: cloud.redhat.com/v1alpha1
3+
kind: ClowdEnvironment
4+
metadata:
5+
name: test-log-app-interface
6+
spec:
7+
targetNamespace: test-log-app-interface
8+
providers:
9+
web:
10+
port: 8000
11+
mode: operator
12+
metrics:
13+
port: 9000
14+
mode: operator
15+
path: "/metrics"
16+
kafka:
17+
mode: none
18+
db:
19+
mode: none
20+
logging:
21+
mode: app-interface
22+
objectStore:
23+
mode: none
24+
inMemoryDb:
25+
mode: none
26+
resourceDefaults:
27+
limits:
28+
cpu: 400m
29+
memory: 1024Mi
30+
requests:
31+
cpu: 30m
32+
memory: 512Mi
33+
---
34+
apiVersion: cloud.redhat.com/v1alpha1
35+
kind: ClowdApp
36+
metadata:
37+
name: puptoo
38+
namespace: test-log-app-interface
39+
spec:
40+
envName: test-log-app-interface
41+
deployments:
42+
- name: processor
43+
podSpec:
44+
image: quay.io/psav/clowder-hello
45+
env:
46+
- name: ENV_VAR_1
47+
value: env_var_1
48+
- name: ENV_VAR_2
49+
value: env_var_2
50+
webServices:
51+
public:
52+
enabled: true
53+
private:
54+
enabled: true
55+
---
56+
apiVersion: v1
57+
data:
58+
aws_access_key_id: YXdzX2FjY2Vzc19rZXk=
59+
aws_region: dXMtZWFzdA==
60+
aws_secret_access_key: YXdzX3NlY3JldF9rZXk=
61+
log_group_name: dGVzdC1hcHAtaW50ZXJmYWNlLXMz
62+
kind: Secret
63+
metadata:
64+
name: cloudwatch
65+
namespace: test-log-app-interface
66+
type: Opaque
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
---
2+
apiVersion: kuttl.dev/v1beta1
3+
kind: TestStep
4+
commands:
5+
- script: sleep 5
6+
- script: kubectl get secret --namespace=test-log-app-interface puptoo -o json > /tmp/test-log-app-interface
7+
- script: jq -r '.data["cdappconfig.json"]' < /tmp/test-log-app-interface | base64 -d > /tmp/test-log-app-interface-json
8+
9+
- script: jq -r '.logging.type == "cloudwatch"' -e < /tmp/test-log-app-interface-json
10+
- script: jq -r '.logging.cloudwatch.region == "us-east"' -e < /tmp/test-log-app-interface-json
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
---
2+
apiVersion: kuttl.dev/v1beta1
3+
kind: TestStep
4+
delete:
5+
- apiVersion: v1
6+
kind: Namespace
7+
name: test-log-app-interface
8+
- apiVersion: cloud.redhat.com/v1alpha1
9+
kind: ClowdEnvironment
10+
name: test-log-app-interface
11+
- apiVersion: v1
12+
kind: Namespace
13+
name: test-log-app-interface-secret
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
---
2+
apiVersion: v1
3+
kind: Namespace
4+
metadata:
5+
name: test-log-none
6+
spec:
7+
finalizers:
8+
- kubernetes
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
---
2+
apiVersion: v1
3+
kind: Secret
4+
metadata:
5+
name: puptoo
6+
namespace: test-log-none
7+
labels:
8+
app: puptoo
9+
ownerReferences:
10+
- apiVersion: cloud.redhat.com/v1alpha1
11+
kind: ClowdApp
12+
name: puptoo
13+
type: Opaque
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,66 @@
1+
---
2+
apiVersion: cloud.redhat.com/v1alpha1
3+
kind: ClowdEnvironment
4+
metadata:
5+
name: test-log-none
6+
spec:
7+
targetNamespace: test-log-none
8+
providers:
9+
web:
10+
port: 8000
11+
mode: operator
12+
metrics:
13+
port: 9000
14+
mode: operator
15+
path: "/metrics"
16+
kafka:
17+
mode: none
18+
db:
19+
mode: none
20+
logging:
21+
mode: none
22+
objectStore:
23+
mode: none
24+
inMemoryDb:
25+
mode: none
26+
resourceDefaults:
27+
limits:
28+
cpu: 400m
29+
memory: 1024Mi
30+
requests:
31+
cpu: 30m
32+
memory: 512Mi
33+
---
34+
apiVersion: cloud.redhat.com/v1alpha1
35+
kind: ClowdApp
36+
metadata:
37+
name: puptoo
38+
namespace: test-log-none
39+
spec:
40+
envName: test-log-none
41+
deployments:
42+
- name: processor
43+
podSpec:
44+
image: quay.io/psav/clowder-hello
45+
env:
46+
- name: ENV_VAR_1
47+
value: env_var_1
48+
- name: ENV_VAR_2
49+
value: env_var_2
50+
webServices:
51+
public:
52+
enabled: true
53+
private:
54+
enabled: true
55+
---
56+
apiVersion: v1
57+
data:
58+
aws_access_key_id: YXdzX2FjY2Vzc19rZXk=
59+
aws_region: dXMtZWFzdA==
60+
aws_secret_access_key: YXdzX3NlY3JldF9rZXk=
61+
log_group_name: dGVzdC1hcHAtaW50ZXJmYWNlLXMz
62+
kind: Secret
63+
metadata:
64+
name: cloudwatch
65+
namespace: test-log-none
66+
type: Opaque
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
---
2+
apiVersion: kuttl.dev/v1beta1
3+
kind: TestStep
4+
commands:
5+
- script: sleep 5
6+
- script: kubectl get secret --namespace=test-log-none puptoo -o json > /tmp/test-log-none
7+
- script: jq -r '.data["cdappconfig.json"]' < /tmp/test-log-none | base64 -d > /tmp/test-log-none-json
8+
9+
- script: jq -r '.logging.type == "null"' -e < /tmp/test-log-none-json
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
---
2+
apiVersion: kuttl.dev/v1beta1
3+
kind: TestStep
4+
delete:
5+
- apiVersion: v1
6+
kind: Namespace
7+
name: test-log-none
8+
- apiVersion: cloud.redhat.com/v1alpha1
9+
kind: ClowdEnvironment
10+
name: test-log-none
11+
- apiVersion: v1
12+
kind: Namespace
13+
name: test-log-none-secret

controllers/cloud.redhat.com/providers/logging/appinterface.go

+1
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,7 @@ func setCloudwatchSecret(ns string, p *providers.Provider, c *config.LoggingConf
4747
Region: string(secret.Data["aws_region"]),
4848
LogGroup: string(secret.Data["log_group_name"]),
4949
}
50+
c.Type = "cloudwatch"
5051

5152
return nil
5253
}

0 commit comments

Comments
 (0)