Skip to content

Conversation

@m-bull
Copy link
Contributor

@m-bull m-bull commented Nov 4, 2025

No description provided.

@m-bull m-bull requested a review from a team as a code owner November 4, 2025 14:32
Copy link

@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 container image tag to address a security vulnerability (bug 2119646), which allows unauthenticated access to EC2/S3 token endpoints. A release note has been added to document this security fix. The changes are correct, but I have one minor suggestion regarding the ordering of services in the YAML configuration file for better maintainability.

Comment on lines +24 to +26
keystone:
rocky-9: zed-rocky-9-20251031T100240
ubuntu-jammy: zed-ubuntu-jammy-20251031T100240

Choose a reason for hiding this comment

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

medium

To maintain consistency and readability, the services in this list should be kept in alphabetical order. The keystone block should be placed after the horizon block.

@bbezak bbezak merged commit ec46618 into stackhpc/zed Nov 4, 2025
10 of 12 checks passed
@bbezak bbezak deleted the zed-bump-images-20251031 branch November 4, 2025 14:47
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