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

Initial Resource Translator Design and Implementation #38

Closed
danehans opened this issue Apr 28, 2022 · 4 comments · Fixed by #193
Closed

Initial Resource Translator Design and Implementation #38

danehans opened this issue Apr 28, 2022 · 4 comments · Fixed by #193
Assignees
Labels
area/translator Issues related to Gateway's translation service, e.g. translating Gateway APIs into the IR. documentation Improvements or additions to documentation help wanted Extra attention is needed kind/enhancement New feature or request
Milestone

Comments

@danehans
Copy link
Contributor

#16 specifies the need for a Gateway API Translator (GAT) to decouple the user APIs from the internal system. The GAT details should be captured in this issue or a linked doc.

@danehans danehans changed the title [Translator] Design Gateway API Translation [Translator] Initial Gateway API Translator Design and Implementation Apr 29, 2022
@danehans danehans added documentation Improvements or additions to documentation kind/enhancement New feature or request help wanted Extra attention is needed labels Apr 29, 2022
@danehans danehans added this to the v0 Release milestone Apr 29, 2022
@skriss
Copy link
Contributor

skriss commented May 3, 2022

For reference, Contour's Gateway API translation logic is largely contained here: https://github.com/projectcontour/contour/blob/main/internal/dag/gatewayapi_processor.go

@danehans danehans changed the title [Translator] Initial Gateway API Translator Design and Implementation Initial Gateway API Translator Design and Implementation May 10, 2022
@danehans danehans added the area/translator Issues related to Gateway's translation service, e.g. translating Gateway APIs into the IR. label May 10, 2022
@danehans
Copy link
Contributor Author

@skriss I assigned this issue to you based on the 5/24/22 community meeting notes. Please unassign yourself if that's not the case.

@arkodg arkodg removed the help wanted Extra attention is needed label Jun 1, 2022
@github-actions
Copy link

github-actions bot commented Jul 2, 2022

This issue has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in the next 7 days unless it is tagged "help wanted" or "no stalebot" or other activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale label Jul 2, 2022
@danehans danehans added help wanted Extra attention is needed and removed stale labels Jul 2, 2022
@danehans danehans changed the title Initial Gateway API Translator Design and Implementation Initial Resource Translator Design and Implementation Jul 12, 2022
@danehans
Copy link
Contributor Author

danehans commented Jul 12, 2022

#145 resolves the design portion of this issue.

skriss added a commit to skriss/gateway that referenced this issue Jul 14, 2022
skriss added a commit to skriss/gateway that referenced this issue Jul 14, 2022
skriss added a commit to skriss/gateway that referenced this issue Jul 14, 2022
skriss added a commit to skriss/gateway that referenced this issue Jul 14, 2022
Updates envoyproxy#38.

Signed-off-by: Steve Kriss <krisss@vmware.com>
danehans pushed a commit that referenced this issue Jul 15, 2022
* Adds initial draft of Gateway API translator

Updates #38.

Signed-off-by: Steve Kriss <krisss@vmware.com>

* Cache interface -> Resources struct

Signed-off-by: Steve Kriss <krisss@vmware.com>

* set unique names for IR routes

Signed-off-by: Steve Kriss <krisss@vmware.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/translator Issues related to Gateway's translation service, e.g. translating Gateway APIs into the IR. documentation Improvements or additions to documentation help wanted Extra attention is needed kind/enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants