Create & teardown Lambda & API Gateway resources for serverless feature server

[tsotnet]

Create & teardown Lambda & API Gateway resources for serverless feature server

Signed-off-by: Tsotne Tabidze [email protected]

What this PR does / why we need it: Here we implement update_infra and teardown_infra methods of AwsProvider, creating and destroying lambda & API gateway resources.

Here's what feast apply looks like:

Deploying feature server...
  Creating AWS Lambda...
  Creating AWS API Gateway...
Registered entity driver_id
Registered feature view driver_hourly_stats
Deploying infrastructure for driver_hourly_stats
Deploying feature server...

And here's what feast teardown looks like:

Tearing down feature server...
  Tearing down AWS Lambda...
  Tearing down AWS API Gateway...

Currently feast endpoint isn't implemented, so we don't show the HTTP endpoint anywhere, although I checked it from AWS Console and sending some cURL requests worked as expected.

Which issue(s) this PR fixes:

Fixes #

Does this PR introduce a user-facing change?:

NONE

[codecov-commenter]

Codecov Report

Merging #1900 (4123b5c) into master (0155284) will decrease coverage by 0.69%.
The diff coverage is 31.81%.

@@            Coverage Diff             @@
##           master    #1900      +/-   ##
==========================================
- Coverage   82.97%   82.27%   -0.70%     
==========================================
  Files          96       96              
  Lines        7376     7470      +94     
==========================================
+ Hits         6120     6146      +26     
- Misses       1256     1324      +68     

Flag	Coverage Δ
integrationtests	73.82% <31.81%> (-0.59%)	⬇️
unittests	60.20% <28.18%> (-0.45%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
sdk/python/feast/infra/passthrough_provider.py	100.00% <ø> (+4.00%)	⬆️
sdk/python/feast/infra/utils/aws_utils.py	71.01% <22.22%> (-12.03%)	⬇️
sdk/python/feast/infra/aws.py	40.62% <27.14%> (-10.87%)	⬇️
sdk/python/feast/errors.py	70.07% <66.66%> (-0.24%)	⬇️
sdk/python/feast/constants.py	100.00% <100.00%> (ø)
sdk/python/feast/usage.py	66.89% <100.00%> (+0.22%)	⬆️
sdk/python/feast/infra/offline_stores/bigquery.py	80.11% <0.00%> (+0.11%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0155284...4123b5c. Read the comment docs.

[achals]

I had some nits, but the PR looks mostly good. Are you planning on adding tests in this pass or is that going to be punted to a subsequent PR?

[felixwang9817]

@achals yes I plan on doing tests in a later PR (since tests are not necessary for a the serverless MVP).

[achals]

@achals yes I plan on doing tests in a later PR (since tests are not necessary for a the serverless MVP).

Cool. I had one more nit but looks good otherwise.

[achals]

/lgtm

[feast-ci-bot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: achals, tsotnet

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [achals,tsotnet]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment