Pre-Built SIPNET Binary for GitHub Actions #3435
Conversation
There was a problem hiding this comment.
Thanks for working on this! To meet the requirements of #3427 you'll need to make some changes:
- Building the binary should be done in the sipnet repo. Determining how to get the built binary from there to here is the crux of the task.
- In all repositories Github Actions workflows have to be in
.github/workflows, not in the project root. - The changes to PEcAn's CI should modify existing workflows rather than add new ones.
Sure I will look into this changes. |
|
i have made the required changes.can you please look into those. |
|
@AritraDey-Dev Looks good to me! I am not sure if Lines 61-73 of ci.yml need to be there, now that the binary building will be done elsewhere. I put a note in PecanProject/sipnet#27 and will update the curl statements once we have a release with binaries. |
|
@infotroph i have Updated upload-artifact to v3. SIPNET installation remains encapsulated in sipnet.yml, so no further modifications were needed. Let me know if anything else is required! |
| @@ -61,4 +61,4 @@ jobs: | |||
| uses: ./.github/workflows/sipnet.yml | |||
| with: | |||
| R-version: ${{ matrix.R }} | |||
| secrets: inherit | |||
| secrets: inherit | |||
There was a problem hiding this comment.
I don't understand this code diff, what has changed here? did you add a extra space or something ? if so why?
There was a problem hiding this comment.
i am also not sure,may be while coding there is some unnecessary spaces that came.I will revert it.
.github/workflows/sipnet.yml
Outdated
| run: | | ||
| cd ${GITHUB_WORKSPACE}/sipnet | ||
| make sipnet | ||
| curl -L -o sipnet-linux "https://drive.google.com/uc?export=download&id=1sRTFfs9Z9osmMrtLlwVTqxeUDxgVlrEv" |
There was a problem hiding this comment.
@dlebauer @infotroph are we sure that this drive link will always be live for our project ? joining mid way so do not have the best context
There was a problem hiding this comment.
That was just a placeholder until I got builds posted with different releases, the latest is 1.3.0 https://github.com/PecanProject/sipnet/releases/tag/v1.3.0
This will always resolve to the latest release https://github.com/PecanProject/sipnet/releases/latest but the version number in the binary name will change.
There was a problem hiding this comment.
This should be the Linux build https://github.com/PecanProject/sipnet/releases/download/v1.3.0/sipnet-linux-v1.3.0
| cd ${GITHUB_WORKSPACE}/sipnet | ||
| make sipnet | ||
| curl -L -o sipnet-linux "https://drive.google.com/uc?export=download&id=1sRTFfs9Z9osmMrtLlwVTqxeUDxgVlrEv" | ||
| chmod +x sipnet-linux |
There was a problem hiding this comment.
Do we make use of sipnet-linux while building ?
There was a problem hiding this comment.
#3427 here the binary shared for linux i have updated that.
AritraDey-Dev
force-pushed
the
fix_pre-built-sipnet
branch
from
606f9fa to
17a0d0b
Compare
AritraDey-Dev
force-pushed
the
fix_pre-built-sipnet
branch
from
de53a10 to
6fd5195
Compare
Fixes PecanProject#3427 Add CI workflow to build and upload SIPNET binary as an artifact and update PEcAn workflow to use pre-built SIPNET binary. * **ci.yml** - Add steps to build SIPNET binary. - Add steps to upload SIPNET binary as an artifact. * **sipnet.yml** - Remove steps to check out and build SIPNET from source. - Add steps to download pre-built SIPNET binary. - Add steps to use pre-built SIPNET binary. --- For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/PecanProject/pecan/issues/3427?shareId=XXXX-XXXX-XXXX-XXXX).
- Added steps to download SIPNET binaries directly from Google Drive. - Removed the compilation step to streamline the workflow.
AritraDey-Dev
force-pushed
the
fix_pre-built-sipnet
branch
from
6fd5195 to
b685ef5
Compare
|
@dlebauer @allgandalf @infotroph I was thinking that how to get the binary file...I have solution please check how much this is possible.After building the binary file based on version or some other parameter,every time it will upload to some location(for now g-drive,or some other location) and then some kind of github Actions which will update the binary file in the sipnet.yml file.What do you think can something like this be done? |
Fixes #3427
Add CI workflow to build and upload SIPNET binary as an artifact and update PEcAn workflow to use pre-built SIPNET binary.
ci.yml
sipnet.yml