Skip to content

Commit b65088d

Browse files
Merge pull request #1400 from thegreyd/rpms_db_data_fix
Fix upstream url for rpms
2 parents e7689d9 + 961e2a9 commit b65088d

File tree

1 file changed

+2
-10
lines changed

1 file changed

+2
-10
lines changed

doozer/doozerlib/cli/rpms.py

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -333,14 +333,6 @@ async def update_konflux_db(runtime, rpm: RPMMetadata, record: dict):
333333

334334
nvr = build["nvr"]
335335

336-
with open(rpm.specfile) as specfile:
337-
rpmspec = specfile.read().splitlines()
338-
try:
339-
commitish = [line for line in rpmspec if "Update to source commit" in line][0].split(" ")[-1].split("/")[-1]
340-
except IndexError:
341-
rpm.logger.warning('Could not determine commitish for rpm %s', rpm.rpm_name)
342-
commitish = ''
343-
344336
build_record = KonfluxBuildRecord(
345337
name=rpm.rpm_name,
346338
group=runtime.group,
@@ -351,8 +343,8 @@ async def update_konflux_db(runtime, rpm: RPMMetadata, record: dict):
351343
arches=rpm.get_arches(),
352344
installed_packages=[],
353345
parent_images=[],
354-
source_repo=convert_remote_git_to_https(rpm.source.git.url),
355-
commitish=commitish,
346+
source_repo=rpm.public_upstream_url,
347+
commitish=rpm.pre_init_sha,
356348
rebase_repo_url=rebase_repo_url,
357349
rebase_commitish=rebase_commitish,
358350
embargoed="p1" in rpm.release.split("."),

0 commit comments

Comments
 (0)