You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by "gansheer (via GitHub)" <gi...@apache.org> on 2023/03/06 14:59:59 UTC
[GitHub] [camel-k] gansheer opened a new pull request, #4106: chore(olm): bump framework operator sdk from v1.16.0 to v1.26.1
gansheer opened a new pull request, #4106:
URL: https://github.com/apache/camel-k/pull/4106
Fixes #4105
**Release Note**
```release-note
Bump framework operator sdk from v1.16.0 to v1.26.1
```
--
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: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] gansheer commented on pull request #4106: chore(olm): bump framework operator sdk from v1.16.0 to v1.26.1
Posted by "gansheer (via GitHub)" <gi...@apache.org>.
gansheer commented on PR #4106:
URL: https://github.com/apache/camel-k/pull/4106#issuecomment-1456448607
I made some test on the bundle generated (in local build) between 1.16 and 1.26.1 and I get this result:
```
diff -r bundle/Dockerfile bundle_1.26/Dockerfile
10c10
< LABEL operators.operatorframework.io.metrics.builder=operator-sdk-v1.15.0+git
---
> LABEL operators.operatorframework.io.metrics.builder=operator-sdk-v1.26.1
diff -r bundle/manifests/camel-k.clusterserviceversion.yaml bundle_1.26/manifests/camel-k.clusterserviceversion.yaml
197c197
< createdAt: "2023-03-06T16:21:10Z"
---
> createdAt: "2023-03-06T16:19:27Z"
200c200
< operators.operatorframework.io/builder: operator-sdk-v1.15.0+git
---
> operators.operatorframework.io/builder: operator-sdk-v1.26.1
329c329,336
< - name: camel-k-operator
---
> - label:
> app: camel-k
> app.kubernetes.io/component: operator
> app.kubernetes.io/name: camel-k
> app.kubernetes.io/version: 2.0.0-SNAPSHOT
> camel.apache.org/component: operator
> name: camel-k-operator
> name: camel-k-operator
diff -r bundle/metadata/annotations.yaml bundle_1.26/metadata/annotations.yaml
9c9
< operators.operatorframework.io.metrics.builder: operator-sdk-v1.15.0+git
---
> operators.operatorframework.io.metrics.builder: operator-sdk-v1.26.1
```
Is there something that should cause some concern in this result for you @oscerd ?
--
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: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] squakez merged pull request #4106: chore(olm): bump framework operator sdk from v1.16.0 to v1.26.1
Posted by "squakez (via GitHub)" <gi...@apache.org>.
squakez merged PR #4106:
URL: https://github.com/apache/camel-k/pull/4106
--
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: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] squakez commented on pull request #4106: chore(olm): bump framework operator sdk from v1.16.0 to v1.26.1
Posted by "squakez (via GitHub)" <gi...@apache.org>.
squakez commented on PR #4106:
URL: https://github.com/apache/camel-k/pull/4106#issuecomment-1457881887
@gansheer all looks good. Waiting your ok to merge this.
--
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: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] squakez commented on pull request #4106: chore(olm): bump framework operator sdk from v1.16.0 to v1.26.1
Posted by "squakez (via GitHub)" <gi...@apache.org>.
squakez commented on PR #4106:
URL: https://github.com/apache/camel-k/pull/4106#issuecomment-1457803180
@gansheer the diff looks okey, no major changes in the bundle, so I don't think there won't be any problem. Let's wait the checks I've re-executed just to make sure there is no problem, in particular in the OLM suite.
--
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: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] oscerd commented on pull request #4106: chore(olm): bump framework operator sdk from v1.16.0 to v1.26.1
Posted by "oscerd (via GitHub)" <gi...@apache.org>.
oscerd commented on PR #4106:
URL: https://github.com/apache/camel-k/pull/4106#issuecomment-1456613900
> I made some test on the bundle generated (in local build) between 1.16 and 1.26.1 and I get this result:
>
> ```
> diff -r bundle/ bundle_1.26/
> diff -r bundle/Dockerfile bundle_1.26/Dockerfile
> 10c10
> < LABEL operators.operatorframework.io.metrics.builder=operator-sdk-v1.15.0+git
> ---
> > LABEL operators.operatorframework.io.metrics.builder=operator-sdk-v1.26.1
> diff -r bundle/manifests/camel-k.clusterserviceversion.yaml bundle_1.26/manifests/camel-k.clusterserviceversion.yaml
> 197c197
> < createdAt: "2023-03-06T16:21:10Z"
> ---
> > createdAt: "2023-03-06T16:19:27Z"
> 200c200
> < operators.operatorframework.io/builder: operator-sdk-v1.15.0+git
> ---
> > operators.operatorframework.io/builder: operator-sdk-v1.26.1
> 329c329,336
> < - name: camel-k-operator
> ---
> > - label:
> > app: camel-k
> > app.kubernetes.io/component: operator
> > app.kubernetes.io/name: camel-k
> > app.kubernetes.io/version: 2.0.0-SNAPSHOT
> > camel.apache.org/component: operator
> > name: camel-k-operator
> > name: camel-k-operator
> diff -r bundle/metadata/annotations.yaml bundle_1.26/metadata/annotations.yaml
> 9c9
> < operators.operatorframework.io.metrics.builder: operator-sdk-v1.15.0+git
> ---
> > operators.operatorframework.io.metrics.builder: operator-sdk-v1.26.1
> ```
>
> Is there something that should cause some concern in this result from your experience @oscerd ?
Nothing looks weird, we'll need to check once we create PR in operator hub and community prod for Openshift. The truth will be when they will test through their pipelines.
--
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: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] gansheer commented on pull request #4106: chore(olm): bump framework operator sdk from v1.16.0 to v1.26.1
Posted by "gansheer (via GitHub)" <gi...@apache.org>.
gansheer commented on PR #4106:
URL: https://github.com/apache/camel-k/pull/4106#issuecomment-1459732156
@squakez LGTM too. Since no other concern has been raised I guess it's ready for merge :+1: .
--
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: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org