profile
viewpoint
Ian Partridge ianpartridge IBM UK Senior cloud engineer @IBM.

apple/swift-corelibs-foundation 4001

The Foundation Project, providing core utilities, internationalization, and OS independence

apple/swift-docker 1085

Docker Official Image packaging for Swift

Evolution-App/iOS 209

Unofficial app for Swift Evolution

Evolution-App/Backend 123

Backend is responsible to provide data to EVOlution App - iOS

dokun1/slackin-swift 32

Invite people to your public slack instance - but in Swift!

ianpartridge/coffeeshop-demo 4

OpenLiberty, Kafka and Reactive are ordering serverless coffee with KEDA

appsody/appsody-buildah 1

A docker image with Appsody CLI installed that can be used for running Appsody with buildah in Tekton pipelines.

issue commentredhat-developer/service-binding-operator

Support air gap scenarios

It's the one in the CSV that needs updating to support disconnected installation, I think.

Currently, the CSV's spec.containerImage is quay.io/redhat-developer/app-binding-operator:72abb50. 72abb50 is a tag not a SHA and https://docs.openshift.com/container-platform/4.5/operators/olm-restricted-networks.html states operators must "Reference all specified images by a digest (SHA) and not by a tag."

https://access.redhat.com/articles/4740011 says that SBO currently does not support disconnected mode.

arthurdm

comment created time in 24 days

issue commentredhat-developer/service-binding-operator

Support air gap scenarios

Hi! Is there any update on this? 🙂

arthurdm

comment created time in a month

PR opened rhd-gitops-example/docs

Update README.md

Markdown formatting and typo.

+3 -2

0 comment

1 changed file

pr created time in 2 months

push eventianpartridge/docs-2

Ian Partridge

commit sha ad73b80c19317b04901262191ad307b313fd8e7e

Update README.md Markdown formatting and typo.

view details

push time in 2 months

fork ianpartridge/docs-2

GitOps Documentation Repository

fork in 2 months

pull request commentappsody/stacks

add splash page for quarkus kafka project

Thanks Paul. I'm happy with this - our need was for this template to have something at the root so when a user deploys it on OCP and clicks "Open URL" they don't see an error. Adding this static page absolutely fulfils that (and moves this closer in sync with the upstream quickstart too 👍 )

josiemundi

comment created time in 2 months

issue commentquarkusio/quarkus-quickstarts

Kafka Quickstart: move JAX-RS endpoint to root

Hi Paul, yes it does. If you're happy to add the static page to the Appsody template then we can close this :)

ianpartridge

comment created time in 2 months

issue commentquarkusio/quarkus-quickstarts

Kafka Quickstart: move JAX-RS endpoint to root

Ah that makes sense. We use this quickstart as a template in the Quarkus Appsody stack: https://github.com/appsody/stacks/tree/master/incubator/quarkus/templates/kafka which won't have the static page. I'd like to keep our template and this quickstart in sync, hence opening this issue :)

ianpartridge

comment created time in 2 months

issue commentquarkusio/quarkus-quickstarts

Kafka Quickstart: move JAX-RS endpoint to root

CC: @gsmet @cescoffier

ianpartridge

comment created time in 2 months

issue openedquarkusio/quarkus-quickstarts

Kafka Quickstart: move JAX-RS endpoint to root

Currently, the JAX-RS endpoint for this demo is at /prices.html:

@Path("/prices") https://quarkus.io/guides/kafka#the-price-resource

This means that when a user deploys the demo onto OpenShift and click the "Open URL" link in the Topology view, they see an error:

RESTEASY003210: Could not find resource for full path: http://my-cluster.example.com/

This is because there is no endpoint defined at the root of the application.

This creates a slightly confusing user experience.

I would like to suggest that we move the /prices endpoint to the root to avoid this problem.

created time in 2 months

issue commentquarkusio/quarkus-quickstarts

Kafka Quickstart: Test Resource opens Kafka Container twice

Looks like this can be closed.

cradloff

comment created time in 2 months

Pull request review commentappsody/stacks

nodejs: Add 'build' and 'prune' lifecycle events

 RUN rm -rf /project/user-app/node_modules && mkdir -p /project/user-app/node_mod  # Install user-app dependencies WORKDIR /project/user-app-RUN npm install --production +RUN npm install --production++# Run a build phase. Projects that need to execute build commands can customize the+# 'build' script in their package.json. The build script should call 'npm install'.+# If no build is required, just install production dependencies.+RUN npm run build --if-present++# Uninstall dev dependencies, leaving only production dependencies. Projects can+# customize the 'prune' script in their package.json.+# Ideally this would just be 'npm prune', but this command does not have pre/post+# hooks. Instead, the user's 'prune' script can itself call 'npm prune' in addition+# to any additional actions required.+RUN npm run prune --production --if-present  # Creating a tar to work around poor copy performance when using buildah 1.9.0 RUN cd / && tar czf project.tgz project  # Copy the dependencies into a slim Node docker image FROM node:12-slim +# Install common dependencies (TLS and CA support)+RUN apt-get update && apt-get install -y libssl1.1 ca-certificates && apt-get clean

FROM node:12-slim is not reproducible anyway though, is it? That image will be updated regularly by upstream.

djones6

comment created time in 2 months

delete branch kabanero-io/kabanero-pipelines

delete branch : typo

delete time in 3 months

push eventianpartridge/coffeeshop-demo

matthewpwilson

commit sha 764e0edeb32607992ffbf7c6abd2d446378b4992

Set perms for Liberty outtput dir (#18) Co-authored-by: Ian Partridge <i.partridge@uk.ibm.com>

view details

push time in 3 months

PR merged ianpartridge/coffeeshop-demo

Set perms for Liberty outtput dir in Dockerfiles

This PR fixes barista-http so the app will start on OpenShift.

Containers in OpenShift run by default as a random user that belongs to the root group (gid=0).
Our Dockerfiles start up Liberty to prime the class cache. This creates some directories under /opt/ol/wlp/output which don't have group write permission. Therefore when running in OpenShift we see errors in the Liberty log about not being able to write to temporary directories. This can be fixed by setting permissions for the group to match the user permissions. I've applied this fix to all 3 Dockerfiles.

The bit i don't quite understand is why this caused a problem with reading the WAR file only for barista-http. My theory is that Liberty needed to unpack this WAR but not the others. But anyway, it does fix the problem.

I am still seeing errors about the classcache not being readable. This could be investigated further, but I think it's more useful to look at applying the KEDA configuration to our Appsodified version instead.

+9 -0

1 comment

3 changed files

matthewpwilson

pr closed time in 3 months

push eventmatthewpwilson/quarkus-coffeeshop-demo

Matthew Wilson

commit sha 4759d70abb76b73ea8e343eef9ff407c0e3bbe82

Update image tags for test deployment

view details

Matthew Wilson

commit sha 2b3d82eff1c850f2493f3903e6066327d6c89586

Fix type in image tag

view details

Matthew Wilson

commit sha 9dfdafc5458dd254db7b45c049471875e5355e4d

Add namespace to images

view details

Matthew Wilson

commit sha 5eb2f53fb91055411ce07bf7ef9f03c666ece311

Avoid explicit docker.io so CI works

view details

Matthew Wilson

commit sha bcd6c78e08c30be7b6795e281af0a1c6dc014e21

Separate image registry in helm chart

view details

Ian Partridge

commit sha dd9cd8f74d965c230b7485447846baf6d087d06b

Merge pull request #19 from matthewpwilson/travis WiP: Update image tags for test deployment

view details

Ian Partridge

commit sha 7245aa7dfd28ac85cfcfedef918c0ff2ca35c102

Merge branch 'master' into perms

view details

push time in 3 months

push eventianpartridge/coffeeshop-demo

Matthew Wilson

commit sha 4759d70abb76b73ea8e343eef9ff407c0e3bbe82

Update image tags for test deployment

view details

Matthew Wilson

commit sha 2b3d82eff1c850f2493f3903e6066327d6c89586

Fix type in image tag

view details

Matthew Wilson

commit sha 9dfdafc5458dd254db7b45c049471875e5355e4d

Add namespace to images

view details

Matthew Wilson

commit sha 5eb2f53fb91055411ce07bf7ef9f03c666ece311

Avoid explicit docker.io so CI works

view details

Matthew Wilson

commit sha bcd6c78e08c30be7b6795e281af0a1c6dc014e21

Separate image registry in helm chart

view details

Ian Partridge

commit sha dd9cd8f74d965c230b7485447846baf6d087d06b

Merge pull request #19 from matthewpwilson/travis WiP: Update image tags for test deployment

view details

push time in 3 months

PR merged ianpartridge/coffeeshop-demo

WiP: Update image tags for test deployment

The CI pushes the microservice images to a local registry before deploying them to test. I've updated the images it pushes to match the images we're now building.

I also separated out the image registry into a separate value in the helm chart so it's easier to parameterise.

+17 -14

0 comment

8 changed files

matthewpwilson

pr closed time in 3 months

PR opened ibm-icpa-coffeeshop/tests

Update README.md

Markdown link was the wrong way round.

+1 -1

0 comment

1 changed file

pr created time in 3 months

create barnchibm-icpa-coffeeshop/tests

branch : readme

created branch time in 3 months

more