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.

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

Add flag for release candidates gem #85

Closed
wants to merge 9 commits into from
Closed

Conversation

wenyikuang
Copy link
Collaborator

Tested with:

  • no environment variable, the Gemfile will install all packages from github
  • set envrionment with export RC_RELEASE=TRUE the Gemfile will install openstudio-standards from rubygem.org
  • set environment to export FINAL_PACKAGE=TRUE the Gemfile will try to install the gem packages from rubgem.org

WHY:
If we use Gemfile to fetch the gem from github, it will fetch all codebase even if we points to ref:. In most of the gems it's fine but openstudio-standards' release is only ~500 MB but the code base is ~5GB.
Which cause the issue in Openstudio-server.

@wenyikuang wenyikuang requested a review from kbenne May 2, 2024 02:57
@wenyikuang
Copy link
Collaborator Author

After this merge and build I will:

  • Update the CMakelist in OS-core
  • Re-build the docker image make sure the size is around ~2G

.github/workflows/build.yml Outdated Show resolved Hide resolved
@jmarrec
Copy link
Collaborator

jmarrec commented May 6, 2024

I don't follow why this is needed. What issue is openstudio-server having?

Copy link
Collaborator

@jmarrec jmarrec left a comment

Choose a reason for hiding this comment

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

If there is a release candidate gem on rubygems, just comment out the git: attribute in the normal section

https://rubygems.org/gems/openstudio-standards has 0.6.0-rc2 so all good

THere is no point adding another env var and this complexity.

I suggest just deleting the PR... and use #87

@brianlball
Copy link
Collaborator

I don't follow why this is needed. What issue is openstudio-server having?

The 5GB size is crashing CI; disk out of space

@wenyikuang wenyikuang closed this May 6, 2024
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.

3 participants