Skip to content
This repository was archived by the owner on Jan 16, 2022. It is now read-only.

Support FillProto for QuotaRequest.#44

Merged
qiwzhang merged 1 commit intoistio:masterfrom
qiwzhang:quota_request
Mar 21, 2017
Merged

Support FillProto for QuotaRequest.#44
qiwzhang merged 1 commit intoistio:masterfrom
qiwzhang:quota_request

Conversation

@qiwzhang
Copy link
Copy Markdown
Contributor

Use two special attributes "quota.name" and "quota.amount" to pass info to protobuf QuotaRequest.

@istio-testing
Copy link
Copy Markdown
Contributor

Jenkins job mixerclient/presubmit passed

Copy link
Copy Markdown
Contributor

@kyessenov kyessenov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Question: are we using quota already?

@qiwzhang
Copy link
Copy Markdown
Contributor Author

Not yet. I am about to it. This is for istio/proxy to call it.

@istio-testing
Copy link
Copy Markdown
Contributor

Jenkins job proxy/mixerclient-pr passed

1 similar comment
@istio-testing
Copy link
Copy Markdown
Contributor

Jenkins job proxy/mixerclient-pr passed

@qiwzhang qiwzhang merged commit b76b513 into istio:master Mar 21, 2017
@qiwzhang qiwzhang deleted the quota_request branch March 21, 2017 00:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants