Skip to content

Commit 9e6e3d1

Browse files
committed
Do not release rpm < 8.0
1 parent 4498768 commit 9e6e3d1

File tree

2 files changed

+17
-4
lines changed

2 files changed

+17
-4
lines changed

src/redis_release/bht/behaviours_docker.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -140,7 +140,7 @@ def update(self) -> Status:
140140
result = Status.SUCCESS
141141

142142
if self.log_once("need_to_release", self.package_meta.ephemeral.log_once_flags):
143-
color_open = "" if result == Status.SUCCESS else "yellow"
143+
color_open = "" if result == Status.SUCCESS else "[yellow]"
144144
color_close = "" if result == Status.SUCCESS else "[/]"
145145
self.logger.info(f"{color_open}{self.feedback_message}{color_close}")
146146
return result

src/redis_release/bht/behaviours_rpm.py

Lines changed: 16 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -109,9 +109,22 @@ def update(self) -> Status:
109109
result = Status.FAILURE
110110

111111
if self.release_version is not None:
112-
self.feedback_message = "Need to release RPM"
113-
result = Status.SUCCESS
112+
if self.release_version.major < 8:
113+
self.feedback_message = (
114+
f"Skip release for RPM {str(self.release_version)} < 8.0"
115+
)
116+
result = Status.FAILURE
117+
else:
118+
self.feedback_message = (
119+
f"Need to release RPM version {str(self.release_version)}"
120+
)
121+
result = Status.SUCCESS
122+
else:
123+
self.feedback_message = "Failed to parse release version"
124+
result = Status.FAILURE
114125

115126
if self.log_once("need_to_release", self.package_meta.ephemeral.log_once_flags):
116-
self.logger.info(self.feedback_message)
127+
color_open = "" if result == Status.SUCCESS else "[yellow]"
128+
color_close = "" if result == Status.SUCCESS else "[/]"
129+
self.logger.info(f"{color_open}{self.feedback_message}{color_close}")
117130
return result

0 commit comments

Comments
 (0)