Hi Luiz, I would say that we either include the missing image (which I guess can be generated from the data index code still existing and working here <https://github.com/apache/incubator-kie-kogito-apps/tree/main/data-index/data-index-service/data-index-service-infinispan>) or make the example work using embedded data index (not using the non existing image). I prefer the first one (it looks to me that this was missing when moving the images to tools repo), but the second will work too.
On Fri, Jan 10, 2025 at 8:57 PM Luiz Motta <luizjoaomo...@gmail.com> wrote: > Hi Francisco, > Thanks for the reply. Since we still support it and the image isn't > available on Apache's Docker Hub, I'm unsure whether this is a deliberate > decision not to publish it or if there's a bug somewhere. Can someone > confirm it? > > On Fri, 10 Jan 2025 at 16:29, Francisco Javier Tirado Sarti < > ftira...@redhat.com> wrote: > > > Hi Alex, > > I think we are even, because in a +1 to remove something, I'm expecting > to > > see what we are going to remove exactly (without context, we can even > > conclude you are planning to remove postgresql examples) > > My proposal is to fix what is needed the example (possibly will be enough > > to remove the reference to the image in the docker compose file, so in a > > sense we agree in the need to remove it ;)) > > Infinispan for data index is still supported and unless there is a > separate > > proposal discussing the benefits/cons of removing it, we should not > remove > > it (or people might think we are following a NFRMWF paradigm: "never fix, > > but remove or move when failing") > > > > > > On Fri, Jan 10, 2025 at 7:25 PM Alex Porcelli <a...@porcelli.me> wrote: > > > > > Francisco, it’s expected from a -1 to have a proposal of an > alternative. > > > > > > > > > On Fri, Jan 10, 2025 at 1:21 PM Francisco Javier Tirado Sarti < > > > ftira...@redhat.com> wrote: > > > > > > > -1 to remove it. > > > > > > > > On Fri, Jan 10, 2025 at 6:30 PM Alex Porcelli <a...@porcelli.me> > > wrote: > > > > > > > > > +1 to remove it! > > > > > > > > > > On Fri, Jan 10, 2025 at 12:03 PM Luiz Motta <ljmo...@apache.org> > > > wrote: > > > > > > > > > > > > Hey there, I could see some examples in the > > > > > `incubator-kie-kogito-examples` > > > > > > that uses `infinispan` for `data-index` [1] and references old ` > > > > > > quay.io/kiegroup` <http://quay.io/kiegroup> < > http://quay.io/kiegroup> < > > http://quay.io/kiegroup> <http://quay.io/kiegroup> > > > <http://quay.io/kiegroup> > > > > images ( > > > > > > quay.io/kiegroup/kogito-data-index-infinispan:${KOGITO_VERSION > <http://quay.io/kiegroup/kogito-data-index-infinispan:$%7BKOGITO_VERSION> > > <http://quay.io/kiegroup/kogito-data-index-infinispan:$%7BKOGITO_VERSION > > > > > < > http://quay.io/kiegroup/kogito-data-index-infinispan:$%7BKOGITO_VERSION > > > > > > > < > > http://quay.io/kiegroup/kogito-data-index-infinispan:$%7BKOGITO_VERSION > > > > > > > > > < > > > > http://quay.io/kiegroup/kogito-data-index-infinispan:$%7BKOGITO_VERSION > > > > >) > > > > > . I > > > > > > searched in the Docker hub registry and couldn't find the > > respective > > > > > image > > > > > > [2]. Do we still support `infinispan` for `data-index`? > > Additionally, > > > > > > should we maintain both Infinispan and Postgresql examples [3]? > > > > > > > > > > > > [1] > > > > > > > > > > > > > > > > > > > > > https://github.com/apache/incubator-kie-kogito-examples/blob/main/kogito-quarkus-examples/kogito-travel-agency/extended/docker-compose/docker-compose.yml > > > > > > > > > > > > [2] > > https://hub.docker.com/search?q=incubator-kie-kogito-data-index- > > > > > > > > > > > > [3] > > > > > > > > > > > > > > > > > > > > > https://github.com/apache/incubator-kie-kogito-examples/tree/main/kogito-quarkus-examples/process-usertasks-timer-quarkus-with-console/docker-compose > > > > > > > > > > > --------------------------------------------------------------------- > > > > > To unsubscribe, e-mail: dev-unsubscr...@kie.apache.org > > > > > For additional commands, e-mail: dev-h...@kie.apache.org > > > > > > > > > > > > > > > > > > > >
