Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix R-builds deploy job #171

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Fix R-builds deploy job #171

wants to merge 4 commits into from

Commits on Jul 10, 2023

  1. Add serverless-package target for local debugging

    @glin
    glin committed Jul 10, 2023
    Configuration menu
    Copy the full SHA
    497b619 View commit details
    Browse the repository at this point in the history

  2. Update serverless npm dependencies

    @glin
    glin committed Jul 10, 2023
    Configuration menu
    Copy the full SHA
    aac9b2b View commit details
    Browse the repository at this point in the history

  3. Fix serverless deploy job in Jenkins 

    * Don't Dockerize pip and stop mounting Docker socket in Jenkins. Mounting
  the socket breaks docker run commands that attempt to mount a directory
  in the container - the mount uses the host rather than the container.
  The serverless-python-requirements plugin will do this and fail to locate
  files. No idea how this was working in Jenkins before, however!
* Switch to jammy Docker image with custom-installed dependencies. The
  language-specific images and lambda images are too tied to distros that
  make dependencies hard to install or maintain. e.g., the nodesource
  install script is broken on Debian 11 right now, and AL2 no longer supports
  newer versions of Node 18+.
* Disable caching in the serverless-python-requirements plugin. For some
  reason, the cache directory in Jenkins changed to a relative .cache
  directory at some point, based on /home/greg, set to . in Jenkins but
  an absolute path like /root typically. The plugin assumes that this cache
  directory is absolute and breaks otherwise with a relative path. To prevent
  future issues, we just disable caching completely rather than explicitly
  setting cacheLocation to an absolute path.
    @glin
    glin committed Jul 10, 2023
    Configuration menu
    Copy the full SHA
    696797a View commit details
    Browse the repository at this point in the history

  4. Remove HOME, restore jenkins user; reenable plugin caching; update co… 

    …mments

* Simplify the deployment as much as possible
* HOME can be removed, but needs the jenkins user restored to have a
  valid HOME dir that's not /, which breaks npm.
    @glin
    glin committed Jul 10, 2023
    Configuration menu
    Copy the full SHA
    fbcdbb2 View commit details
    Browse the repository at this point in the history