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.

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

CF-CLI input in CATS task is not used correctly #189

Open
johha opened this issue Apr 5, 2024 · 2 comments
Open

CF-CLI input in CATS task is not used correctly #189

johha opened this issue Apr 5, 2024 · 2 comments

Comments

@johha
Copy link

johha commented Apr 5, 2024

The CATS tasks allows to provide a cf-cli input. However in the task itself it expects that CF CLI is provided as binary which the CF CLI Github release does not provide.
Additionally it's unclear if the PATH is set correctly so that CATS would use the provided CF-CLI.

@ctlong
Copy link
Member

ctlong commented Apr 5, 2024

Hi @johha,

❓ Out of curiosity, what is causing you to want to provide the run-cats task with a cf CLI input?

the task itself it expects that CF CLI is provided as binary which the CF CLI Github release does not provide.

True, it would be more convenient for the task to extract a binary from a provided cf CLI GH release. I imagine it may not have been added because that creates more potential tech debt for the task itself 🤷

✍️ If you want to PR this change, I would accept it.

In the meantime, you could write an in-between task in your concourse pipeline to extract the appropriate binary and provide it to the run-cats task.

it's unclear if the PATH is set correctly so that CATS would use the provided CF-CLI.

What makes you say that? It seems like a cf binary in the cf-cli input path, if provided, would be set at the front of the PATH.

@johha
Copy link
Author

johha commented Apr 8, 2024

Hi @ctlong 👋

last week when we had this regex issue in CATS (cloudfoundry/cf-acceptance-tests#1105) we wanted to downgrade the CF CLI version, however as described above this didn't work.
@philippthun and me tried to fix it by adjusting the task to download the CLI (db9978c). However CATS still was using the newer CLI version (part of the image) 🤷‍♂️
I'd say it's not a urgent issue but something we would look into eventually. If you want I can assign it to myself.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants