Skip to content

Commit cc06a5e

Browse files
author
Jacob Truman
committed
Fix issue parsing version
1 parent 51a9522 commit cc06a5e

File tree

1 file changed

+3
-15
lines changed

1 file changed

+3
-15
lines changed

buildrunner/config/loader.py

Lines changed: 3 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -31,11 +31,12 @@
3131
load_config,
3232
)
3333

34+
from buildrunner import __version__ as BUILDRUNNER_VERSION
35+
3436
from .models import GlobalConfig
3537

3638
from . import fetch, jinja_context
3739

38-
3940
MASTER_GLOBAL_CONFIG_FILE = "/etc/buildrunner/buildrunner.yaml"
4041
VERSION_FILE_PATH = (
4142
f"{os.path.realpath(os.path.join(os.path.dirname(__file__), '..'))}/version.py"
@@ -124,27 +125,14 @@ def _validate_version(config: dict) -> None:
124125
buildrunner. If the config version is greater than the buildrunner version or any parsing error occurs
125126
it will raise a buildrunner exception.
126127
"""
127-
buildrunner_version = None
128-
129128
if not os.path.exists(VERSION_FILE_PATH):
130129
LOGGER.warning(
131130
f"File {VERSION_FILE_PATH} does not exist. This could indicate an error with "
132131
f"the buildrunner installation. Unable to validate version."
133132
)
134133
return
135134

136-
with open(VERSION_FILE_PATH, "r", encoding="utf-8") as version_file:
137-
for line in version_file.readlines():
138-
if "__version__" in line:
139-
try:
140-
version_values = (
141-
line.split("=")[1].strip().replace("'", "").split(".")
142-
)
143-
buildrunner_version = f"{version_values[0]}.{version_values[1]}"
144-
except IndexError as exception:
145-
raise ConfigVersionFormatError(
146-
f'couldn\'t parse version from "{line}"'
147-
) from exception
135+
buildrunner_version = ".".join((BUILDRUNNER_VERSION.split(".") + ["0", "0"])[:2])
148136

149137
if not buildrunner_version:
150138
raise BuildRunnerVersionError("unable to determine buildrunner version")

0 commit comments

Comments
 (0)