Skip to content

feat: add Apache OpenWhisk plugin#5518

Merged
membphis merged 64 commits into
apache:masterfrom
bzp2010:feat-plugin-openwhisk
Nov 25, 2021
Merged

feat: add Apache OpenWhisk plugin#5518
membphis merged 64 commits into
apache:masterfrom
bzp2010:feat-plugin-openwhisk

Conversation

@bzp2010

@bzp2010 bzp2010 commented Nov 15, 2021

Copy link
Copy Markdown
Contributor

What this PR does / why we need it:

Support for Apache OpenWhisk serverless platform.
Implement: #5485

Pre-submission checklist:

  • Did you explain what problem does this PR solve? Or what new features have been added?
  • Have you added corresponding test cases?
  • Have you modified the corresponding document?
  • Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first

@bzp2010 bzp2010 added the plugin label Nov 15, 2021
@bzp2010 bzp2010 self-assigned this Nov 15, 2021
@bzp2010
bzp2010 force-pushed the feat-plugin-openwhisk branch 6 times, most recently from 6896677 to 184e5da Compare November 17, 2021 16:32
@bzp2010 bzp2010 linked an issue Nov 18, 2021 that may be closed by this pull request
@bzp2010
bzp2010 force-pushed the feat-plugin-openwhisk branch from 43998d8 to c150d15 Compare November 18, 2021 11:21
@bzp2010
bzp2010 marked this pull request as ready for review November 18, 2021 11:40
Comment thread t/plugin/openwhisk.t
Comment thread apisix/plugins/openwhisk.lua
Comment thread conf/config-default.yaml Outdated
Comment thread apisix/plugins/openwhisk.lua Outdated
Comment thread apisix/plugins/openwhisk.lua
Comment thread apisix/plugins/openwhisk.lua Outdated
Comment thread apisix/plugins/openwhisk.lua Outdated
Comment thread apisix/plugins/openwhisk.lua
Comment thread apisix/plugins/openwhisk.lua
Comment thread apisix/plugins/openwhisk.lua Outdated
@bzp2010
bzp2010 force-pushed the feat-plugin-openwhisk branch from 57f7792 to b20ec70 Compare November 19, 2021 02:12
Comment thread apisix/plugins/openwhisk.lua Outdated
Comment thread apisix/plugins/openwhisk.lua Outdated
Comment thread apisix/plugins/openwhisk.lua
Comment thread docs/en/latest/plugins/openwhisk.md Outdated
Comment thread docs/en/latest/plugins/openwhisk.md Outdated
Comment thread docs/en/latest/plugins/openwhisk.md Outdated
Comment thread apisix/plugins/openwhisk.lua Outdated
Comment thread t/core/request.t Outdated
Comment thread apisix/plugins/openwhisk.lua Outdated
@bzp2010

bzp2010 commented Nov 22, 2021

Copy link
Copy Markdown
Contributor Author

These change will implement in next PR

Update

Based on a tip from an Apache OpenWhisk community member in #5485 (comment) and more document reading, I found that the current code implementation has some limitations and I will modify part of the implementation.

  • Add packages support
  • Add JSON response parse
  • update test case

cc @spacewander @tokers

@bzp2010
bzp2010 force-pushed the feat-plugin-openwhisk branch from 8084ac6 to 70494f4 Compare November 22, 2021 08:08
Comment thread apisix/core/request.lua Outdated
Comment thread apisix/plugins/openwhisk.lua
Comment thread docs/en/latest/plugins/openwhisk.md Outdated
Comment thread docs/zh/latest/plugins/openwhisk.md Outdated
Comment thread docs/en/latest/plugins/openwhisk.md Outdated
Comment thread docs/en/latest/plugins/openwhisk.md Outdated
Comment thread docs/en/latest/plugins/openwhisk.md Outdated
@bzp2010
bzp2010 requested a review from spacewander November 23, 2021 05:12
Comment thread t/plugin/openwhisk.t
@bzp2010
bzp2010 requested a review from tzssangglass November 23, 2021 09:33
@bzp2010
bzp2010 force-pushed the feat-plugin-openwhisk branch from 728f4d8 to 1f98c5e Compare November 25, 2021 07:29
@bzp2010
bzp2010 requested a review from tokers November 25, 2021 08:20
@bzp2010

bzp2010 commented Nov 25, 2021

Copy link
Copy Markdown
Contributor Author

Hi, @spacewander @membphis @tzssangglass @tokers. The CI error has been fixed.

@membphis

Copy link
Copy Markdown
Member

we also need to update the REAME.md @bzp2010

@membphis
membphis merged commit 4f02605 into apache:master Nov 25, 2021
@membphis

Copy link
Copy Markdown
Member

we also need to update the REAME.md @bzp2010

please create a new issue about this, do this later.

@membphis

Copy link
Copy Markdown
Member

we also need to update the REAME.md @bzp2010

please create a new issue about this, do this later.

@bzp2010 any news?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Proposal: support Apache OpenWhisk serverless framework plugin

5 participants