Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit 55d331f

Browse files
committedJan 17, 2017
Update to StatefulSet after 3.5 rebase, fix use of valueFrom for env vars on statefulSets
1 parent ce591fa commit 55d331f

File tree

5 files changed

+190
-158
lines changed

5 files changed

+190
-158
lines changed
 

‎app/scripts/controllers/deployments.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -66,9 +66,9 @@ angular.module('openshiftConsole')
6666
// TODO: keep petset/stateful set here or give it its own page?
6767
// I lean toward dedicated page.
6868
watches.push(DataService.watch({
69-
resource: 'petsets',
69+
resource: 'statefulsets',
7070
group: 'apps',
71-
version: 'v1alpha1'
71+
version: 'v1beta1'
7272
}, context, function(statefulSets) {
7373
$scope.statefulSets = statefulSets.by('metadata.name');
7474
}));

‎app/scripts/controllers/statefulSet.js

+12-4
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,14 @@ angular
2323
});
2424
$scope.emptyMessage = "Loading...";
2525

26+
var altTextForValueFrom = $filter('altTextForValueFrom');
27+
var updateEnvVars = function(statefulSet) {
28+
_.each(statefulSet.spec.template.spec.containers, function(container) {
29+
_.each(container.env, altTextForValueFrom);
30+
});
31+
return statefulSet;
32+
};
33+
2634
AlertMessageService.getAlerts().forEach(function(alert) {
2735
$scope.alerts[alert.name] = alert.data;
2836
});
@@ -39,9 +47,9 @@ angular
3947
};
4048

4149
var resourceGroupVersion = {
42-
resource: 'petsets',// TODO: statefulSets
50+
resource: 'statefulsets',
4351
group: 'apps',
44-
version: 'v1alpha1'
52+
version: 'v1beta1'
4553
};
4654

