lift icon indicating copy to clipboard operation
lift copied to clipboard

Will there be serverless v4 compatibility?

Open cmcnicholas opened this issue 1 year ago • 7 comments

Start from the Use-case

Serverless v4 is here. https://github.com/serverless/serverless/releases/tag/v4.0.26

Example Config

No response

Implementation Idea

No response

cmcnicholas avatar May 25 '24 08:05 cmcnicholas

Is that incompatible at the moment? Do you get any error?

mnapoli avatar May 27 '24 07:05 mnapoli

Is that incompatible at the moment? Do you get any error?

Just the version dependency currently.

"serverless": "^3.28.0",

cmcnicholas avatar May 27 '24 08:05 cmcnicholas

Understood, we can definitely relax that 👍 Before doing so, it would be great to get confirmation that it works fine :)

mnapoli avatar May 27 '24 08:05 mnapoli

FWIW our intention is to support the Lift plugin in Serverless Framework V.4, as always. We've done some testing recently and discovered a bug. We're working on a fix now and tracking it here: https://github.com/serverless/serverless/issues/12571

austencollins avatar May 31 '24 22:05 austencollins

For anyone looking for a work-around for this, pinning serverless to the latest v3 gets around this issue for the moment, but not ideal.

rj-xy avatar Jun 05 '24 20:06 rj-xy

Lift should work seamlessly with V.4 now as of this release. We are continuing to test and monitor everything. Please let us know if you see further issues.

austencollins avatar Jun 05 '24 21:06 austencollins

I just started using lift with serverless and struggling with npm dependencies and layers. I use following v4 config:

build:
  esbuild:
    bundle: true
    minify: false
    target: node20
    packages: external

constructs:
  drawing-queue:
    type: queue
    worker:
      handler: src/drawing/worker.handler
      layers:
        - arn:aws:lambda:xxx:layer:chromiumLayer:3
        - arn:aws:lambda:xxx:layer:sharpLayer:2

I bump into the issue that defined layers of construct fields of the yaml not work as expected, also the npm dependencies seem not to work or doesnt seemp to be bundled as the other functions. Am I missing something?

Just to make sure: the dependencies and layers work in the function calls, but I want to make the switch to queue as some tasks are running too long.

dohomi avatar Jun 15 '24 18:06 dohomi

hey guys, any updates on it? it's still impossible to npm install lift with serverless v4

Vakintoshka avatar Aug 26 '24 13:08 Vakintoshka

https://github.com/serverless/serverless/issues/12571 still mentions issues, can anyone confirm whether Lift works with v4?

Also PRs are welcome.

mnapoli avatar Aug 26 '24 14:08 mnapoli

@Vakintoshka Can you elaborate on the install issue you are seeing? If you are seeing the error with a serverless command, can you run serverless support --github and share the output.

skierkowski avatar Aug 27 '24 16:08 skierkowski

@mnapoli Hi 👋 , long time! Yes, Serverless Framework v4 works with Lift. I'm confirming this based on analytics and testing. Having said that, there may still be some issues, like the one mentioned in the issue you linked, and the install issue @Vakintoshka mentioned. I'm trying to get the repro steps so we can fix them.

skierkowski avatar Aug 27 '24 16:08 skierkowski

Hey @skierkowski! Thanks for the follow-up, I've merged a PR that allows installing lift with v4 👍

mnapoli avatar Aug 28 '24 13:08 mnapoli