Skip to content

merge upstream/master into oadp-dev#30

Open
Joeavaikath wants to merge 10 commits into
migtools:oadp-devfrom
oadp-rebasebot:rebase-bot-oadp-dev
Open

merge upstream/master into oadp-dev#30
Joeavaikath wants to merge 10 commits into
migtools:oadp-devfrom
oadp-rebasebot:rebase-bot-oadp-dev

Conversation

@Joeavaikath

@Joeavaikath Joeavaikath commented Jul 21, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown

Auto-rebase of upstream filebrowser/filebrowser master into oadp-dev.

Created manually after CI run #29866559352 failed to open a PR due to a rebasebot bug (GitHub API returned non-JSON response, crashed before logging the HTTP status).

Posted via Claude Code

Summary by CodeRabbit

  • Documentation
    • Corrected the Docker Bare Alpine Image installation guide to accurately identify the default container user and group IDs as 1000.

hacdias and others added 10 commits July 21, 2026 19:18
@hacdias
docs: fix typo
b7dc392
@oadp-maintainers
merge upstream/master into oadp-dev
93e03a8
@mpryc @oadp-maintainers
UPSTREAM: <carry> Add disableUserProfile branding option
0f496a4 
New config option:
  --branding.disableUserProfile

This option allows to disable User Profile together
with User settings.

Signed-off-by: Michal Pryc <mpryc@redhat.com>
@mpryc @oadp-maintainers
UPSTREAM: <carry> Add defaultLoginUser branding option
86945ea 
New config option:
  --branding.defaultLoginUser "username"

This option allows to provide default Username, which will result
in hiding username field from the welcome page.

Signed-off-by: Michal Pryc <mpryc@redhat.com>
@mpryc @oadp-maintainers
UPSTREAM: <carry> Disable delete and rename from Help
536b223 
Removes Help options "Delete" and "Rename" if the
following permissions are revoked from a particular
user:

  --perm.delete=false
  --perm.rename=false

Signed-off-by: Michal Pryc <mpryc@redhat.com>
@mpryc @oadp-maintainers
UPSTREAM: <carry> Containerfile to be used with UBI
7a16846 
Add multi-stage Containerfile for UBI-based builds.

Signed-off-by: Michal Pryc <mpryc@redhat.com>
@mpryc @oadp-maintainers
UPSTREAM: <carry> Add OWNERS file
13f6f18 
Adds OWNERS file.

Signed-off-by: Michal Pryc <mpryc@redhat.com>
@mpryc @oadp-maintainers
UPSTREAM: <carry> Fix broken rebase of branding options
a4331b4 
Fix downstream changes to match upstream refactored functions.

Signed-off-by: Michal Pryc <mpryc@redhat.com>
@mpryc @oadp-maintainers
UPSTREAM: <carry>: fix github actions
5bacc30 
Disable Validate Title check (semantic commit PR titles are
not used downstream). Fix Go version to 1.25 and add oadp-*
branches to CI triggers.
@oadp-maintainers
Add dependabot config grouping all Go module dependency updates
fb611f6
@openshift-ci
openshift-ci Bot requested review from kaovilai and mpryc July 21, 2026 20:51
@openshift-ci

openshift-ci Bot commented Jul 21, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Joeavaikath

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

@coderabbitai

coderabbitai Bot commented Jul 21, 2026 •
edited
Loading

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 2a30b2a1-81f8-4702-b548-b02909978bb6

📥 Commits

Reviewing files that changed from the base of the PR and between 4765d1d and fb611f6.

📒 Files selected for processing (1)
  • www/docs/installation.md
📝 Walkthrough

Walkthrough

The Docker Bare Alpine Image installation guide replaces an incorrect PID 1000 reference with UID 1000, while retaining GID 1000.

Changes

Installation documentation

Layer / File(s) Summary
Correct container identity reference
www/docs/installation.md		 The Bare Alpine Image section now identifies the default container user with UID 1000 and GID 1000.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects that this PR merges upstream master into oadp-dev.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

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

Reviewers

@kaovilai kaovilai Awaiting requested review from kaovilai
@mpryc mpryc Awaiting requested review from mpryc

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

approved

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@Joeavaikath @hacdias @oadp-maintainers @mpryc