Skip to content

Conversation

@loicmathieu
Copy link
Contributor

Small enhancement to the Google Cloud Functions HTTP bindings:

  • Add a test (on the RESTEasy layer, not using the Function invoker)
  • Set the extension as exeprimental (as the other ones)
  • Packaging improvements

@loicmathieu
Copy link
Contributor Author

@patriot1burke @gsmet I forgot to send this PR, sorry, it's a bit later but it align the GCF HTTP binging with the other extension (regarding deployment) and more important, it set this extension as experimental so I would like it to be merged asap and backported to 1.6.

Copy link
Member

@gsmet gsmet left a comment

Choose a reason for hiding this comment

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

I added a few minor suggestions. Looks safe to backport.

It uses Quarkus Vert.x instead of a function invoker so it is not testing the Functon infrastructure.
@loicmathieu loicmathieu force-pushed the gcp/http-functions-improvements branch from 45e2115 to 66a349a Compare June 26, 2020 12:48
@loicmathieu
Copy link
Contributor Author

@gsmet PR updated based on your requested changes.

@gsmet gsmet added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Jun 26, 2020
@gsmet gsmet merged commit 2810852 into quarkusio:master Jun 28, 2020
@gsmet gsmet added this to the 1.6.0.Final milestone Jun 30, 2020
@loicmathieu loicmathieu deleted the gcp/http-functions-improvements branch June 30, 2020 17:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/documentation triage/waiting-for-ci Ready to merge when CI successfully finishes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants