Skip to content

Commit 26a06a7

Browse files
durandxkrisstern
andauthored
Fix the save/load cluster configuration on disk and in memory (#233)
Tekton Cluster configuration is not persisted on disk (save/load) and is lost at next reboot. Moreover, configuration update are not properly sotred in memory, preventing users to access to new configuration added (other than the default one). Fixes #106 Co-authored-by: Kris Stern <krisstern@outlook.com>
1 parent e9f8aa6 commit 26a06a7

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

src/main/java/org/waveywaves/jenkins/plugins/tekton/client/global/TektonGlobalConfiguration.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
@Extension
1818
public class TektonGlobalConfiguration extends GlobalConfiguration {
1919
private static final Logger logger = Logger.getLogger(TektonGlobalConfiguration.class.getName());
20-
private transient List<ClusterConfig> clusterConfigs = new ArrayList<>();
20+
private List<ClusterConfig> clusterConfigs = new ArrayList<>();
2121

2222
public TektonGlobalConfiguration(){
2323
load();
@@ -39,7 +39,8 @@ public static TektonGlobalConfiguration get() {
3939
@Override
4040
public boolean configure(final StaplerRequest2 req, final JSONObject formData) {
4141
Jenkins.getInstance().checkPermission(Jenkins.ADMINISTER);
42-
setClusterConfigs(req.bindJSONToList(ClusterConfig.class, formData.get("clusterConfigs")));
42+
setClusterConfigs(req.bindJSONToList(ClusterConfig.class, formData.get("clusterConfigs")));
43+
configChange();
4344
save();
4445
return true;
4546
}

0 commit comments

Comments
 (0)