Skip to content

Fixed htx ssl issue while accessing the HTX RPM#2935

Closed
SamirMulani wants to merge 1 commit intoavocado-framework-tests:masterfrom
SamirMulani:htx-sslissue-fix
Closed

Fixed htx ssl issue while accessing the HTX RPM#2935
SamirMulani wants to merge 1 commit intoavocado-framework-tests:masterfrom
SamirMulani:htx-sslissue-fix

Conversation

@SamirMulani
Copy link
Contributor

Due to an SSL issue on the LPAR, the installation of the HTX RPM using rpm -ivh is failing.
To resolve this, we have implemented a curl-based
approach to download and install the HTX RPM packages

Due to an SSL issue on the LPAR, the installation of
the HTX RPM using rpm -ivh is failing.
To resolve this, we have implemented a curl-based
approach to download and install the HTX RPM packages

Signed-off-by: Samir Mulani<samir@linux.vnet.ibm.com>

Fixed pycodestyle warnings.

Ran the below command to resolve the pycodestyle issues.

Signed-off-by: “Samir <“samir@linux.vnet.ibm.com”>
'--force' % (self.rpm_link, self.latest_htx_rpm),
htx_latest_rpm = distro_specific_htx_versions[0]
self.latest_htx_rpm = htx_latest_rpm
tmp_dir = "/tmp/" + htx_latest_rpm
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the variable is slightly confusing, can you please rename it like

tmp_htx_rpm

if process.system(cmd,
shell=True, ignore_status=True):
self.cancel("rpm download failed")
cmd = "rpm -ivh %s" % (tmp_dir)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

even have --nodeps in safer side

if htxd_pid:
self.log.info("HTXD is already running with PID: %s. Killing it.", htxd_pid)
self.log.info(
"HTXD is already running with PID: %s. Killing it.", htxd_pid)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why this change ?

@SamirMulani SamirMulani closed this by deleting the head repository Mar 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants