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

Fix cf pushapp for cf cli v8 #82

Draft
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

mafolz
Copy link
Collaborator

@mafolz mafolz commented Aug 2, 2024

Description

Since the update to the CF CLI Version 8, the Client tried to use the CC v3 API.
A backport vor v7action was done, together with a better detection of LogCache-Endpoints.

Motivation and Context

How Has This Been Tested?

  • start stratos locally or use an existing deployment
  • use (cf-example-staticfile)[https://github.com/govau/cf-example-staticfile] for start a deployment

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • Docs update
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have followed the guidelines in CONTRIBUTING.md, including the required formatting of the commit message

@mafolz
Copy link
Collaborator Author

mafolz commented Aug 2, 2024

This Fix is not 100% complete.

What still doesn’t work, is the Update of the FrontEnd after the Deployment is finished.

Screenshot

The Summary Buttons are still deactivated even where the Notification recognizes the deployment.

So there must be a Problem here i guess: If anyone is able to look at this, we would greatly appreciate some input from the community

@mafolz mafolz self-assigned this Oct 10, 2024
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

Successfully merging this pull request may close these issues.

1 participant