Skip to content

Commit cf89d8d

Browse files
committed
milestone for traefik
1 parent 23945e9 commit cf89d8d

File tree

4 files changed

+20
-18
lines changed

4 files changed

+20
-18
lines changed

webui/src/js/utils/ingress-resource-generator.js

Lines changed: 19 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -23,14 +23,7 @@ define(['models/wkt-project', 'js-yaml'],
2323
break;
2424

2525
case 'traefik':
26-
let ingressTraefikMiddlewares = this.createTraefikMiddlewaresAsYaml(route);
27-
let useMiddlewares = false;
28-
if (ingressTraefikMiddlewares) {
29-
lines.push(ingressTraefikMiddlewares, '');
30-
lines.push('---');
31-
useMiddlewares = true;
32-
}
33-
ingressRouteData = this.createTraefikRoutesAsYaml(route, useMiddlewares);
26+
ingressRouteData = this.createTraefikRoutesAsYaml(route);
3427
break;
3528

3629
case 'nginx':
@@ -124,10 +117,9 @@ define(['models/wkt-project', 'js-yaml'],
124117

125118
if (this.isTraefikSSLTerminateAtIngress(item)) {
126119
if (item['isConsoleService'].includes('yes')) {
127-
console.log('at tthere');
128120
result.spec = {
129-
sslRedirct: true,
130121
headers: {
122+
sslRedirect: true,
131123
customRequestHeaders: {
132124
'X-Custom-Request-Header': '',
133125
'X-Forwarded-For': '',
@@ -150,7 +142,14 @@ define(['models/wkt-project', 'js-yaml'],
150142

151143
}
152144

153-
createTraefikRoutesAsYaml(item, useMiddlewares) {
145+
createTraefikRoutesAsYaml(item) {
146+
147+
let ingressTraefikMiddlewares = this.createTraefikMiddlewaresAsYaml(item);
148+
let useMiddlewares = false;
149+
if (ingressTraefikMiddlewares) {
150+
useMiddlewares = true;
151+
}
152+
154153
const namespace = item['targetServiceNameSpace'] || 'default';
155154

156155
const result = {
@@ -198,7 +197,6 @@ define(['models/wkt-project', 'js-yaml'],
198197
result.spec.tls = { secretName: item['tlsSecretName'] };
199198
}
200199
// SSL passthrough
201-
console.log(item);
202200
if (this.project.ingress.specifyIngressTLSSecret.value && this.isTraefikSSLPassThrough(item)) {
203201
const obj = { passthrough: true };
204202
result.spec.tls = [ obj ];
@@ -210,11 +208,18 @@ define(['models/wkt-project', 'js-yaml'],
210208
}
211209

212210
if (useMiddlewares) {
213-
result.spec.routes[0].middleware = item['name'] + '-middleware';
211+
result.spec.routes[0].middlewares = [ {name: item['name'] + '-middleware'} ];
214212
}
215213

216214
this.addAnnotations(result, item);
217-
return jsYaml.dump(result);
215+
216+
let yaml = '';
217+
if (ingressTraefikMiddlewares) {
218+
yaml = ingressTraefikMiddlewares;
219+
yaml += '\n---\n';
220+
}
221+
yaml += jsYaml.dump(result);
222+
return yaml;
218223
}
219224

220225
_createStandardRoutesAsYaml(item) {

webui/src/js/viewModels/ingress-design-view-impl.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -218,8 +218,6 @@ function(i18n, accUtils, ko, ArrayDataProvider, BufferingDataProvider, project,
218218
// no result indicates operation was cancelled
219219
if (result) {
220220
let changed = false;
221-
console.log('after calls');
222-
console.log(result);
223221
project.ingress.ingressRouteKeys.forEach(key => {
224222
if ((key !== 'uid') && result.hasOwnProperty(key)) {
225223
route[key] = result[key];

webui/src/js/viewModels/route-edit-dialog.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -168,7 +168,6 @@ function(accUtils, ko, i18n, project, viewHelper, ArrayDataProvider, BufferingDa
168168
result[propertyName] = property.value;
169169
}
170170
});
171-
console.log(result);
172171

173172
// add annotations if any value has changed
174173
const changedAnnotations = {};

webui/src/test/ingress-definition-test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ describe('ingress-definition', function () {
5757
it('generate traefik ingress', function () {
5858
const yaml = ingressResource.createTraefikRoutesAsYaml(genericIngress);
5959
const json = jsyaml.load(yaml);
60-
expect(json['apiVersion']).to.equal('networking.k8s.io/v1');
60+
expect(json['apiVersion']).to.equal('traefik.containo.us/v1alpha1');
6161
});
6262
});
6363
});

0 commit comments

Comments
 (0)