Skip to content

Conversation

@0lai0
Copy link
Contributor

@0lai0 0lai0 commented Dec 4, 2025

What changes were proposed in this pull request?

Update k3s version
update install scripts and doc

What is the link to the Apache JIRA

HDDS-14051

@adoroszlai adoroszlai requested a review from jojochuang December 4, 2025 10:55
Copy link
Contributor

@adoroszlai adoroszlai left a comment

Choose a reason for hiding this comment

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

Thanks @0lai0 for the patch. LGTM.

@jojochuang please take a look

Copy link
Contributor

@jojochuang jojochuang left a comment

Choose a reason for hiding this comment

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

LGTM.
If possible I'd like to include this in Ozone 2.1

# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
apiVersion: storage.k8s.io/v1beta1
Copy link
Contributor

Choose a reason for hiding this comment

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

Warning: rbac.authorization.k8s.io/v1beta1 ClusterRoleBinding is deprecated in v1.17+, unavailable in v1.22+;

https://stackoverflow.com/questions/65068168/rbac-authorization-k8s-io-v1beta1-is-deprecated-warning

@adoroszlai adoroszlai merged commit 13f8a67 into apache:master Dec 4, 2025
17 checks passed
@adoroszlai
Copy link
Contributor

Thanks @0lai0 for the patch, @jojochuang for the review.

If possible I'd like to include this in Ozone 2.1

These are just test/example files. I don't think it's worth rolling another RC for this alone. Helm chart may need to be tweaked similarly, though.

@0lai0
Copy link
Contributor Author

0lai0 commented Dec 5, 2025

@jojochuang @adoroszlai Thanks for review.

@0lai0 0lai0 deleted the HDDS-14051 branch December 8, 2025 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants