Skip to content

Update ABI to Proxy-Wasm ABI v0.2.1.#64

Merged
PiotrSikora merged 3 commits intoproxy-wasm:masterfrom
PiotrSikora:abi_0.2.1
Jul 13, 2021
Merged

Update ABI to Proxy-Wasm ABI v0.2.1.#64
PiotrSikora merged 3 commits intoproxy-wasm:masterfrom
PiotrSikora:abi_0.2.1

Conversation

@PiotrSikora
Copy link
Copy Markdown
Member

Signed-off-by: Piotr Sikora piotrsikora@google.com

Signed-off-by: Piotr Sikora <piotrsikora@google.com>
@PiotrSikora PiotrSikora requested a review from gbrail December 7, 2020 22:27
Copy link
Copy Markdown
Contributor

@gbrail gbrail left a comment

Choose a reason for hiding this comment

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

LGTM -- it's great to be able to use the whole new ABI. Thanks.

unleashed pushed a commit to 3scale/proxy-wasm-rust-sdk that referenced this pull request Dec 22, 2020
Signed-off-by: Piotr Sikora <piotrsikora@google.com>
@nullpo-head
Copy link
Copy Markdown
Contributor

Hi @PiotrSikora , I am curious about the status of this PR. Do you have any idea when this PR will be merged, or is there some kind of blocker?

By the way, I would also appreciate it if you could tell me if there is a written specification of ABI v0.2.0/1 apart from the implementation (https://github.com/proxy-wasm/proxy-wasm-cpp-host/blob/master/src/wasm.cc#L128-L140).

@PiotrSikora
Copy link
Copy Markdown
Member Author

Hi @PiotrSikora , I am curious about the status of this PR. Do you have any idea when this PR will be merged, or is there some kind of blocker?

There is no blocker, it should have been merged months ago, but I wanted to add one more change before breaking API, and it felt behind the schedule. It was quite silly in hindsight.

I'll try to make that change over the weekend, but even if it doesn't happen, I'll merge this PR on Monday.

Feel free to ping me here if it's not done by then.

By the way, I would also appreciate it if you could tell me if there is a written specification of ABI v0.2.0/1 apart from the implementation (https://github.com/proxy-wasm/proxy-wasm-cpp-host/blob/master/src/wasm.cc#L128-L140).

Nope, sorry.

@nullpo-head
Copy link
Copy Markdown
Contributor

Hi @PiotrSikora , I am curious about the status of this PR. Do you have any idea when this PR will be merged, or is there some kind of blocker?

There is no blocker, it should have been merged months ago, but I wanted to add one more change before breaking API, and it felt behind the schedule. It was quite silly in hindsight.

I'll try to make that change over the weekend, but even if it doesn't happen, I'll merge this PR on Monday.

Feel free to ping me here if it's not done by then.

By the way, I would also appreciate it if you could tell me if there is a written specification of ABI v0.2.0/1 apart from the implementation (https://github.com/proxy-wasm/proxy-wasm-cpp-host/blob/master/src/wasm.cc#L128-L140).

Nope, sorry.

Thank you so much for your quick answer! It helps a lot!

@jcruger-godaddy
Copy link
Copy Markdown

@PiotrSikora just wanted to follow up here on the plans for merge. Appreciate the hard work on this

@jcrugzz
Copy link
Copy Markdown

jcrugzz commented May 26, 2021

@PiotrSikora hey wanted to check in if there are any blockers on this. Would be great to get a release out when you can.

@PiotrSikora
Copy link
Copy Markdown
Member Author

@jcrugzz thanks for the nudge. I want to spin one more 0.1.x release before merging this, since it's a breaking change.

The new release was mostly blocked on the gRPC support (merged a few days ago), but there are a few small fixes that I want get in... It should be done by the end of this week, and I'll merge this afterwards.

@jcrugzz
Copy link
Copy Markdown

jcrugzz commented May 27, 2021

@PiotrSikora Thanks for the response, appreciate your work here

@jcrugzz
Copy link
Copy Markdown

jcrugzz commented Jun 1, 2021

@PiotrSikora hey just checking in here. I know how busy things can be so just seeing where we are at. All the best

Signed-off-by: Piotr Sikora <piotrsikora@google.com>
Signed-off-by: Piotr Sikora <piotrsikora@google.com>
@PiotrSikora PiotrSikora requested a review from mathetake July 13, 2021 09:13
@PiotrSikora PiotrSikora merged commit abd0f54 into proxy-wasm:master Jul 13, 2021
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.

6 participants