m1a2st commented on code in PR #19420:
URL: https://github.com/apache/kafka/pull/19420#discussion_r2034904360


##########
docker/README.md:
##########
@@ -30,8 +30,8 @@ Building image and running tests using github actions
 - kafka-url - This is the url to download kafka tarball from. For example 
kafka tarball url from (https://archive.apache.org/dist/kafka). For building RC 
image this will be an RC tarball url.

Review Comment:
   Could you also update this link to markdown pattern 
   
   `(https://archive.apache.org/dist/kafka)` -> `[Kafka 
archive](https://archive.apache.org/dist/kafka)`



##########
docker/README.md:
##########
@@ -30,8 +30,8 @@ Building image and running tests using github actions
 - kafka-url - This is the url to download kafka tarball from. For example 
kafka tarball url from (https://archive.apache.org/dist/kafka). For building RC 
image this will be an RC tarball url.
 
 - image-type - This is the type of image that we intend to build. This will be 
dropdown menu type selection in the workflow.
-  - `jvm` image type is for official docker image (to be hosted on 
apache/kafka) as described in 
[KIP-975](https://cwiki.apache.org/confluence/display/KAFKA/KIP-975%3A+Docker+Image+for+Apache+Kafka)
-  - `native` image type is for graalvm based `native` kafka docker image (to 
be hosted on apache/kafka-native) as described in 
[KIP-974](https://cwiki.apache.org/confluence/display/KAFKA/KIP-974%3A+Docker+Image+for+GraalVM+based+Native+Kafka+Broker#KIP974:DockerImageforGraalVMbasedNativeKafkaBroker-ImageNaming)
+  - `jvm` image type is for official docker image (to be hosted on 
apache/kafka) as described in 
[KIP-975](https://cwiki.apache.org/confluence/x/z5izDw)
+  - `native` image type is for graalvm based `native` kafka docker image (to 
be hosted on apache/kafka-native) as described in 
[KIP-974](https://cwiki.apache.org/confluence/x/KZizDw#KIP974:DockerImageforGraalVMbasedNativeKafkaBroker-ImageNaming).
 Or you can see [native/README.md](native/README.md) for more information.

Review Comment:
   "kafka docker image" could be "Kafka docker image"



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to