fix proxy build broken by global_dictionary.yaml#615
fix proxy build broken by global_dictionary.yaml#615yangminzhu wants to merge 1 commit intoistio:masterfrom
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: If they are not already assigned, you can assign the PR to them by writing The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
@yangminzhu can you verify that the resulting file is still a valid yaml file? @bianpengyuan You had faced an issue with |
|
@mandarjog instead of removing it, I added some logic to escape |
|
@bianpengyuan so I think the right way to fix is to also update create_global_dictionary.py to do the same escape? If so I'll update this PR as well. Thanks. |
|
@yangminzhu Yes I think so. Let's do that. |
I made a comment in https://github.com/istio/api/pull/605/files but think it's faster to send out a PR directly.
@mandarjog
Line 316
- "-"break the proxy build as create_global_dictionary.py parse it to:""-"",which is invalid in.ccfile.Could you suggest the proper way to fix this? Should we just remove the
"or make it\"-\"or even update the script?