Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update readiness condition based on new Camel health check format #2886

Closed
astefanutti opened this issue Jan 18, 2022 · 3 comments · Fixed by #3310
Closed

Update readiness condition based on new Camel health check format #2886

astefanutti opened this issue Jan 18, 2022 · 3 comments · Fixed by #3310
Assignees
@tadayosi
Labels
status/never-stale
Milestone
1.9.3

Comments

@astefanutti
Copy link
Member

The parsing of the health check results from the integration Pod(s) readiness endpoint(s) should be adapted once apache/camel#6779 is pulled.
@github-actions
Copy link
Contributor

This issue has been automatically marked as stale due to 90 days of inactivity.
It will be closed if no further activity occurs within 15 days.
If you think that’s incorrect or the issue should never stale, please simply write any comment.
Thanks for your contributions!

@github-actions github-actions bot closed this as completed May 4, 2022
@lburgazzoli lburgazzoli reopened this May 25, 2022
@squakez squakez added this to the 1.9.3 milestone May 26, 2022
@tadayosi tadayosi self-assigned this May 27, 2022
@tadayosi
Copy link
Member

OK, now I'm investigating why health_test.go always fails on CI. Is this the reason why it always does? If so, I'm taking care of it.

@squakez
Copy link
Contributor

squakez commented May 27, 2022

Very very probably :)

tadayosi added a commit to tadayosi/camel-k that referenced this issue May 27, 2022
@tadayosi
fix(health): support new Camel health check format
b18247a 
Fix apache#2886
@tadayosi tadayosi mentioned this issue May 27, 2022
Merged
tadayosi added a commit that referenced this issue May 30, 2022
@tadayosi
fix(health): support new Camel health check format
de09f4c 
Fix #2886
tadayosi added a commit to tadayosi/camel-k that referenced this issue May 30, 2022
@tadayosi
fix(health): support new Camel health check format
0b933db 
Fix apache#2886
tadayosi added a commit that referenced this issue May 31, 2022
@tadayosi
fix(health): support new Camel health check format
80fd27e 
Fix #2886
squakez pushed a commit to jboss-fuse/camel-k that referenced this issue May 31, 2022
@tadayosi @squakez
fix(health): support new Camel health check format
cefe850 
Fix apache#2886

(cherry picked from commit apache/camel-k@80fd27e07)
squakez pushed a commit to jboss-fuse/camel-k that referenced this issue May 31, 2022
@tadayosi @squakez
fix(health): support new Camel health check format
8a565c6 
Fix apache#2886

(cherry picked from commit apache/camel-k@80fd27e07)
squakez pushed a commit to jboss-fuse/camel-k that referenced this issue May 31, 2022
@tadayosi @squakez
fix(health): support new Camel health check format
9444bb9 
Fix apache#2886

(cherry picked from commit apache/camel-k@80fd27e07)
squakez pushed a commit to jboss-fuse/camel-k that referenced this issue May 31, 2022
@tadayosi @squakez
fix(health): support new Camel health check format
2ea695f 
Fix apache#2886

(cherry picked from commit apache/camel-k@80fd27e07)
squakez pushed a commit to jboss-fuse/camel-k that referenced this issue Jun 1, 2022
@tadayosi @squakez
fix(health): support new Camel health check format
9b984f9 
Fix apache#2886

(cherry picked from commit apache/camel-k@80fd27e07)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tadayosi tadayosi

Labels
status/never-stale
Projects
None yet
Milestone
1.9.3
Development

Successfully merging a pull request may close this issue.

fix(health): support new Camel health check format tadayosi/camel-k
4 participants
@tadayosi @astefanutti @lburgazzoli @squakez