NREL / OpenStudio

OpenStudio is a cross-platform collection of software tools to support whole building energy modeling using EnergyPlus and advanced daylight analysis using Radiance.
https://www.openstudio.net/
Other
501 stars 191 forks source link

[possible bug?] Openstudio-extension-gem's Different behaviour in os 3.7.0rc1 and 3.7.0rc2 (fb8d0fc903) #5027

Closed wenyikuang closed 11 months ago

wenyikuang commented 11 months ago

Issue overview

In 3.7.0rc1 the running of bundle exec rake will generate a test folder in the end but not in 3.7.0rc2 (fb8d0fc903).

Current Behavior

The behaviour is mismatch in rc1 and rc2 for some reason. In rc1 it will not raise exception but in rc2 it will.

Expected Behavior

Steps to Reproduce

In 370rc2, the fastest way to reproduce:

  1. docker pull kuangwenyi/os-370rc2903 and docker run --name os370rc2903 -tid kuangwenyi/os-370rc2903
  2. docker run -ti os370rc2903 bash
  3. git clone https://github.com/NREL/openstudio-extension-gem
  4. cd openstudio-extension-gem && bunlde install
  5. bundle exec rake

The samething has being ran on ci pipeline here: https://ci.openstudio.net/job/openstudio-extension-gem/job/PR-172/2 You could notice the version:

openstudio openstudio_version
3.7.0-rc2+fb8d0fc903

In 370rc1, the fastest way to reproduce:

  1. docker pull nrel/openstudio:3.7.0-rc1 and docker run --name os370rc1 -tid nrel/openstudio:3.7.0-rc1
  2. docker run -ti os370rc1 bash
  3. git clone https://github.com/NREL/openstudio-extension-gem
  4. cd openstudio-extension-gem && bunlde install
  5. bundle exec rake

The samething has being ran on ci pipeline here: https://ci.openstudio.net/job/openstudio-extension-gem/job/PR-172/3

+ openstudio openstudio_version
3.7.0-rc1+211bb633b0

Possible Solution

Details

Environment

Some additional details about your environment for this issue (if relevant):

Context

Since we the os is consuming openstudio-extension-gem, I need to modify the os-extension-gem to fit the new behaviour of os itself in order to use it in the os release.

wenyikuang commented 11 months ago

Dockerfile I used to build the rc2 dockerimage:

Please remove the .txt to build it.

Dockerfile.txt

wenyikuang commented 11 months ago

Test here

Possiblly fixed by: https://github.com/NREL/OpenStudio/pull/5041