Skip to content
This repository was archived by the owner on Sep 9, 2026. It is now read-only.

Bump osac-fulfillment-service: NO-ISSUE: Remove verbose debug logging from token sources - #147

Closed
osac-dev-bot wants to merge 1 commit into
mainfrom
bump/osac-fulfillment-service/pr-562
Closed

osac-dev-bot wants to merge 1 commit into
mainfrom
bump/osac-fulfillment-service/pr-562

Conversation

@osac-dev-bot

Copy link
Copy Markdown
Contributor

Bumps osac-fulfillment-service submodule.

Original PR: osac-project/fulfillment-service#562
Author: @jhernand
Merge commit: a8632de185342bdafc06fec45350b95f0629cd7c

@openshift-ci-robot

Copy link
Copy Markdown

@osac-dev-bot: This pull request explicitly references no jira issue.

Details

In response to this:

Bumps osac-fulfillment-service submodule.

Original PR: osac-project/fulfillment-service#562
Author: @jhernand
Merge commit: a8632de185342bdafc06fec45350b95f0629cd7c

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci
openshift-ci Bot requested review from danmanor and larsks May 21, 2026 08:22
@coderabbitai

coderabbitai Bot commented May 21, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Enterprise

Run ID: ed952a00-9fec-4cf8-9ad4-62930b711332

📥 Commits

Reviewing files that changed from the base of the PR and between da088d7 and e281463.

📒 Files selected for processing (2)
  • base/kustomization.yaml
  • base/osac-fulfillment-service

Summary by CodeRabbit

  • Chores
    • Updated the fulfillment service to a new version.

Walkthrough

This PR updates the fulfillment-service deployment to a new commit version (a8632de). The Kustomize configuration in base/kustomization.yaml has its image tag override changed from sha-201abdd to sha-a8632de. The base/osac-fulfillment-service submodule pointer is correspondingly advanced from commit 201abdddb2e3ec6b408b0798799b7500a59deafd to commit a8632de185342bdafc06fec45350b95f0629cd7c.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: bumping the osac-fulfillment-service submodule to remove verbose debug logging from token sources.
Description check ✅ Passed The description is related to the changeset, providing context about the submodule bump, upstream PR reference, author, and specific merge commit.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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


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.

jhernand
jhernand approved these changes May 21, 2026
@openshift-ci openshift-ci Bot added the lgtm label May 21, 2026
@openshift-ci

openshift-ci Bot commented May 21, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jhernand, osac-dev-bot

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci

openshift-ci Bot commented May 21, 2026

Copy link
Copy Markdown

@osac-dev-bot: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-vmaas e281463 link true /test e2e-vmaas

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@openshift-ci

openshift-ci Bot commented May 21, 2026

Copy link
Copy Markdown

PR needs rebase.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@omer-vishlitzky

Copy link
Copy Markdown
Contributor

/retest

@omer-vishlitzky

Copy link
Copy Markdown
Contributor

bumped here #163
/close

@openshift-ci openshift-ci Bot closed this May 25, 2026
@openshift-ci

openshift-ci Bot commented May 25, 2026

Copy link
Copy Markdown

@omer-vishlitzky: Closed this PR.

Details

In response to this:

bumped here #163
/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants