Skip to content

Update Dockerfile for main branch building #40

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.

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

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

Conversation

tuhaihe
Copy link
Member

@tuhaihe tuhaihe commented Apr 16, 2025

In this PR, these changes are made:

  • Update the timezone to Los Angeles
  • Remove the passwd as set NOPASSWD for gpadmin and root in the file
    /etc/sudoers
  • Add PAX dependencies protobuf-devel and --enable-pax option
  • Update the git clone command to fetch the submodules
  • Enable lz4 and gpcloud support

closes: #39

Change logs

Describe your change clearly, including what problem is being solved or what document is being added or updated.

Contributor's checklist

Here are some reminders before you submit your pull request:

  • Make sure that your Pull Request has a clear title and commit message. You can take the Git commit template as a reference.
  • Sign the Contributor License Agreement as prompted for your first-time contribution (One-time setup).
  • Learn the code contribution and doc contribution guides for better collaboration.
  • List your communications in the GitHub Issues or Discussions (if has or needed).
  • Feel free to ask for other people to help review and approve.

@tuhaihe
Copy link
Member Author

tuhaihe commented Apr 16, 2025

I will test again. Once it's ready, will let you know.

In this PR, these changes are made:
* Update the timezone to Los Angeles
* Remove the passwd as set `NOPASSWD` for gpadmin and root in the file
  /etc/sudoers
* Add PAX dependencies protobuf-devel and `--enable-pax` option
* Update the git clone command to fetch the submodules
* Enable lz4 and gpcloud support
@tuhaihe
Copy link
Member Author

tuhaihe commented Apr 16, 2025

I tested it on a new cloud machine and can run it successfully in the following ways:

./run.sh -c main

./run.sh -c main -m

Now, it's ready for review. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Run failed because of PAX introducing
1 participant