4755
ProjectsService
@@ -53,14 +61,14 @@ angular
5361
.get(resourceGroupVersion, $scope.statefulSetName, context)
5462
.then(function(statefulSet) {
5563
angular.extend($scope, {
56-
statefulSet: statefulSet,
64+
statefulSet: updateEnvVars(statefulSet),
5765
projectContext: context,
5866
loaded: true
5967
});
6068
watches.push(DataService.watchObject(resourceGroupVersion, $scope.statefulSetName, context, function(statefulSet) {
6169
angular.extend($scope, {
6270
resourceGroupVersion: resourceGroupVersion,
63-
statefulSet: statefulSet
71+
statefulSet: updateEnvVars(statefulSet)
6472
// TODO: support scaling
6573
// scale: function() {},
6674
// isScalable: function() { return true; }

‎app/scripts/filters/canI.js

-4
Original file line numberDiff line numberDiff line change
@@ -65,10 +65,6 @@ angular
6565
'projects': [
6666
{group: '', resource: 'projects', verbs: ['delete', 'update']}
6767
],
68-
// todo: eliminate when we get the name change to statefulsets
69-
'petsets': [
70-
{group: 'apps', resource: 'petsets', verbs: ['update', 'delete']}
71-
],
7268
'statefulsets': [
7369
{group: 'apps', resource: 'statefulsets', verbs: ['update', 'delete']}
7470
]

‎dist/scripts/scripts.js

+44-34
Original file line numberDiff line numberDiff line change
@@ -362,9 +362,9 @@ reloadOnSearch:!1
362362
}).when("/project/:project/edit/dc/:deploymentconfig", {
363363
templateUrl:"views/edit/deployment-config.html",
364364
controller:"EditDeploymentConfigController"
365-
}).when("/project/:project/browse/petsets/:petset", {
366-
templateUrl:"views/browse/pet-set.html",
367-
controller:"PetSetController",
365+
}).when("/project/:project/browse/stateful-sets/:statefulset", {
366+
templateUrl:"views/browse/stateful-set.html",
367+
controller:"StatefulSetController",
368368
reloadOnSearch:!1
369369
}).when("/project/:project/browse/rs/:replicaSet", {
370370
templateUrl:"views/browse/replica-set.html",
@@ -2243,7 +2243,8 @@ i.segment("images").segmentCoded(h.substring(0, m)).segmentCoded(h.substring(m +
22432243
break;
22442244

22452245
case "PetSet":
2246-
i.segment("petsets").segmentCoded(h);
2246+
case "StatefulSet":
2247+
i.segment("stateful-sets").segmentCoded(h);
22472248
break;
22482249

22492250
case "PersistentVolumeClaim":
@@ -5876,11 +5877,11 @@ details:"The active filters are hiding all deployments."
58765877
}) :void delete a.alerts.deployments;
58775878
}
58785879
a.project = c, o.push(e.watch({
5879-
resource:"petsets",
5880+
resource:"statefulsets",
58805881
group:"apps",
5881-
version:"v1alpha1"
5882+
version:"v1beta1"
58825883
}, d, function(b) {
5883-
console.log("petsets?", b.by("metadata.name")), a.petsets = b.by("metadata.name");
5884+
a.statefulSets = b.by("metadata.name");
58845885
})), o.push(e.watch("replicationcontrollers", d, function(c, d, e) {
58855886
a.replicationControllers = c.by("metadata.name");
58865887
var j, k;
@@ -6453,39 +6454,48 @@ e.then(g);
64536454
g.unwatchAll(w);
64546455
});
64556456
}));
6456-
} ]), angular.module("openshiftConsole").controller("PetSetController", [ "$filter", "$scope", "$routeParams", "AlertMessageService", "BreadcrumbsService", "DataService", "ProjectsService", function(a, b, c, d, e, f, g) {
6457-
b.projectName = c.project, b.petSetName = c.petset, b.forms = {}, b.alerts = {}, b.breadcrumbs = e.getBreadcrumbs({
6458-
name:c.petset,
6459-
kind:"PetSet",
6457+
} ]), angular.module("openshiftConsole").controller("StatefulSetController", [ "$filter", "$scope", "$routeParams", "AlertMessageService", "BreadcrumbsService", "DataService", "ProjectsService", function(a, b, c, d, e, f, g) {
6458+
b.projectName = c.project, b.statefulSetName = c.statefulset, b.forms = {}, b.alerts = {}, b.breadcrumbs = e.getBreadcrumbs({
6459+
name:b.statefulSetName,
6460+
kind:"statefulSet",
64606461
namespace:c.project
6461-
}), b.emptyMessage = "Loading...", d.getAlerts().forEach(function(a) {
6462+
}), b.emptyMessage = "Loading...";
6463+
var h = a("altTextForValueFrom"), i = function(a) {
6464+
return _.each(a.spec.template.spec.containers, function(a) {
6465+
_.each(a.env, h);
6466+
}), a;
6467+
};
6468+
d.getAlerts().forEach(function(a) {
64626469
b.alerts[a.name] = a.data;
64636470
}), d.clearAlerts();
6464-
var h, i = [], j = function(a, b) {
6465-
if (a && b) return _.filter(a, function(a) {
6466-
return b.matches(a);
6467-
});
6468-
}, k = {
6469-
resource:"petsets",
6471+
var j, k = [], l = function(a, b) {
6472+
if (a && b) return b.select(a);
6473+
}, m = {
6474+
resource:"statefulsets",
64706475
group:"apps",
6471-
version:"v1alpha1"
6476+
version:"v1beta1"
64726477
};
6473-
g.get(c.project).then(_.spread(function(a, d) {
6474-
h = d, i.push(f.watchObject(k, c.petset, d, function(a) {
6478+
g.get(c.project).then(_.spread(function(a, c) {
6479+
j = c, f.get(m, b.statefulSetName, c).then(function(a) {
64756480
angular.extend(b, {
6476-
resourceGroupVersion:k,
6477-
petSet:a,
6481+
statefulSet:i(a),
6482+
projectContext:c,
64786483
loaded:!0
6479-
});
6480-
var c, e;
6481-
b.$watch("petSet.spec.selector", function() {
6482-
e = new LabelSelector(b.petSet.spec.selector), b.podsForPetSet = j(c, e);
6483-
}, !0), i.push(f.watch("pods", d, function(a) {
6484-
c = a.by("metadata.name"), b.podsForPetSet = j(c, e);
6484+
}), k.push(f.watchObject(m, b.statefulSetName, c, function(a) {
6485+
angular.extend(b, {
6486+
resourceGroupVersion:m,
6487+
statefulSet:i(a)
6488+
});
6489+
var d, e;
6490+
b.$watch("statefulSet.spec.selector", function() {
6491+
e = new LabelSelector(b.statefulSet.spec.selector), b.podsForStatefulSet = l(d, e);
6492+
}, !0), k.push(f.watch("pods", c, function(a) {
6493+
d = a.by("metadata.name"), b.podsForStatefulSet = l(d, e);
64856494
}));
64866495
}));
6496+
});
64876497
})), b.$on("$destroy", function() {
6488-
f.unwatchAll(i);
6498+
f.unwatchAll(k);
64896499
});
64906500
} ]), angular.module("openshiftConsole").controller("ServicesController", [ "$routeParams", "$scope", "AlertMessageService", "DataService", "ProjectsService", "$filter", "LabelFilter", "Logger", function(a, b, c, d, e, f, g, h) {
64916501
b.projectName = a.project, b.services = {}, b.unfilteredServices = {}, b.routesByService = {}, b.routes = {}, b.labelSuggestions = {}, b.alerts = b.alerts || {}, b.emptyMessage = "Loading...", b.emptyMessageRoutes = "Loading...", c.getAlerts().forEach(function(a) {
@@ -14247,14 +14257,14 @@ group:"",
1424714257
resource:"projects",
1424814258
verbs:[ "delete", "update" ]
1424914259
} ],
14250-
petsets:[ {
14260+
statefulsets:[ {
1425114261
group:"apps",
14252-
resource:"petsets",
14253-
verbs:[ "create", "update", "delete" ]
14262+
resource:"statefulsets",
14263+
verbs:[ "update", "delete" ]
1425414264
} ]
1425514265
};
1425614266
return function(c) {
14257-
return console.log("canIDoAny", c), _.some(b[c], function(b) {
14267+
return _.some(b[c], function(b) {
1425814268
return _.some(b.verbs, function(c) {
1425914269
return a({
1426014270
resource:b.resource,

0 commit comments

Comments
 (0)
Please sign in to comment.