Skip to content

S3 Vector Text changes#2738

Merged
openshift-merge-bot[bot] merged 1 commit into
red-hat-storage:masterfrom
Soundarya189r:S3VectorTextUpdates
May 13, 2026
Merged

S3 Vector Text changes#2738
openshift-merge-bot[bot] merged 1 commit into
red-hat-storage:masterfrom
Soundarya189r:S3VectorTextUpdates

Conversation

@Soundarya189r
Copy link
Copy Markdown

@Soundarya189r Soundarya189r commented Apr 29, 2026

@Soundarya189r
Copy link
Copy Markdown
Author

image image image image image

Comment thread locales/en/plugin__odf-console.json Outdated
"A unique name to identify and manage this protection.": "A unique name to identify and manage this protection.",
"A unique name within this vector bucket.": "A unique name within this vector bucket.",
"A vector bucket stores vector indexes for semantic search and similar workloads. You can create one with an Object Bucket Claim or directly through the S3 Vectors API.": "A vector bucket stores vector indexes for semantic search and similar workloads. You can create one with an Object Bucket Claim or directly through the S3 Vectors API.",
"A vector bucket is a cloud storage container that organizes and manages vector datasets (indexes), allowing users to store, retrieve and control access to data efficiently.": "A vector bucket is a cloud storage container that organizes and manages vector datasets (indexes), allowing users to store, retrieve and control access to data efficiently.",
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"A vector bucket is a cloud storage container that organizes and manages vector datasets (indexes), allowing users to store, retrieve and control access to data efficiently.": "A vector bucket is a cloud storage container that organizes and manages vector datasets (indexes), allowing users to store, retrieve and control access to data efficiently.",
"A vector bucket is a storage container that organizes and manages vector datasets (indexes), allowing users to store, retrieve and control access to data efficiently.": "A vector bucket is a storage container that organizes and manages vector datasets (indexes), allowing users to store, retrieve and control access to data efficiently.",

<TextInput
id="vector-bucket-subpath-obc"
type="text"
placeholder="Enter a valid unix path"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

use t...

<TextInput
id="vector-bucket-subpath-s3"
type="text"
placeholder="Enter a valid unix path"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

use t...

Comment on lines +100 to +101
title={t('Buckets')}
helpText={t('View and manage your vector buckets')}
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

what do we show for general S3 ?
both places should match, right ?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

We have this helpText={t('Browse, upload, and manage objects in buckets.')} for S3
@SanjalKatiyar

Comment on lines +159 to +160
<Label color="green">{t('S3 Vector')}</Label>
</Td>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

why ??
we are already in S3 vector tab, why we need this column ?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

also, we should create a new bug and remove "Storage endpoint" column from General S3 table as well... that's redundant as well...

Copy link
Copy Markdown
Author

@Soundarya189r Soundarya189r May 7, 2026

Choose a reason for hiding this comment

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

got it, reverted, I will also raise a bug for S3

@SanjalKatiyar
Copy link
Copy Markdown
Collaborator

/approve
/lgtm

@SanjalKatiyar
Copy link
Copy Markdown
Collaborator

/cherry-pick release-4.22

@openshift-cherrypick-robot
Copy link
Copy Markdown

@SanjalKatiyar: once the present PR merges, I will cherry-pick it on top of release-4.22 in a new PR and assign it to you.

Details

In response to this:

/cherry-pick release-4.22

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.

@Soundarya189r
Copy link
Copy Markdown
Author

/retest-required

@Soundarya189r
Copy link
Copy Markdown
Author

/test odf-console-e2e-aws

@Soundarya189r Soundarya189r force-pushed the S3VectorTextUpdates branch from df84b79 to 94f0549 Compare May 11, 2026 06:08
@openshift-ci openshift-ci Bot removed the lgtm label May 11, 2026
@Soundarya189r
Copy link
Copy Markdown
Author

/cherry-pick release-4.22

@openshift-cherrypick-robot
Copy link
Copy Markdown

@Soundarya189r: once the present PR merges, I will cherry-pick it on top of release-4.22 in a new PR and assign it to you.

Details

In response to this:

/cherry-pick release-4.22

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.

@bipuladh
Copy link
Copy Markdown
Contributor

/lgtm

@Soundarya189r Soundarya189r force-pushed the S3VectorTextUpdates branch from f29e364 to 44e9833 Compare May 12, 2026 10:20
@SanjalKatiyar
Copy link
Copy Markdown
Collaborator

/lgtm

@openshift-ci openshift-ci Bot added the lgtm label May 13, 2026
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 13, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bipuladh, SanjalKatiyar, Soundarya189r

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:
  • OWNERS [SanjalKatiyar,bipuladh]

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

@openshift-merge-bot openshift-merge-bot Bot merged commit a682e41 into red-hat-storage:master May 13, 2026
5 checks passed
@openshift-cherrypick-robot
Copy link
Copy Markdown

@Soundarya189r: new pull request created: #2804

Details

In response to this:

/cherry-pick release-4.22

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 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.

4 participants