pecos / tps

Torch Plasma Simulator
BSD 3-Clause "New" or "Revised" License
8 stars 2 forks source link

General wall boundary condition #134

Closed dreamer2368 closed 2 years ago

dreamer2368 commented 2 years ago
koomie commented 2 years ago

@dreamer2368, just a head's up that I have landed some modifications to the CI setup on this branch. In particular, the supporting docker container environment derived from docker/test/Dockerfile will be built automatically when the file is changed and then subsequently used in later tests. The image is hosted by a new account and is available as pecosut/tps_env:latest. I had to refactor things and merge two of our previous workflows so you should only see one top-level CI event kicked off.

It looks to be working ok now and pulls in your updated MASA version. When you are happy with the remaining PR, we can merge.

dreamer2368 commented 2 years ago

@koomie , it looks good to me. A question is, when a PR updates the container (presumably as pecosut/tps_env:latest), is this updated container reflected to the main CI test before the merge, or only after the PR is merged? Another way to put it is, do these two branches use the same tag of the container or separate tags specified for each branch?

Regardless of the answer, i think the current PR is sufficient to merge to main.

koomie commented 2 years ago

@koomie , it looks good to me. A question is, when a PR updates the container (presumably as pecosut/tps_env:latest), is this updated container reflected to the main CI test before the merge, or only after the PR is merged? Another way to put it is, do these two branches use the same tag of the container or separate tags specified for each branch?

Regardless of the answer, i think the current PR is sufficient to merge to main.

Once this is landed, the main branch will also use the same container. In the current config, the container is updated by the PR and will should be rebuilt again when it lands on main. In general, it will be the exact same container, so we waste a few minutes doing it again, but this allows you to update the container and test the PR.