Skip to content

update Container-common-scripts#370

Merged
phracek merged 1 commit intomasterfrom
update_container_common-scripts
Mar 19, 2026
Merged

update Container-common-scripts#370
phracek merged 1 commit intomasterfrom
update_container_common-scripts

Conversation

@phracek
Copy link
Copy Markdown
Member

@phracek phracek commented Mar 19, 2026

Deliver fix for copying version during the tests

sclorg/container-common-scripts#422

This is needed to have MD files in downstream GitLab repos

Summary by CodeRabbit

  • Chores
    • Updated internal dependencies to latest versions.

Deliver fix for copying version during the tests

sclorg/container-common-scripts#422

This is needed to have MD files in downstream GitLab repos

Signed-off-by: Petr "Stone" Hracek <phracek@redhat.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 19, 2026

📝 Walkthrough

Walkthrough

The common submodule is updated to reference a new commit, advancing the pointer from 18f00751b5e5cb1b23c040a20bcf3f65aa667c49 to bf3bb466071024741fdf7a19d64fd394aa994a31. No direct code changes are visible in this repository.

Changes

Cohort / File(s) Summary
Submodule Reference Update
common
Updated submodule commit pointer to latest version.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A hop through the commits, a skip and a bound,
The common module's updated, fresh code is found!
From hash to new hash, we leap and we bound,
One line changed, yet improvements abound!

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'update Container-common-scripts' is vague and lacks specificity about what the update addresses or what change was made. Consider using a more descriptive title that explains the purpose of the update, such as 'Fix version copying in tests by updating container-common-scripts' to better convey the intent.
✅ Passed checks (2 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch update_container_common-scripts
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@phracek
Copy link
Copy Markdown
Member Author

phracek commented Mar 19, 2026

[test]

@github-actions
Copy link
Copy Markdown

Pull Request validation

Failed

🔴 Failed or pending statuses:

  • CodeRabbit[pending]

🔴 Review - Missing review from a member (1 required)

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 19, 2026

Testing Farm results

namecomposearchstatusstarted (UTC)timelogs
Fedora - 1.26Fedora-latestx86_64✅ passed19.03.2026 06:38:417min 4stest pipeline
CentOS Stream 9 - 1.22-microCentOS-Stream-9x86_64✅ passed19.03.2026 06:38:409min 54stest pipeline
CentOS Stream 10 - 1.26CentOS-Stream-10x86_64✅ passed19.03.2026 06:38:419min 57stest pipeline
CentOS Stream 9 - 1.26CentOS-Stream-9x86_64✅ passed19.03.2026 06:38:3910min 22stest pipeline
RHEL8 - 1.24RHEL-8.10.0-Nightlyx86_64✅ passed19.03.2026 06:38:3911min 55stest pipeline
RHEL10 - 1.26RHEL-10-Nightlyx86_64✅ passed19.03.2026 06:38:4211min 39stest pipeline
CentOS Stream 9 - 1.24CentOS-Stream-9x86_64✅ passed19.03.2026 06:38:4010min 35stest pipeline
RHEL10 - Unsubscribed host - 1.26RHEL-10-Nightlyx86_64✅ passed19.03.2026 06:38:3911min 17stest pipeline
RHEL8 - 1.22RHEL-8.10.0-Nightlyx86_64✅ passed19.03.2026 06:38:4012min 14stest pipeline
RHEL9 - Unsubscribed host - 1.20RHEL-9.6.0-Nightlyx86_64✅ passed19.03.2026 06:38:4015min 35stest pipeline
RHEL9 - Unsubscribed host - 1.22RHEL-9.6.0-Nightlyx86_64✅ passed19.03.2026 06:38:4014min 57stest pipeline
RHEL9 - 1.26RHEL-9.6.0-Nightlyx86_64✅ passed19.03.2026 06:38:4015min 60stest pipeline
RHEL9 - Unsubscribed host - 1.26RHEL-9.6.0-Nightlyx86_64✅ passed19.03.2026 06:38:4115min 34stest pipeline
RHEL9 - 1.24RHEL-9.6.0-Nightlyx86_64✅ passed19.03.2026 06:38:4117min 35stest pipeline
RHEL9 - 1.22RHEL-9.6.0-Nightlyx86_64✅ passed19.03.2026 06:38:4116min 27stest pipeline
RHEL9 - 1.20RHEL-9.6.0-Nightlyx86_64✅ passed19.03.2026 06:38:4016min 32stest pipeline

@phracek phracek merged commit 9d7aa7a into master Mar 19, 2026
21 checks passed
@phracek phracek deleted the update_container_common-scripts branch March 19, 2026 07:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant