Skip to content

Conversation

pratik-mahalle
Copy link
Contributor

Description of Changes

Added a missing step in the manual for building a custom registry to instruct users to first clone the [registry-support](https://github.com/devfile/registry-support) repository and navigate to the build-tools folder before running the build_image.sh script.
This ensures new contributors know where the script is located and can run it successfully.


Related Issue(s)

Fixes: [devfile/registry#<ISSUE_NUMBER>]
(Replace <ISSUE_NUMBER> with the actual issue number you are working on)


Acceptance Criteria


Tests Performed

  • Cloned the registry-support repository locally.
  • Navigated to build-tools and successfully ran ./build_image.sh using a sample registry content directory to confirm the steps work as expected.

How To Test

  1. Open the updated documentation page for building a custom registry.
  2. Follow the new Step 1 (clone registry-support and cd into build-tools).
  3. Run the ./build_image.sh <path-to-registry-content> command.
  4. Verify that the image is built successfully without any missing-file errors.

Notes To Reviewer

This is a documentation-only change aimed at improving clarity for new contributors.
No code, configuration, or functionality is affected.

Signed-off-by: Pratik Mahalle <mahallepratik683@gmail.com>
@openshift-ci openshift-ci bot requested review from elsony and Jdubrick September 20, 2025 13:32
Copy link
Member

@michael-valdron michael-valdron left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution @pratik-mahalle!

I've left a bit of feedback on your initial step addition.

Signed-off-by: Pratik Mahalle <mahallepratik683@gmail.com>
Copy link
Member

@michael-valdron michael-valdron left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Sep 22, 2025
Copy link

openshift-ci bot commented Sep 22, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: michael-valdron, pratik-mahalle

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

The pull request process is described here

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

@michael-valdron
Copy link
Member

/ok-to-test

@michael-valdron
Copy link
Member

/retest

@michael-valdron michael-valdron merged commit 94f380e into devfile:main Sep 23, 2025
11 of 12 checks passed
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.

2 participants