Test-infra: prow: dogs win the fighting

Created on 23 Aug 2018  路  24Comments  路  Source: kubernetes/test-infra

When I call my cat, I can't find it. (https://github.com/kubernetes/kubernetes/pull/62582#issuecomment-415313522)

When I call my dog, it appears. (https://github.com/kubernetes/kubernetes/pull/62582#issuecomment-415313654)

Anyway, interesting plugins!

/cc @fejta

kin馃惗 prioritcritical-urgent

Most helpful comment

@ixdy: cat image

In response to this:

/meow boxes

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

All 24 comments

/meow

@xiangpengzhao: http://thecatapi.com appears to be down

In response to this:

/meow

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/woof

@xiangpengzhao: dog image

In response to this:

/woof

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/woof

@stevekuznetsov: dog image

In response to this:

/woof

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

noooooo 馃惐

/meow

@ixdy: http://thecatapi.com appears to be down

In response to this:

/meow

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

This is a new version as of August 2018 - now with added JSON support!

Maybe related?

it looks like the cat api no longer returns a source_url, but we don't use that, so it's fine.

working on a fix now.
/assign

actually, it seems like the API is completely different, and it no longer even has categories. :(

this is more involved, punting to someone else.
/unassign

actually, it seems like the API is completely different, and it no longer even has categories. :(

馃槶

why is that button even there
/woof woof

/assign

ok I have a fix after all

/meow

@ixdy: cat image

In response to this:

/meow

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

(this was fixed upstream as well)

/meow space

@ixdy: cat image

In response to this:

/meow space

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/meow boxes

@ixdy: cat image

In response to this:

/meow boxes

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Thanks @ixdy for helping the cats be back! Guess they will be fighting with dogs again :) I like the plugins!

Was this page helpful?
0 / 5 - 0 ratings

Related issues

BenTheElder picture BenTheElder  路  3Comments

Aisuko picture Aisuko  路  3Comments

cjwagner picture cjwagner  路  3Comments

spzala picture spzala  路  4Comments

zacharysarah picture zacharysarah  路  3Comments