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

Logs of failing deployments cannot be retrieved anymore with newest cf #459

Open
stefanlay opened this issue Feb 17, 2023 · 6 comments
Open
Assignees
Labels

Comments

@stefanlay
Copy link

With loggregator-release v107.0.0 the RecentLogsHandler was removed from the trafficcontroller. It was also removed in the noaa client.

This endpoint is still used in the appdeployers and the v3appdeployers code to retrieve logs when application staging failed.

Instead of the noaa client and the trafficcontroller recentlogs endpoint the LogCache API endpoint GET /api/v1/read/<source-id>?envelope_types=LOG&limit=? should be used.

@stefanlay stefanlay changed the title Logs of failing deployments cannot retrieved anymore with newest cf Logs of failing deployments cannot be retrieved anymore with newest cf Feb 17, 2023
@chombium
Copy link

chombium commented Feb 17, 2023

There is a go-log-cache client which can be used instead of the NOAA Client ;)

@loafoe loafoe added the bug label Feb 17, 2023
@chombium
Copy link

Hi @loafoe, @Thanhphan1147,
we (me and @ZPascal) will start working on this issue. Can you please assign us?

TIA,
Jovan

@loafoe loafoe assigned loafoe and chombium and unassigned loafoe Jul 28, 2023
@loafoe
Copy link
Member

loafoe commented Jan 29, 2024

@chombium any progress?

@chombium
Copy link

Hi @loafoe,
I've been really busy lately :( I'll give my best to open a PR in the next week or two.

@ZPascal
Copy link

ZPascal commented Feb 6, 2024

Hi @loafoe, FYI we've got currently the problem that the requirements of the log-cache functionality are not available inside the replaced community release of the CF CLI implementation. Could you please describe the circumstances for the replacement process? cc: @chombium

@chombium
Copy link

chombium commented Feb 7, 2024

I've checked the state of the state of the cloudfoundry-cli in the cloudfoundry-community org and the state is 2667 commits behind cloudfoundry/cli:main. We could try exchanging the cli library with the latest upstream, but it would be hard to tell what other things might go wrong and if this project has enough test.
@loafoe We can try to use the upstream cloudfoundry-cli from code.cloudfoundry, practically without the replace in go.mod, but we don't know if we're going to break something...

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

No branches or pull requests

4 participants