Hi Krish,

Thanks for the updated KIP. a few comments below.

> "These actions can be carried out by the RM or any contributor post the 
> release process."
Maybe as part of the release process, RM can create a JIRA for this
task. This can be taken by RM or any comitter or any contributor (with
some help from commiters to run "Docker Image Preparation via GitHub
Actions:"

> "Perform Docker build tests to ensure image integrity"
Is this using GitHub Actions workflow? or manual testing?

> "The RM will manually raise the final PR to Docker Hub’s official images 
> repository using the contents of the generated file"
 Is it mandatory for RM/comitters to raise the PR to Docker Hub’s
official images repository (or) can it be done by any contributor.

Also I was thinking, once the KIP gets voted, we should try to release
kafka:3.7.0 (or 3.7.1) Docker Official image. This will help us to
validate the process and allow us to fix any changes suggested by
Dockerhub before the 3.8.0 release.


Thanks,

On Mon, Apr 8, 2024 at 2:33 PM Krish Vora <krishvor...@gmail.com> wrote:
>
> Hi Manikumar and Luke.
> Thanks for the questions.
>
> 1. No, the Docker inventory files and configurations will not be the same
> for Open Source Software (OSS) Images and Docker Official Images (DOI).
>
> For OSS images, the Dockerfile located in docker/jvm/dockerfile is
> utilized. This process is integrated with the existing release pipeline as
> outlined in KIP-975
> <https://cwiki.apache.org/confluence/display/KAFKA/KIP-975%3A+Docker+Image+for+Apache+Kafka#KIP975:DockerImageforApacheKafka-Status>,
> where the Kafka URL is provided as a build argument. This method allows for
> building, testing, and releasing OSS images dynamically. The OSS images
> will continue to be released under the standard release process .
>
> In contrast, the release process for DOIs requires providing the Docker Hub
> team with a specific directory for each version release that contains a
> standalone Dockerfile. These Dockerfiles are designed to be
> self-sufficient, hence require hardcoded values instead of relying on build
> arguments. To accommodate this, in our proposed approach, a new directory
> named docker_official_images has been created. This directory contains
> version-specific directories, having Dockerfiles with hardcoded
> configurations for each release, acting as the source of truth for DOI
> releases. The hardcoded dockerfiles will be created using the
> docker/jvm/dockerfile as a template. Thus, as part of post release we will
> be creating a Dockerfile that will be reviewed by the Dockerhub community
> and might need changes as per their review. This approach ensures that DOIs
> are built consistently and meet the specific requirements set by Docker Hub.
>
> 2. Yes Manikumar, transitioning the release of Docker Official Images (DOI)
> to a post-release activity does address the concerns about complicating the
> release process. Initially, we considered incorporating DOI release
> directly into Kafka's release workflow. However, this approach
> significantly increased the RMs workload due to the addition of numerous
> steps, complicating the process. By designating the DOI release as a
> post-release task, we maintain the original release process. This
> adjustment allows for the DOI release to be done after the main release. We
> have revised the KIP to reflect that DOI releases will now occur after the
> main release phase. Please review the updated document and provide any
> feedback you might have.
>
> Thanks,
> Krish.
>
> On Wed, Apr 3, 2024 at 3:35 PM Luke Chen <show...@gmail.com> wrote:
>
> > Hi Krishna,
> >
> > I also have the same question as Manikumar raised:
> > 1. Will the Docker inventory files/etc are the same for OSS Image and
> > Docker Official Images?
> > If no, then why not? Could we make them identical so that we don't have to
> > build 2 images for each release?
> >
> > Thank you.
> > Luke
> >
> > On Wed, Apr 3, 2024 at 12:41 AM Manikumar <manikumar.re...@gmail.com>
> > wrote:
> >
> > > Hi Krishna,
> > >
> > > Thanks for the KIP.
> > >
> > > I think Docker Official Images will be beneficial to the Kafka community.
> > > Few queries below.
> > >
> > > 1. Will the Docker inventory files/etc are the same for OSS Image and
> > > Docker Official Images
> > > 2. I am a bit worried about the new steps to the release process. Maybe
> > we
> > > should consider Docker Official Images release as Post-Release activity.
> > >
> > > Thanks,
> > >
> > > On Fri, Mar 22, 2024 at 3:29 PM Krish Vora <krishvor...@gmail.com>
> > wrote:
> > >
> > > > Hi Hector,
> > > >
> > > > Thanks for reaching out. This KIP builds on top of KIP-975
> > > > <
> > > >
> > >
> > https://cwiki.apache.org/confluence/display/KAFKA/KIP-975%3A+Docker+Image+for+Apache+Kafka
> > > > >
> > > > and
> > > > aims to introduce a JVM-based Docker Official Image (DOI
> > > > <https://docs.docker.com/trusted-content/official-images/>) for Apache
> > > > Kafka that will be visible under Docker Official Images
> > > > <https://hub.docker.com/search?image_filter=official&q=>. Once
> > > implemented
> > > > for Apache Kafka, for each release, there will be one more JVM-based
> > > Docker
> > > > image available to users.
> > > >
> > > > Currently, we already have an OSS sponsored image, which was introduced
> > > via
> > > > KIP-975 (apache/kafka <https://hub.docker.com/r/apache/kafka/tags>)
> > > which
> > > > comes under The Apache Software Foundation <
> > > > https://hub.docker.com/u/apache> in
> > > > Docker Hub. The new Docker Image is the Docker Official Image (DOI),
> > > which
> > > > will be built and maintained by Docker Community.
> > > >
> > > > For example, for a release version like 3.8.0 we will have two JVM
> > based
> > > > docker images:-
> > > >
> > > >    - apache/kafka:3.8.0 (OSS sponsored image)
> > > >    - kafka:3.8.0 (Docker Official image)
> > > >
> > > >
> > > > I have added the same in the KIP too for everyone's reference.
> > > > Thanks,
> > > > Krish.
> > > >
> > > > On Fri, Mar 22, 2024 at 2:50 AM Hector Geraldino (BLOOMBERG/ 919 3RD
> > A) <
> > > > hgerald...@bloomberg.net> wrote:
> > > >
> > > > > Hi,
> > > > >
> > > > > What is the difference between this KIP and KIP-975: Docker Image for
> > > > > Apache Kafka?
> > > > >
> > > > > From: dev@kafka.apache.org At: 03/21/24 07:30:07 UTC-4:00To:
> > > > > dev@kafka.apache.org
> > > > > Subject: [DISCUSS] KIP-1028: Docker Official Image for Apache Kafka
> > > > >
> > > > > Hi everyone,
> > > > >
> > > > > I would like to start the discussion on
> > > > >
> > > > >
> > > >
> > >
> > https://cwiki.apache.org/confluence/display/KAFKA/KIP-1028%3A+Docker+Official+Im
> > > > > age+for+Apache+Kafka
> > > > >  .
> > > > >
> > > > > This KIP aims to introduce JVM based Docker Official Image (DOI) for
> > > > Apache
> > > > > Kafka.
> > > > >
> > > > > Regards,
> > > > > Krish.
> > > > >
> > > > >
> > > > >
> > > >
> > >
> >

Reply via email to