Skip to content

Conversation

douglas-reid
Copy link
Contributor

When we removed goimports from the fixed_cfg.pb.go generation,
we lost the ability to clean up the now unused google_protobuf
import. This PR restores that cleaning.

@googlebot googlebot added the cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. label Feb 3, 2018
@geeknoid geeknoid merged commit c5185ac into istio:master Feb 3, 2018
incfly pushed a commit to incfly/api that referenced this pull request Jun 13, 2018
* Added unit tests for ApiManager and RequestHandler

* Fixed code formatting

* Simplified duplicated variables

* Removed unused constant variables

* Added more comments
incfly pushed a commit to incfly/api that referenced this pull request Jun 13, 2018
Automatic merge from submit-queue.

[DO NOT MERGE] Auto PR to update dependencies of mixerclient

This PR will be merged automatically once checks are successful.
```release-note
none
```
nacx pushed a commit to nacx/api that referenced this pull request Feb 23, 2022
…ure (istio#356)

* Return AppliedObjectList instead of Emoty at service CLI Apply procedure

When an apply aperation is invoqued and no error is returned, the invoquer has not information if the applied object was created, updated, or unchanged.
To give more information about the apply's opperation, we will return:
- The status of each individual applied object.
- The applied object itself. The whole object is needed to being able to construct the object's fqn.

A new message called `AppliedObject` is created to encapsulate the status and the applied object.

Signed-off-by: Marc Navarro Sonnenfeld <[email protected]>
nacx added a commit to nacx/api that referenced this pull request Feb 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants