Skip to content

App and service metadata #966

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

Merged
merged 2 commits into from
Aug 12, 2019

Conversation

petko-boyadzhiev
Copy link
Contributor

This pull request should resolve some of the requirements of
Issue link: https://github.com/cloudfoundry/cf-java-client/issues/946
It adds support for application and service-instance metadata. Also it adds the possibility for service-instance to be modified (required by the metadata task).

Please note that extensive testing on real CF is currently in progress, so i will notify here when it can be merged (if all other things are ok).

@cfdreddbot
Copy link

❌ Hey petko-boyadzhiev!

All pull request submitters and commit authors must have a Contributor License Agreement (CLA). Click here for details on the CLA process.

The following github user @petko-boyadzhiev is not covered by a CLA.

After the CLA process is complete, this pull request will need to be closed & reopened. DreddBot will then validate the CLA(s).

@twoseat twoseat self-assigned this May 22, 2019
@petko-boyadzhiev
Copy link
Contributor Author

Hi, @twoseat
I finished my testing and all seems to work. Can you take a look and see if anything needs to be changed?

@twoseat twoseat added this to the 3.16.0.RELEASE milestone Aug 7, 2019
@twoseat twoseat merged commit ecf74ab into cloudfoundry:master Aug 12, 2019
@twoseat
Copy link
Contributor

twoseat commented Aug 12, 2019

Thanks for the PR @petko-boyadzhiev (and apologies for the delay merging). I tweaked some formatting and other details to fit with our conventions, but nothing major. Look for a release shortly.

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.

3 participants