Skip to content

DNM: Bump keystone yoga test#2251

Closed
seunghun1ee wants to merge 2 commits intostackhpc/yogafrom
bump-keystone-yoga-test
Closed

DNM: Bump keystone yoga test#2251
seunghun1ee wants to merge 2 commits intostackhpc/yogafrom
bump-keystone-yoga-test

Conversation

@seunghun1ee
Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the Keystone image tags in the Kolla configuration and modifies the kayobe dependency in requirements.txt to point to a test branch. The review feedback highlights that the dependency should point to a stable branch or specific commit to ensure environment stability and notes that the removal of a trailing newline is an unnecessary stylistic change.

@@ -93,4 +93,3 @@ prometheus_ceph_mgr_exporter_endpoints:
{% endif %}

#############################################################################
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

This change removes a trailing blank line, which is a purely stylistic modification unrelated to the main goal of the pull request. According to the general rules, such changes should be avoided.

#############################################################################\n
References
  1. Avoid making purely stylistic changes that are unrelated to the main goal of the pull request.

@seunghun1ee seunghun1ee changed the title Bump keystone yoga test DNM: Bump keystone yoga test Apr 1, 2026
@seunghun1ee seunghun1ee force-pushed the bump-keystone-yoga-test branch from 86b40ba to 64e661a Compare April 2, 2026 09:45
@seunghun1ee seunghun1ee closed this Apr 2, 2026
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.

1 participant