root / tools / cfgupgrade @ f8af377d
History | View | Annotate | Download (20.2 kB)
1 | 0006af7d | Michael Hanselmann | #!/usr/bin/python |
---|---|---|---|
2 | 0006af7d | Michael Hanselmann | # |
3 | 0006af7d | Michael Hanselmann | |
4 | fdb85e3d | Bernardo Dal Seno | # Copyright (C) 2007, 2008, 2009, 2010, 2011, 2012, 2013 Google Inc. |
5 | 0006af7d | Michael Hanselmann | # |
6 | 0006af7d | Michael Hanselmann | # This program is free software; you can redistribute it and/or modify |
7 | 0006af7d | Michael Hanselmann | # it under the terms of the GNU General Public License as published by |
8 | 0006af7d | Michael Hanselmann | # the Free Software Foundation; either version 2 of the License, or |
9 | 0006af7d | Michael Hanselmann | # (at your option) any later version. |
10 | 0006af7d | Michael Hanselmann | # |
11 | 0006af7d | Michael Hanselmann | # This program is distributed in the hope that it will be useful, but |
12 | 0006af7d | Michael Hanselmann | # WITHOUT ANY WARRANTY; without even the implied warranty of |
13 | 0006af7d | Michael Hanselmann | # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU |
14 | 0006af7d | Michael Hanselmann | # General Public License for more details. |
15 | 0006af7d | Michael Hanselmann | # |
16 | 0006af7d | Michael Hanselmann | # You should have received a copy of the GNU General Public License |
17 | 0006af7d | Michael Hanselmann | # along with this program; if not, write to the Free Software |
18 | 0006af7d | Michael Hanselmann | # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA |
19 | 0006af7d | Michael Hanselmann | # 02110-1301, USA. |
20 | 0006af7d | Michael Hanselmann | |
21 | 0006af7d | Michael Hanselmann | |
22 | 0006af7d | Michael Hanselmann | """Tool to upgrade the configuration file. |
23 | 0006af7d | Michael Hanselmann | |
24 | a421fdeb | Iustin Pop | This code handles only the types supported by simplejson. As an |
25 | a421fdeb | Iustin Pop | example, 'set' is a 'list'. |
26 | 0006af7d | Michael Hanselmann | |
27 | 0006af7d | Michael Hanselmann | """ |
28 | 0006af7d | Michael Hanselmann | |
29 | 0006af7d | Michael Hanselmann | |
30 | 0006af7d | Michael Hanselmann | import os |
31 | 0006af7d | Michael Hanselmann | import os.path |
32 | 0006af7d | Michael Hanselmann | import sys |
33 | 0006af7d | Michael Hanselmann | import optparse |
34 | eda37a5a | Michael Hanselmann | import logging |
35 | 7939f60c | Michael Hanselmann | import time |
36 | 7939f60c | Michael Hanselmann | from cStringIO import StringIO |
37 | 0006af7d | Michael Hanselmann | |
38 | 95e4a814 | Michael Hanselmann | from ganeti import constants |
39 | 95e4a814 | Michael Hanselmann | from ganeti import serializer |
40 | 319856a9 | Michael Hanselmann | from ganeti import utils |
41 | f97c7901 | Michael Hanselmann | from ganeti import cli |
42 | a421fdeb | Iustin Pop | from ganeti import bootstrap |
43 | ac4d25b6 | Iustin Pop | from ganeti import config |
44 | 011974df | Michael Hanselmann | from ganeti import netutils |
45 | 09bf5d24 | Michael Hanselmann | from ganeti import pathutils |
46 | 0006af7d | Michael Hanselmann | |
47 | 0006af7d | Michael Hanselmann | |
48 | 319856a9 | Michael Hanselmann | options = None |
49 | 319856a9 | Michael Hanselmann | args = None |
50 | 0006af7d | Michael Hanselmann | |
51 | 6f285030 | Iustin Pop | |
52 | 93fd9bb1 | Iustin Pop | #: Target major version we will upgrade to |
53 | 93fd9bb1 | Iustin Pop | TARGET_MAJOR = 2 |
54 | 93fd9bb1 | Iustin Pop | #: Target minor version we will upgrade to |
55 | 1fe10404 | Klaus Aehlig | TARGET_MINOR = 10 |
56 | 1709435e | Bernardo Dal Seno | #: Target major version for downgrade |
57 | 1709435e | Bernardo Dal Seno | DOWNGRADE_MAJOR = 2 |
58 | 1709435e | Bernardo Dal Seno | #: Target minor version for downgrade |
59 | 1fe10404 | Klaus Aehlig | DOWNGRADE_MINOR = 9 |
60 | 93fd9bb1 | Iustin Pop | |
61 | 93fd9bb1 | Iustin Pop | |
62 | 319856a9 | Michael Hanselmann | class Error(Exception): |
63 | 319856a9 | Michael Hanselmann | """Generic exception""" |
64 | 319856a9 | Michael Hanselmann | pass |
65 | 0006af7d | Michael Hanselmann | |
66 | 0006af7d | Michael Hanselmann | |
67 | eda37a5a | Michael Hanselmann | def SetupLogging(): |
68 | eda37a5a | Michael Hanselmann | """Configures the logging module. |
69 | eda37a5a | Michael Hanselmann | |
70 | eda37a5a | Michael Hanselmann | """ |
71 | eda37a5a | Michael Hanselmann | formatter = logging.Formatter("%(asctime)s: %(message)s") |
72 | eda37a5a | Michael Hanselmann | |
73 | eda37a5a | Michael Hanselmann | stderr_handler = logging.StreamHandler() |
74 | eda37a5a | Michael Hanselmann | stderr_handler.setFormatter(formatter) |
75 | eda37a5a | Michael Hanselmann | if options.debug: |
76 | eda37a5a | Michael Hanselmann | stderr_handler.setLevel(logging.NOTSET) |
77 | eda37a5a | Michael Hanselmann | elif options.verbose: |
78 | eda37a5a | Michael Hanselmann | stderr_handler.setLevel(logging.INFO) |
79 | eda37a5a | Michael Hanselmann | else: |
80 | 011974df | Michael Hanselmann | stderr_handler.setLevel(logging.WARNING) |
81 | eda37a5a | Michael Hanselmann | |
82 | eda37a5a | Michael Hanselmann | root_logger = logging.getLogger("") |
83 | eda37a5a | Michael Hanselmann | root_logger.setLevel(logging.NOTSET) |
84 | eda37a5a | Michael Hanselmann | root_logger.addHandler(stderr_handler) |
85 | eda37a5a | Michael Hanselmann | |
86 | eda37a5a | Michael Hanselmann | |
87 | 011974df | Michael Hanselmann | def CheckHostname(path): |
88 | 011974df | Michael Hanselmann | """Ensures hostname matches ssconf value. |
89 | 011974df | Michael Hanselmann | |
90 | 011974df | Michael Hanselmann | @param path: Path to ssconf file |
91 | 011974df | Michael Hanselmann | |
92 | 011974df | Michael Hanselmann | """ |
93 | 011974df | Michael Hanselmann | ssconf_master_node = utils.ReadOneLineFile(path) |
94 | 011974df | Michael Hanselmann | hostname = netutils.GetHostname().name |
95 | 011974df | Michael Hanselmann | |
96 | 011974df | Michael Hanselmann | if ssconf_master_node == hostname: |
97 | 011974df | Michael Hanselmann | return True |
98 | 011974df | Michael Hanselmann | |
99 | 011974df | Michael Hanselmann | logging.warning("Warning: ssconf says master node is '%s', but this" |
100 | 011974df | Michael Hanselmann | " machine's name is '%s'; this tool must be run on" |
101 | 011974df | Michael Hanselmann | " the master node", ssconf_master_node, hostname) |
102 | 011974df | Michael Hanselmann | return False |
103 | 011974df | Michael Hanselmann | |
104 | 3c286190 | Dimitris Aragiorgis | |
105 | e94fc80c | Bernardo Dal Seno | def _FillIPolicySpecs(default_ipolicy, ipolicy): |
106 | e94fc80c | Bernardo Dal Seno | if "minmax" in ipolicy: |
107 | 41044e04 | Bernardo Dal Seno | for (key, spec) in ipolicy["minmax"][0].items(): |
108 | 41044e04 | Bernardo Dal Seno | for (par, val) in default_ipolicy["minmax"][0][key].items(): |
109 | e94fc80c | Bernardo Dal Seno | if par not in spec: |
110 | e94fc80c | Bernardo Dal Seno | spec[par] = val |
111 | e94fc80c | Bernardo Dal Seno | |
112 | e94fc80c | Bernardo Dal Seno | |
113 | e94fc80c | Bernardo Dal Seno | def UpgradeIPolicy(ipolicy, default_ipolicy, isgroup): |
114 | 0b94cda8 | Bernardo Dal Seno | minmax_keys = ["min", "max"] |
115 | 0b94cda8 | Bernardo Dal Seno | if any((k in ipolicy) for k in minmax_keys): |
116 | 0b94cda8 | Bernardo Dal Seno | minmax = {} |
117 | 0b94cda8 | Bernardo Dal Seno | for key in minmax_keys: |
118 | 0b94cda8 | Bernardo Dal Seno | if key in ipolicy: |
119 | e94fc80c | Bernardo Dal Seno | if ipolicy[key]: |
120 | e94fc80c | Bernardo Dal Seno | minmax[key] = ipolicy[key] |
121 | 0b94cda8 | Bernardo Dal Seno | del ipolicy[key] |
122 | e94fc80c | Bernardo Dal Seno | if minmax: |
123 | 41044e04 | Bernardo Dal Seno | ipolicy["minmax"] = [minmax] |
124 | e94fc80c | Bernardo Dal Seno | if isgroup and "std" in ipolicy: |
125 | e94fc80c | Bernardo Dal Seno | del ipolicy["std"] |
126 | e94fc80c | Bernardo Dal Seno | _FillIPolicySpecs(default_ipolicy, ipolicy) |
127 | 0b94cda8 | Bernardo Dal Seno | |
128 | 0b94cda8 | Bernardo Dal Seno | |
129 | 58bf877f | Dimitris Aragiorgis | def UpgradeNetworks(config_data): |
130 | 58bf877f | Dimitris Aragiorgis | networks = config_data.get("networks", None) |
131 | 58bf877f | Dimitris Aragiorgis | if not networks: |
132 | 58bf877f | Dimitris Aragiorgis | config_data["networks"] = {} |
133 | 58bf877f | Dimitris Aragiorgis | |
134 | 58bf877f | Dimitris Aragiorgis | |
135 | 0b94cda8 | Bernardo Dal Seno | def UpgradeCluster(config_data): |
136 | 0b94cda8 | Bernardo Dal Seno | cluster = config_data.get("cluster", None) |
137 | 0b94cda8 | Bernardo Dal Seno | if cluster is None: |
138 | 0b94cda8 | Bernardo Dal Seno | raise Error("Cannot find cluster") |
139 | e94fc80c | Bernardo Dal Seno | ipolicy = cluster.setdefault("ipolicy", None) |
140 | 0b94cda8 | Bernardo Dal Seno | if ipolicy: |
141 | e94fc80c | Bernardo Dal Seno | UpgradeIPolicy(ipolicy, constants.IPOLICY_DEFAULTS, False) |
142 | 0b94cda8 | Bernardo Dal Seno | |
143 | 0b94cda8 | Bernardo Dal Seno | |
144 | 58bf877f | Dimitris Aragiorgis | def UpgradeGroups(config_data): |
145 | e94fc80c | Bernardo Dal Seno | cl_ipolicy = config_data["cluster"].get("ipolicy") |
146 | 58bf877f | Dimitris Aragiorgis | for group in config_data["nodegroups"].values(): |
147 | 58bf877f | Dimitris Aragiorgis | networks = group.get("networks", None) |
148 | 58bf877f | Dimitris Aragiorgis | if not networks: |
149 | 58bf877f | Dimitris Aragiorgis | group["networks"] = {} |
150 | 0b94cda8 | Bernardo Dal Seno | ipolicy = group.get("ipolicy", None) |
151 | 0b94cda8 | Bernardo Dal Seno | if ipolicy: |
152 | e94fc80c | Bernardo Dal Seno | if cl_ipolicy is None: |
153 | e94fc80c | Bernardo Dal Seno | raise Error("A group defines an instance policy but there is no" |
154 | e94fc80c | Bernardo Dal Seno | " instance policy at cluster level") |
155 | e94fc80c | Bernardo Dal Seno | UpgradeIPolicy(ipolicy, cl_ipolicy, True) |
156 | 58bf877f | Dimitris Aragiorgis | |
157 | 011974df | Michael Hanselmann | |
158 | c69b147d | Bernardo Dal Seno | def GetExclusiveStorageValue(config_data): |
159 | c69b147d | Bernardo Dal Seno | """Return a conservative value of the exclusive_storage flag. |
160 | c69b147d | Bernardo Dal Seno | |
161 | c69b147d | Bernardo Dal Seno | Return C{True} if the cluster or at least a nodegroup have the flag set. |
162 | c69b147d | Bernardo Dal Seno | |
163 | c69b147d | Bernardo Dal Seno | """ |
164 | c69b147d | Bernardo Dal Seno | ret = False |
165 | c69b147d | Bernardo Dal Seno | cluster = config_data["cluster"] |
166 | c69b147d | Bernardo Dal Seno | ndparams = cluster.get("ndparams") |
167 | c69b147d | Bernardo Dal Seno | if ndparams is not None and ndparams.get("exclusive_storage"): |
168 | c69b147d | Bernardo Dal Seno | ret = True |
169 | c69b147d | Bernardo Dal Seno | for group in config_data["nodegroups"].values(): |
170 | c69b147d | Bernardo Dal Seno | ndparams = group.get("ndparams") |
171 | c69b147d | Bernardo Dal Seno | if ndparams is not None and ndparams.get("exclusive_storage"): |
172 | c69b147d | Bernardo Dal Seno | ret = True |
173 | c69b147d | Bernardo Dal Seno | return ret |
174 | c69b147d | Bernardo Dal Seno | |
175 | c69b147d | Bernardo Dal Seno | |
176 | f032d55c | Dimitris Aragiorgis | def UpgradeInstances(config_data): |
177 | f032d55c | Dimitris Aragiorgis | network2uuid = dict((n["name"], n["uuid"]) |
178 | f032d55c | Dimitris Aragiorgis | for n in config_data["networks"].values()) |
179 | bb553e5a | Bernardo Dal Seno | if "instances" not in config_data: |
180 | bb553e5a | Bernardo Dal Seno | raise Error("Can't find the 'instances' key in the configuration!") |
181 | bb553e5a | Bernardo Dal Seno | |
182 | c69b147d | Bernardo Dal Seno | missing_spindles = False |
183 | bb553e5a | Bernardo Dal Seno | for instance, iobj in config_data["instances"].items(): |
184 | bb553e5a | Bernardo Dal Seno | for nic in iobj["nics"]: |
185 | f032d55c | Dimitris Aragiorgis | name = nic.get("network", None) |
186 | f032d55c | Dimitris Aragiorgis | if name: |
187 | f032d55c | Dimitris Aragiorgis | uuid = network2uuid.get(name, None) |
188 | f032d55c | Dimitris Aragiorgis | if uuid: |
189 | f032d55c | Dimitris Aragiorgis | print("NIC with network name %s found." |
190 | f032d55c | Dimitris Aragiorgis | " Substituting with uuid %s." % (name, uuid)) |
191 | f032d55c | Dimitris Aragiorgis | nic["network"] = uuid |
192 | f8af377d | Dimitris Aragiorgis | try: |
193 | f8af377d | Dimitris Aragiorgis | del nic["idx"] |
194 | f8af377d | Dimitris Aragiorgis | print("Deleting deprecated idx") |
195 | f8af377d | Dimitris Aragiorgis | except KeyError: |
196 | f8af377d | Dimitris Aragiorgis | pass |
197 | f032d55c | Dimitris Aragiorgis | |
198 | bb553e5a | Bernardo Dal Seno | if "disks" not in iobj: |
199 | bb553e5a | Bernardo Dal Seno | raise Error("Instance '%s' doesn't have a disks entry?!" % instance) |
200 | bb553e5a | Bernardo Dal Seno | disks = iobj["disks"] |
201 | bb553e5a | Bernardo Dal Seno | for idx, dobj in enumerate(disks): |
202 | bb553e5a | Bernardo Dal Seno | expected = "disk/%s" % idx |
203 | bb553e5a | Bernardo Dal Seno | current = dobj.get("iv_name", "") |
204 | bb553e5a | Bernardo Dal Seno | if current != expected: |
205 | bb553e5a | Bernardo Dal Seno | logging.warning("Updating iv_name for instance %s/disk %s" |
206 | bb553e5a | Bernardo Dal Seno | " from '%s' to '%s'", |
207 | bb553e5a | Bernardo Dal Seno | instance, idx, current, expected) |
208 | bb553e5a | Bernardo Dal Seno | dobj["iv_name"] = expected |
209 | c69b147d | Bernardo Dal Seno | if not "spindles" in dobj: |
210 | c69b147d | Bernardo Dal Seno | missing_spindles = True |
211 | f8af377d | Dimitris Aragiorgis | try: |
212 | f8af377d | Dimitris Aragiorgis | del dobj["idx"] |
213 | f8af377d | Dimitris Aragiorgis | print("Deleting deprecated idx") |
214 | f8af377d | Dimitris Aragiorgis | except KeyError: |
215 | f8af377d | Dimitris Aragiorgis | pass |
216 | f8af377d | Dimitris Aragiorgis | |
217 | f8af377d | Dimitris Aragiorgis | for attr in ("dev_idxs", "hotplug_info", "hotplugs", "pci_reservations"): |
218 | f8af377d | Dimitris Aragiorgis | try: |
219 | f8af377d | Dimitris Aragiorgis | del iobj[attr] |
220 | f8af377d | Dimitris Aragiorgis | print("Deleting deprecated %s" % attr) |
221 | f8af377d | Dimitris Aragiorgis | except KeyError: |
222 | f8af377d | Dimitris Aragiorgis | pass |
223 | c69b147d | Bernardo Dal Seno | |
224 | c69b147d | Bernardo Dal Seno | if GetExclusiveStorageValue(config_data) and missing_spindles: |
225 | c69b147d | Bernardo Dal Seno | # We cannot be sure that the instances that are missing spindles have |
226 | c69b147d | Bernardo Dal Seno | # exclusive storage enabled (the check would be more complicated), so we |
227 | c69b147d | Bernardo Dal Seno | # give a noncommittal message |
228 | c69b147d | Bernardo Dal Seno | logging.warning("Some instance disks could be needing to update the" |
229 | c69b147d | Bernardo Dal Seno | " spindles parameter; you can check by running" |
230 | c69b147d | Bernardo Dal Seno | " 'gnt-cluster verify', and fix any problem with" |
231 | c69b147d | Bernardo Dal Seno | " 'gnt-cluster repair-disk-sizes'") |
232 | bb553e5a | Bernardo Dal Seno | |
233 | bb553e5a | Bernardo Dal Seno | |
234 | bb553e5a | Bernardo Dal Seno | def UpgradeRapiUsers(): |
235 | bb553e5a | Bernardo Dal Seno | if (os.path.isfile(options.RAPI_USERS_FILE_PRE24) and |
236 | bb553e5a | Bernardo Dal Seno | not os.path.islink(options.RAPI_USERS_FILE_PRE24)): |
237 | bb553e5a | Bernardo Dal Seno | if os.path.exists(options.RAPI_USERS_FILE): |
238 | bb553e5a | Bernardo Dal Seno | raise Error("Found pre-2.4 RAPI users file at %s, but another file" |
239 | bb553e5a | Bernardo Dal Seno | " already exists at %s" % |
240 | bb553e5a | Bernardo Dal Seno | (options.RAPI_USERS_FILE_PRE24, options.RAPI_USERS_FILE)) |
241 | bb553e5a | Bernardo Dal Seno | logging.info("Found pre-2.4 RAPI users file at %s, renaming to %s", |
242 | bb553e5a | Bernardo Dal Seno | options.RAPI_USERS_FILE_PRE24, options.RAPI_USERS_FILE) |
243 | bb553e5a | Bernardo Dal Seno | if not options.dry_run: |
244 | bb553e5a | Bernardo Dal Seno | utils.RenameFile(options.RAPI_USERS_FILE_PRE24, options.RAPI_USERS_FILE, |
245 | bb553e5a | Bernardo Dal Seno | mkdir=True, mkdir_mode=0750) |
246 | bb553e5a | Bernardo Dal Seno | |
247 | bb553e5a | Bernardo Dal Seno | # Create a symlink for RAPI users file |
248 | bb553e5a | Bernardo Dal Seno | if (not (os.path.islink(options.RAPI_USERS_FILE_PRE24) or |
249 | bb553e5a | Bernardo Dal Seno | os.path.isfile(options.RAPI_USERS_FILE_PRE24)) and |
250 | bb553e5a | Bernardo Dal Seno | os.path.isfile(options.RAPI_USERS_FILE)): |
251 | bb553e5a | Bernardo Dal Seno | logging.info("Creating symlink from %s to %s", |
252 | bb553e5a | Bernardo Dal Seno | options.RAPI_USERS_FILE_PRE24, options.RAPI_USERS_FILE) |
253 | bb553e5a | Bernardo Dal Seno | if not options.dry_run: |
254 | bb553e5a | Bernardo Dal Seno | os.symlink(options.RAPI_USERS_FILE, options.RAPI_USERS_FILE_PRE24) |
255 | bb553e5a | Bernardo Dal Seno | |
256 | bb553e5a | Bernardo Dal Seno | |
257 | bb553e5a | Bernardo Dal Seno | def UpgradeWatcher(): |
258 | bb553e5a | Bernardo Dal Seno | # Remove old watcher state file if it exists |
259 | bb553e5a | Bernardo Dal Seno | if os.path.exists(options.WATCHER_STATEFILE): |
260 | bb553e5a | Bernardo Dal Seno | logging.info("Removing watcher state file %s", options.WATCHER_STATEFILE) |
261 | bb553e5a | Bernardo Dal Seno | if not options.dry_run: |
262 | bb553e5a | Bernardo Dal Seno | utils.RemoveFile(options.WATCHER_STATEFILE) |
263 | bb553e5a | Bernardo Dal Seno | |
264 | bb553e5a | Bernardo Dal Seno | |
265 | bb553e5a | Bernardo Dal Seno | def UpgradeFileStoragePaths(config_data): |
266 | bb553e5a | Bernardo Dal Seno | # Write file storage paths |
267 | bb553e5a | Bernardo Dal Seno | if not os.path.exists(options.FILE_STORAGE_PATHS_FILE): |
268 | bb553e5a | Bernardo Dal Seno | cluster = config_data["cluster"] |
269 | bb553e5a | Bernardo Dal Seno | file_storage_dir = cluster.get("file_storage_dir") |
270 | bb553e5a | Bernardo Dal Seno | shared_file_storage_dir = cluster.get("shared_file_storage_dir") |
271 | bb553e5a | Bernardo Dal Seno | del cluster |
272 | bb553e5a | Bernardo Dal Seno | |
273 | bb553e5a | Bernardo Dal Seno | logging.info("Ganeti 2.7 and later only allow whitelisted directories" |
274 | bb553e5a | Bernardo Dal Seno | " for file storage; writing existing configuration values" |
275 | bb553e5a | Bernardo Dal Seno | " into '%s'", |
276 | bb553e5a | Bernardo Dal Seno | options.FILE_STORAGE_PATHS_FILE) |
277 | bb553e5a | Bernardo Dal Seno | |
278 | bb553e5a | Bernardo Dal Seno | if file_storage_dir: |
279 | bb553e5a | Bernardo Dal Seno | logging.info("File storage directory: %s", file_storage_dir) |
280 | bb553e5a | Bernardo Dal Seno | if shared_file_storage_dir: |
281 | bb553e5a | Bernardo Dal Seno | logging.info("Shared file storage directory: %s", |
282 | bb553e5a | Bernardo Dal Seno | shared_file_storage_dir) |
283 | bb553e5a | Bernardo Dal Seno | |
284 | bb553e5a | Bernardo Dal Seno | buf = StringIO() |
285 | bb553e5a | Bernardo Dal Seno | buf.write("# List automatically generated from configuration by\n") |
286 | bb553e5a | Bernardo Dal Seno | buf.write("# cfgupgrade at %s\n" % time.asctime()) |
287 | bb553e5a | Bernardo Dal Seno | if file_storage_dir: |
288 | bb553e5a | Bernardo Dal Seno | buf.write("%s\n" % file_storage_dir) |
289 | bb553e5a | Bernardo Dal Seno | if shared_file_storage_dir: |
290 | bb553e5a | Bernardo Dal Seno | buf.write("%s\n" % shared_file_storage_dir) |
291 | bb553e5a | Bernardo Dal Seno | utils.WriteFile(file_name=options.FILE_STORAGE_PATHS_FILE, |
292 | bb553e5a | Bernardo Dal Seno | data=buf.getvalue(), |
293 | bb553e5a | Bernardo Dal Seno | mode=0600, |
294 | bb553e5a | Bernardo Dal Seno | dry_run=options.dry_run, |
295 | bb553e5a | Bernardo Dal Seno | backup=True) |
296 | bb553e5a | Bernardo Dal Seno | |
297 | bb553e5a | Bernardo Dal Seno | |
298 | b555101c | Thomas Thrainer | def GetNewNodeIndex(nodes_by_old_key, old_key, new_key_field): |
299 | b555101c | Thomas Thrainer | if old_key not in nodes_by_old_key: |
300 | b555101c | Thomas Thrainer | logging.warning("Can't find node '%s' in configuration, assuming that it's" |
301 | b555101c | Thomas Thrainer | " already up-to-date", old_key) |
302 | b555101c | Thomas Thrainer | return old_key |
303 | b555101c | Thomas Thrainer | return nodes_by_old_key[old_key][new_key_field] |
304 | b555101c | Thomas Thrainer | |
305 | b555101c | Thomas Thrainer | |
306 | b555101c | Thomas Thrainer | def ChangeNodeIndices(config_data, old_key_field, new_key_field): |
307 | b555101c | Thomas Thrainer | def ChangeDiskNodeIndices(disk): |
308 | b555101c | Thomas Thrainer | if disk["dev_type"] in constants.LDS_DRBD: |
309 | b555101c | Thomas Thrainer | for i in range(0, 2): |
310 | b555101c | Thomas Thrainer | disk["logical_id"][i] = GetNewNodeIndex(nodes_by_old_key, |
311 | b555101c | Thomas Thrainer | disk["logical_id"][i], |
312 | b555101c | Thomas Thrainer | new_key_field) |
313 | b555101c | Thomas Thrainer | if "children" in disk: |
314 | b555101c | Thomas Thrainer | for child in disk["children"]: |
315 | b555101c | Thomas Thrainer | ChangeDiskNodeIndices(child) |
316 | b555101c | Thomas Thrainer | |
317 | b555101c | Thomas Thrainer | nodes_by_old_key = {} |
318 | b555101c | Thomas Thrainer | nodes_by_new_key = {} |
319 | b555101c | Thomas Thrainer | for (_, node) in config_data["nodes"].items(): |
320 | b555101c | Thomas Thrainer | nodes_by_old_key[node[old_key_field]] = node |
321 | b555101c | Thomas Thrainer | nodes_by_new_key[node[new_key_field]] = node |
322 | b555101c | Thomas Thrainer | |
323 | b555101c | Thomas Thrainer | config_data["nodes"] = nodes_by_new_key |
324 | b555101c | Thomas Thrainer | |
325 | b555101c | Thomas Thrainer | cluster = config_data["cluster"] |
326 | b555101c | Thomas Thrainer | cluster["master_node"] = GetNewNodeIndex(nodes_by_old_key, |
327 | b555101c | Thomas Thrainer | cluster["master_node"], |
328 | b555101c | Thomas Thrainer | new_key_field) |
329 | b555101c | Thomas Thrainer | |
330 | b555101c | Thomas Thrainer | for inst in config_data["instances"].values(): |
331 | b555101c | Thomas Thrainer | inst["primary_node"] = GetNewNodeIndex(nodes_by_old_key, |
332 | b555101c | Thomas Thrainer | inst["primary_node"], |
333 | b555101c | Thomas Thrainer | new_key_field) |
334 | b555101c | Thomas Thrainer | for disk in inst["disks"]: |
335 | b555101c | Thomas Thrainer | ChangeDiskNodeIndices(disk) |
336 | b555101c | Thomas Thrainer | |
337 | b555101c | Thomas Thrainer | |
338 | 4d33e134 | Thomas Thrainer | def ChangeInstanceIndices(config_data, old_key_field, new_key_field): |
339 | 4d33e134 | Thomas Thrainer | insts_by_old_key = {} |
340 | 4d33e134 | Thomas Thrainer | insts_by_new_key = {} |
341 | 4d33e134 | Thomas Thrainer | for (_, inst) in config_data["instances"].items(): |
342 | 4d33e134 | Thomas Thrainer | insts_by_old_key[inst[old_key_field]] = inst |
343 | 4d33e134 | Thomas Thrainer | insts_by_new_key[inst[new_key_field]] = inst |
344 | 4d33e134 | Thomas Thrainer | |
345 | 4d33e134 | Thomas Thrainer | config_data["instances"] = insts_by_new_key |
346 | 4d33e134 | Thomas Thrainer | |
347 | 4d33e134 | Thomas Thrainer | |
348 | b555101c | Thomas Thrainer | def UpgradeNodeIndices(config_data): |
349 | b555101c | Thomas Thrainer | ChangeNodeIndices(config_data, "name", "uuid") |
350 | b555101c | Thomas Thrainer | |
351 | b555101c | Thomas Thrainer | |
352 | 4d33e134 | Thomas Thrainer | def UpgradeInstanceIndices(config_data): |
353 | 4d33e134 | Thomas Thrainer | ChangeInstanceIndices(config_data, "name", "uuid") |
354 | 4d33e134 | Thomas Thrainer | |
355 | 4d33e134 | Thomas Thrainer | |
356 | bb553e5a | Bernardo Dal Seno | def UpgradeAll(config_data): |
357 | bb553e5a | Bernardo Dal Seno | config_data["version"] = constants.BuildVersion(TARGET_MAJOR, |
358 | bb553e5a | Bernardo Dal Seno | TARGET_MINOR, 0) |
359 | bb553e5a | Bernardo Dal Seno | UpgradeRapiUsers() |
360 | bb553e5a | Bernardo Dal Seno | UpgradeWatcher() |
361 | bb553e5a | Bernardo Dal Seno | UpgradeFileStoragePaths(config_data) |
362 | bb553e5a | Bernardo Dal Seno | UpgradeNetworks(config_data) |
363 | 0b94cda8 | Bernardo Dal Seno | UpgradeCluster(config_data) |
364 | bb553e5a | Bernardo Dal Seno | UpgradeGroups(config_data) |
365 | bb553e5a | Bernardo Dal Seno | UpgradeInstances(config_data) |
366 | b555101c | Thomas Thrainer | UpgradeNodeIndices(config_data) |
367 | 4d33e134 | Thomas Thrainer | UpgradeInstanceIndices(config_data) |
368 | bb553e5a | Bernardo Dal Seno | |
369 | f032d55c | Dimitris Aragiorgis | |
370 | d8819d84 | Sebastian Gebhard | def DowngradeInstances(config_data): |
371 | d8819d84 | Sebastian Gebhard | if "instances" not in config_data: |
372 | d8819d84 | Sebastian Gebhard | raise Error("Cannot find the 'instances' key in the configuration!") |
373 | d8819d84 | Sebastian Gebhard | for (iname, iobj) in config_data["instances"].items(): |
374 | d8819d84 | Sebastian Gebhard | DowngradeNicParamsVLAN(iobj["nics"], iname) |
375 | d8819d84 | Sebastian Gebhard | |
376 | d8819d84 | Sebastian Gebhard | |
377 | d8819d84 | Sebastian Gebhard | def DowngradeNicParamsVLAN(nics, owner): |
378 | d8819d84 | Sebastian Gebhard | for nic in nics: |
379 | d8819d84 | Sebastian Gebhard | vlan = nic["nicparams"].get("vlan", None) |
380 | d8819d84 | Sebastian Gebhard | if vlan: |
381 | d8819d84 | Sebastian Gebhard | logging.warning("Instance with name %s found. Removing VLAN information" |
382 | d8819d84 | Sebastian Gebhard | " %s.", owner, vlan) |
383 | d8819d84 | Sebastian Gebhard | del nic["nicparams"]["vlan"] |
384 | d8819d84 | Sebastian Gebhard | |
385 | d8819d84 | Sebastian Gebhard | |
386 | 1709435e | Bernardo Dal Seno | def DowngradeAll(config_data): |
387 | 1709435e | Bernardo Dal Seno | # Any code specific to a particular version should be labeled that way, so |
388 | 1709435e | Bernardo Dal Seno | # it can be removed when updating to the next version. |
389 | 77018a43 | Michele Tartara | config_data["version"] = constants.BuildVersion(DOWNGRADE_MAJOR, |
390 | 77018a43 | Michele Tartara | DOWNGRADE_MINOR, 0) |
391 | d8819d84 | Sebastian Gebhard | DowngradeInstances(config_data) |
392 | 1709435e | Bernardo Dal Seno | |
393 | 1709435e | Bernardo Dal Seno | |
394 | 6d691282 | Michael Hanselmann | def main(): |
395 | 6d691282 | Michael Hanselmann | """Main program. |
396 | 6d691282 | Michael Hanselmann | |
397 | 6d691282 | Michael Hanselmann | """ |
398 | b459a848 | Andrea Spadaccini | global options, args # pylint: disable=W0603 |
399 | 6d691282 | Michael Hanselmann | |
400 | 0006af7d | Michael Hanselmann | # Option parsing |
401 | 95e4a814 | Michael Hanselmann | parser = optparse.OptionParser(usage="%prog [--debug|--verbose] [--force]") |
402 | 3ccb3a64 | Michael Hanselmann | parser.add_option("--dry-run", dest="dry_run", |
403 | 60edf71e | Michael Hanselmann | action="store_true", |
404 | f4bc1f2c | Michael Hanselmann | help="Try to do the conversion, but don't write" |
405 | f4bc1f2c | Michael Hanselmann | " output file") |
406 | f97c7901 | Michael Hanselmann | parser.add_option(cli.FORCE_OPT) |
407 | eda37a5a | Michael Hanselmann | parser.add_option(cli.DEBUG_OPT) |
408 | 9cdb9578 | Iustin Pop | parser.add_option(cli.VERBOSE_OPT) |
409 | 011974df | Michael Hanselmann | parser.add_option("--ignore-hostname", dest="ignore_hostname", |
410 | 011974df | Michael Hanselmann | action="store_true", default=False, |
411 | 011974df | Michael Hanselmann | help="Don't abort if hostname doesn't match") |
412 | 3ccb3a64 | Michael Hanselmann | parser.add_option("--path", help="Convert configuration in this" |
413 | 09bf5d24 | Michael Hanselmann | " directory instead of '%s'" % pathutils.DATA_DIR, |
414 | 09bf5d24 | Michael Hanselmann | default=pathutils.DATA_DIR, dest="data_dir") |
415 | 7939f60c | Michael Hanselmann | parser.add_option("--confdir", |
416 | 7939f60c | Michael Hanselmann | help=("Use this directory instead of '%s'" % |
417 | 7939f60c | Michael Hanselmann | pathutils.CONF_DIR), |
418 | 7939f60c | Michael Hanselmann | default=pathutils.CONF_DIR, dest="conf_dir") |
419 | 02e1292d | Michael Hanselmann | parser.add_option("--no-verify", |
420 | 02e1292d | Michael Hanselmann | help="Do not verify configuration after upgrade", |
421 | 02e1292d | Michael Hanselmann | action="store_true", dest="no_verify", default=False) |
422 | 1709435e | Bernardo Dal Seno | parser.add_option("--downgrade", |
423 | 1709435e | Bernardo Dal Seno | help="Downgrade to the previous stable version", |
424 | 1709435e | Bernardo Dal Seno | action="store_true", dest="downgrade", default=False) |
425 | 0006af7d | Michael Hanselmann | (options, args) = parser.parse_args() |
426 | 0006af7d | Michael Hanselmann | |
427 | ac4d25b6 | Iustin Pop | # We need to keep filenames locally because they might be renamed between |
428 | ac4d25b6 | Iustin Pop | # versions. |
429 | 0cddd44d | Iustin Pop | options.data_dir = os.path.abspath(options.data_dir) |
430 | ac4d25b6 | Iustin Pop | options.CONFIG_DATA_PATH = options.data_dir + "/config.data" |
431 | ac4d25b6 | Iustin Pop | options.SERVER_PEM_PATH = options.data_dir + "/server.pem" |
432 | ac4d25b6 | Iustin Pop | options.KNOWN_HOSTS_PATH = options.data_dir + "/known_hosts" |
433 | ac4d25b6 | Iustin Pop | options.RAPI_CERT_FILE = options.data_dir + "/rapi.pem" |
434 | b6267745 | Andrea Spadaccini | options.SPICE_CERT_FILE = options.data_dir + "/spice.pem" |
435 | b6267745 | Andrea Spadaccini | options.SPICE_CACERT_FILE = options.data_dir + "/spice-ca.pem" |
436 | fdd9ac5b | Michael Hanselmann | options.RAPI_USERS_FILE = options.data_dir + "/rapi/users" |
437 | fdd9ac5b | Michael Hanselmann | options.RAPI_USERS_FILE_PRE24 = options.data_dir + "/rapi_users" |
438 | 6b7d5878 | Michael Hanselmann | options.CONFD_HMAC_KEY = options.data_dir + "/hmac.key" |
439 | fc0726b9 | Michael Hanselmann | options.CDS_FILE = options.data_dir + "/cluster-domain-secret" |
440 | 011974df | Michael Hanselmann | options.SSCONF_MASTER_NODE = options.data_dir + "/ssconf_master_node" |
441 | a292020f | Michael Hanselmann | options.WATCHER_STATEFILE = options.data_dir + "/watcher.data" |
442 | 7939f60c | Michael Hanselmann | options.FILE_STORAGE_PATHS_FILE = options.conf_dir + "/file-storage-paths" |
443 | ac4d25b6 | Iustin Pop | |
444 | eda37a5a | Michael Hanselmann | SetupLogging() |
445 | eda37a5a | Michael Hanselmann | |
446 | 0006af7d | Michael Hanselmann | # Option checking |
447 | 0006af7d | Michael Hanselmann | if args: |
448 | 95e4a814 | Michael Hanselmann | raise Error("No arguments expected") |
449 | 1709435e | Bernardo Dal Seno | if options.downgrade and not options.no_verify: |
450 | 1709435e | Bernardo Dal Seno | options.no_verify = True |
451 | 0006af7d | Michael Hanselmann | |
452 | 011974df | Michael Hanselmann | # Check master name |
453 | 011974df | Michael Hanselmann | if not (CheckHostname(options.SSCONF_MASTER_NODE) or options.ignore_hostname): |
454 | 011974df | Michael Hanselmann | logging.error("Aborting due to hostname mismatch") |
455 | 011974df | Michael Hanselmann | sys.exit(constants.EXIT_FAILURE) |
456 | 011974df | Michael Hanselmann | |
457 | 319856a9 | Michael Hanselmann | if not options.force: |
458 | 1709435e | Bernardo Dal Seno | if options.downgrade: |
459 | 1709435e | Bernardo Dal Seno | usertext = ("The configuration is going to be DOWNGRADED to version %s.%s" |
460 | 1709435e | Bernardo Dal Seno | " Some configuration data might be removed if they don't fit" |
461 | 1709435e | Bernardo Dal Seno | " in the old format. Please make sure you have read the" |
462 | 1709435e | Bernardo Dal Seno | " upgrade notes (available in the UPGRADE file and included" |
463 | 1709435e | Bernardo Dal Seno | " in other documentation formats) to understand what they" |
464 | 1709435e | Bernardo Dal Seno | " are. Continue with *DOWNGRADING* the configuration?" % |
465 | 1709435e | Bernardo Dal Seno | (DOWNGRADE_MAJOR, DOWNGRADE_MINOR)) |
466 | 1709435e | Bernardo Dal Seno | else: |
467 | 1709435e | Bernardo Dal Seno | usertext = ("Please make sure you have read the upgrade notes for" |
468 | 1709435e | Bernardo Dal Seno | " Ganeti %s (available in the UPGRADE file and included" |
469 | 1709435e | Bernardo Dal Seno | " in other documentation formats). Continue with upgrading" |
470 | 1709435e | Bernardo Dal Seno | " configuration?" % constants.RELEASE_VERSION) |
471 | f97c7901 | Michael Hanselmann | if not cli.AskUser(usertext): |
472 | a9221f09 | Michael Hanselmann | sys.exit(constants.EXIT_FAILURE) |
473 | 319856a9 | Michael Hanselmann | |
474 | 95e4a814 | Michael Hanselmann | # Check whether it's a Ganeti configuration directory |
475 | ac4d25b6 | Iustin Pop | if not (os.path.isfile(options.CONFIG_DATA_PATH) and |
476 | 30acff6c | Michael Hanselmann | os.path.isfile(options.SERVER_PEM_PATH) and |
477 | ac4d25b6 | Iustin Pop | os.path.isfile(options.KNOWN_HOSTS_PATH)): |
478 | a9221f09 | Michael Hanselmann | raise Error(("%s does not seem to be a Ganeti configuration" |
479 | ac4d25b6 | Iustin Pop | " directory") % options.data_dir) |
480 | 95e4a814 | Michael Hanselmann | |
481 | 7939f60c | Michael Hanselmann | if not os.path.isdir(options.conf_dir): |
482 | 7939f60c | Michael Hanselmann | raise Error("Not a directory: %s" % options.conf_dir) |
483 | 7939f60c | Michael Hanselmann | |
484 | 11c31f5c | Michael Hanselmann | config_data = serializer.LoadJson(utils.ReadFile(options.CONFIG_DATA_PATH)) |
485 | a421fdeb | Iustin Pop | |
486 | 11c31f5c | Michael Hanselmann | try: |
487 | 11c31f5c | Michael Hanselmann | config_version = config_data["version"] |
488 | 11c31f5c | Michael Hanselmann | except KeyError: |
489 | 11c31f5c | Michael Hanselmann | raise Error("Unable to determine configuration version") |
490 | 0006af7d | Michael Hanselmann | |
491 | 11c31f5c | Michael Hanselmann | (config_major, config_minor, config_revision) = \ |
492 | 11c31f5c | Michael Hanselmann | constants.SplitVersion(config_version) |
493 | 319856a9 | Michael Hanselmann | |
494 | 11c31f5c | Michael Hanselmann | logging.info("Found configuration version %s (%d.%d.%d)", |
495 | 11c31f5c | Michael Hanselmann | config_version, config_major, config_minor, config_revision) |
496 | 319856a9 | Michael Hanselmann | |
497 | 11c31f5c | Michael Hanselmann | if "config_version" in config_data["cluster"]: |
498 | 11c31f5c | Michael Hanselmann | raise Error("Inconsistent configuration: found config_version in" |
499 | 11c31f5c | Michael Hanselmann | " configuration file") |
500 | 95e4a814 | Michael Hanselmann | |
501 | 1709435e | Bernardo Dal Seno | # Downgrade to the previous stable version |
502 | 1709435e | Bernardo Dal Seno | if options.downgrade: |
503 | f2e4363c | Michele Tartara | if not ((config_major == TARGET_MAJOR and config_minor == TARGET_MINOR) or |
504 | f2e4363c | Michele Tartara | (config_major == DOWNGRADE_MAJOR and |
505 | f2e4363c | Michele Tartara | config_minor == DOWNGRADE_MINOR)): |
506 | 1709435e | Bernardo Dal Seno | raise Error("Downgrade supported only from the latest version (%s.%s)," |
507 | 1709435e | Bernardo Dal Seno | " found %s (%s.%s.%s) instead" % |
508 | 1709435e | Bernardo Dal Seno | (TARGET_MAJOR, TARGET_MINOR, config_version, config_major, |
509 | 1709435e | Bernardo Dal Seno | config_minor, config_revision)) |
510 | 1709435e | Bernardo Dal Seno | DowngradeAll(config_data) |
511 | 1709435e | Bernardo Dal Seno | |
512 | ea2ee4b0 | Michele Tartara | # Upgrade from 2.{0..7} to 2.9 |
513 | ea2ee4b0 | Michele Tartara | elif config_major == 2 and config_minor in range(0, 10): |
514 | aeb0c953 | Michael Hanselmann | if config_revision != 0: |
515 | a9221f09 | Michael Hanselmann | logging.warning("Config revision is %s, not 0", config_revision) |
516 | bb553e5a | Bernardo Dal Seno | UpgradeAll(config_data) |
517 | 904910c4 | Iustin Pop | |
518 | 93fd9bb1 | Iustin Pop | elif config_major == TARGET_MAJOR and config_minor == TARGET_MINOR: |
519 | a9221f09 | Michael Hanselmann | logging.info("No changes necessary") |
520 | a9221f09 | Michael Hanselmann | |
521 | a9221f09 | Michael Hanselmann | else: |
522 | a9221f09 | Michael Hanselmann | raise Error("Configuration version %d.%d.%d not supported by this tool" % |
523 | a9221f09 | Michael Hanselmann | (config_major, config_minor, config_revision)) |
524 | aeb0c953 | Michael Hanselmann | |
525 | 11c31f5c | Michael Hanselmann | try: |
526 | 11c31f5c | Michael Hanselmann | logging.info("Writing configuration file to %s", options.CONFIG_DATA_PATH) |
527 | 11c31f5c | Michael Hanselmann | utils.WriteFile(file_name=options.CONFIG_DATA_PATH, |
528 | 11c31f5c | Michael Hanselmann | data=serializer.DumpJson(config_data), |
529 | 11c31f5c | Michael Hanselmann | mode=0600, |
530 | 11c31f5c | Michael Hanselmann | dry_run=options.dry_run, |
531 | 11c31f5c | Michael Hanselmann | backup=True) |
532 | a421fdeb | Iustin Pop | |
533 | a421fdeb | Iustin Pop | if not options.dry_run: |
534 | 5ae4945a | Iustin Pop | bootstrap.GenerateClusterCrypto( |
535 | 5ae4945a | Iustin Pop | False, False, False, False, False, |
536 | 5ae4945a | Iustin Pop | nodecert_file=options.SERVER_PEM_PATH, |
537 | 5ae4945a | Iustin Pop | rapicert_file=options.RAPI_CERT_FILE, |
538 | 5ae4945a | Iustin Pop | spicecert_file=options.SPICE_CERT_FILE, |
539 | 5ae4945a | Iustin Pop | spicecacert_file=options.SPICE_CACERT_FILE, |
540 | 5ae4945a | Iustin Pop | hmackey_file=options.CONFD_HMAC_KEY, |
541 | 5ae4945a | Iustin Pop | cds_file=options.CDS_FILE) |
542 | aeb0c953 | Michael Hanselmann | |
543 | a9221f09 | Michael Hanselmann | except Exception: |
544 | 11c31f5c | Michael Hanselmann | logging.critical("Writing configuration failed. It is probably in an" |
545 | 95e4a814 | Michael Hanselmann | " inconsistent state and needs manual intervention.") |
546 | 95e4a814 | Michael Hanselmann | raise |
547 | 0006af7d | Michael Hanselmann | |
548 | ac4d25b6 | Iustin Pop | # test loading the config file |
549 | fdb85e3d | Bernardo Dal Seno | all_ok = True |
550 | 02e1292d | Michael Hanselmann | if not (options.dry_run or options.no_verify): |
551 | ac4d25b6 | Iustin Pop | logging.info("Testing the new config file...") |
552 | ac4d25b6 | Iustin Pop | cfg = config.ConfigWriter(cfg_file=options.CONFIG_DATA_PATH, |
553 | 959b6fe5 | Apollon Oikonomopoulos | accept_foreign=options.ignore_hostname, |
554 | ac4d25b6 | Iustin Pop | offline=True) |
555 | ac4d25b6 | Iustin Pop | # if we reached this, it's all fine |
556 | ac4d25b6 | Iustin Pop | vrfy = cfg.VerifyConfig() |
557 | ac4d25b6 | Iustin Pop | if vrfy: |
558 | ac4d25b6 | Iustin Pop | logging.error("Errors after conversion:") |
559 | ac4d25b6 | Iustin Pop | for item in vrfy: |
560 | 07b8a2b5 | Iustin Pop | logging.error(" - %s", item) |
561 | fdb85e3d | Bernardo Dal Seno | all_ok = False |
562 | fdb85e3d | Bernardo Dal Seno | else: |
563 | fdb85e3d | Bernardo Dal Seno | logging.info("File loaded successfully after upgrading") |
564 | ac4d25b6 | Iustin Pop | del cfg |
565 | ac4d25b6 | Iustin Pop | |
566 | 1709435e | Bernardo Dal Seno | if options.downgrade: |
567 | 1709435e | Bernardo Dal Seno | action = "downgraded" |
568 | 1709435e | Bernardo Dal Seno | out_ver = "%s.%s" % (DOWNGRADE_MAJOR, DOWNGRADE_MINOR) |
569 | 1709435e | Bernardo Dal Seno | else: |
570 | 1709435e | Bernardo Dal Seno | action = "upgraded" |
571 | 1709435e | Bernardo Dal Seno | out_ver = constants.RELEASE_VERSION |
572 | fdb85e3d | Bernardo Dal Seno | if all_ok: |
573 | 1709435e | Bernardo Dal Seno | cli.ToStderr("Configuration successfully %s to version %s.", |
574 | 1709435e | Bernardo Dal Seno | action, out_ver) |
575 | fdb85e3d | Bernardo Dal Seno | else: |
576 | 1709435e | Bernardo Dal Seno | cli.ToStderr("Configuration %s to version %s, but there are errors." |
577 | 1709435e | Bernardo Dal Seno | "\nPlease review the file.", action, out_ver) |
578 | 66a66fa7 | Michael Hanselmann | |
579 | 6d691282 | Michael Hanselmann | |
580 | 6d691282 | Michael Hanselmann | if __name__ == "__main__": |
581 | 6d691282 | Michael Hanselmann | main() |