Fix: Local yum repo not enabled by default in centos8 and fedora35 entrypoints

[sanjay20m]

Summary

This patch fixes an issue in the entrypoint.sh scripts for both CentOS 8 and Fedora 35 where the local YUM repository was not being used by default during test runs.

Issue

Even though the script creates a local YUM repository file (local.repo), it was configured with enabled=0, which means it is disabled by default. While yum-config-manager --enable local-repository was called, this is not always reliable in automated or minimal environments (e.g. CI containers) and may fail silently or be skipped.

As a result, any locally built RPM packages are not actually used during testing, making the local-repository setup ineffective.

Fix

• Changed enabled=0 to enabled=1 directly in the .repo file to ensure it's active by default.
• Set the correct baseurl using the ${LOCAL_REPO_DIRECTORY} environment variable.
• Removed the redundant call to yum-config-manager --enable local-repository.

Impact

• Ensures local RPMs are tested properly in CI.
• Fixes misleading behavior where the script pretended to use the local repo but didn’t.
• Helps catch packaging or integration issues earlier in the CI pipeline.

[copy-pr-bot]

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

[sanjay20m]

any changes needed !!

[Copilot]

Pull Request Overview

This PR fixes a critical issue where local YUM repositories were not being enabled by default in CentOS 8 and Fedora 35 test environments, causing locally built RPM packages to be ignored during testing.

• Changed repository configuration from enabled=0 to enabled=1 to activate local repos by default
• Updated baseurl to use the ${LOCAL_REPO_DIRECTORY} environment variable instead of hardcoded path
• Removed unreliable yum-config-manager --enable command that could fail silently in CI environments

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
tests/release/docker/fedora35/entrypoint.sh	Updated local repo configuration to enable by default and use correct baseurl
tests/release/docker/centos8/entrypoint.sh	Updated local repo configuration to enable by default and use correct baseurl

[ArangoGutierrez]

Hey @sanjay20m thanks for the contribution, could you squash the 2 commits into one, thanks!

[sanjay20m]

Hey @ArangoGutierrez
I'm currently facing some issues squashing the commits locally due to setup constraints. If possible, could you please squash the commits while merging on your end?

Let me know if you'd prefer I keep trying locally — I’ll do my best. Thanks again for your time and help!

[elezar]

We should also update this to use the envvar.

[sanjay20m]

check this out : commit 798726f68fa675f3a34a5e902e3a4206540b758c
!!

[sanjay20m]

Hi @ArangoGutierrez , @elezar

Apologies for the multiple commits made for a single logical change — I encountered some issues and glitches while trying to squash them locally and through GitHub's web interface. I understand the preference for clean commit history, and I truly appreciate your patience.

If possible, could you please squash the commits while merging the PR? I’d be really grateful for your cooperation and understanding.

Thanks again for reviewing and guiding the patch!
Sanjay Jangid

[sanjay20m]

Very sorry, I am trying to fix it, but it is failing due to some issues. Please cooperate...