Skip to content

Replace Centos-stream image with Fedora#962

Merged
pragya811 merged 19 commits into
redhat-performance:mainfrom
pragya811:jenkins-aws-issue
Dec 10, 2025
Merged

Replace Centos-stream image with Fedora#962
pragya811 merged 19 commits into
redhat-performance:mainfrom
pragya811:jenkins-aws-issue

Conversation

@pragya811

Copy link
Copy Markdown
Member

Type of change

Note: Fill x in []

  • bug
  • enhancement
  • documentation
  • dependencies

Description

Replace centos-stream9 image with Fedora38

For security reasons, all pull requests need to be approved first before running any automated CI

@pragya811 pragya811 self-assigned this Dec 10, 2025
@pragya811 pragya811 added the ok-to-test PR ok to test label Dec 10, 2025
@pragya811 pragya811 requested review from ebattat and inntran December 10, 2025 08:24
@ebattat

ebattat commented Dec 10, 2025

Copy link
Copy Markdown
Member

@pragya811,
Can we add the jenkins docker file into secret, so next time we will change only the secret ?

pipeline {

environment {
    DOCKER_IMAGE = credentials('QUAY_JENKINS_IMAGE')
    QUAY_CLOUD_GOVERNANCE_REPOSITORY = credentials('QUAY_CLOUD_GOVERNANCE_REPOSITORY')
    . . .
}

agent {
    docker {
        label 'cloud-governance-worker'
        image "${DOCKER_IMAGE}"
        args '-u root -v /etc/postfix/main.cf:/etc/postfix/main.cf --privileged'
    }
}

if "/etc/localtime" not in volume_mounts:
volume_mounts.append("/etc/localtime")
cmd = f"podman run --rm {container_name} {generate_env_vars(**kwargs)} {generate_volume_mounts(volume_mounts)} \
cmd = f"podman run --rm --net=\"host\" {container_name} {generate_env_vars(**kwargs)} {generate_volume_mounts(volume_mounts)} \

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Did u verify that it is running properly ?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yes, running as expected

@ebattat ebattat left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/LGTM

@ebattat ebattat added the lgtm LGTM label Dec 10, 2025
@pragya811 pragya811 merged commit 7a64850 into redhat-performance:main Dec 10, 2025
36 checks passed
@github-project-automation github-project-automation Bot moved this from In progress to Done in Cloud-Governance project Dec 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm LGTM ok-to-test PR ok to test

Projects

Development

Successfully merging this pull request may close these issues.

2 participants