Closed AlexAxthelm closed 4 months ago
Commit time | Git sha | Project code | Holdings date | Language | Peer group | Report | Image |
---|---|---|---|---|---|---|---|
2024-05-15T17:07:46Z | acf272011af271037074cc9c3f408b575a882a08, | GENERAL | 2022Q4 | EN | other | Report | transitionmonitordockerregistry.azurecr.io/rmi_pacta_2022q4_general:20240515T171342Z |
2024-05-15T17:07:46Z | acf272011af271037074cc9c3f408b575a882a08, | GENERAL | 2023Q4 | EN | other | Report | transitionmonitordockerregistry.azurecr.io/rmi_pacta_2023q4_general:20240515T171340Z |
2024-05-15T17:07:46Z | acf272011af271037074cc9c3f408b575a882a08, | PA2024CH | 2023Q4 | DE | asset_manager | Report | transitionmonitordockerregistry.azurecr.io/rmi_pacta_2023q4_pa2024ch:20240515T171341Z |
2024-05-15T17:07:46Z | acf272011af271037074cc9c3f408b575a882a08, | PA2024CH | 2023Q4 | DE | civil_society | Report | transitionmonitordockerregistry.azurecr.io/rmi_pacta_2023q4_pa2024ch:20240515T171341Z |
2024-05-15T17:07:46Z | acf272011af271037074cc9c3f408b575a882a08, | PA2024CH | 2023Q4 | DE | other | Report | transitionmonitordockerregistry.azurecr.io/rmi_pacta_2023q4_pa2024ch:20240515T171341Z |
2024-05-15T17:07:46Z | acf272011af271037074cc9c3f408b575a882a08, | PA2024CH | 2023Q4 | DE | pension_fund | Report | transitionmonitordockerregistry.azurecr.io/rmi_pacta_2023q4_pa2024ch:20240515T171341Z |
2024-05-15T17:07:46Z | acf272011af271037074cc9c3f408b575a882a08, | PA2024CH | 2023Q4 | DE | politician | Report | transitionmonitordockerregistry.azurecr.io/rmi_pacta_2023q4_pa2024ch:20240515T171341Z |
2024-05-15T17:07:46Z | acf272011af271037074cc9c3f408b575a882a08, | PA2024CH | 2023Q4 | DE | researcher | Report | transitionmonitordockerregistry.azurecr.io/rmi_pacta_2023q4_pa2024ch:20240515T171341Z |
2024-05-15T17:07:46Z | acf272011af271037074cc9c3f408b575a882a08, | PA2024CH | 2023Q4 | EN | asset_manager | Report | transitionmonitordockerregistry.azurecr.io/rmi_pacta_2023q4_pa2024ch:20240515T171341Z |
2024-05-15T17:07:46Z | acf272011af271037074cc9c3f408b575a882a08, | PA2024CH | 2023Q4 | EN | civil_society | Report | transitionmonitordockerregistry.azurecr.io/rmi_pacta_2023q4_pa2024ch:20240515T171341Z |
2024-05-15T17:07:46Z | acf272011af271037074cc9c3f408b575a882a08, | PA2024CH | 2023Q4 | EN | other | Report | transitionmonitordockerregistry.azurecr.io/rmi_pacta_2023q4_pa2024ch:20240515T171341Z |
2024-05-15T17:07:46Z | acf272011af271037074cc9c3f408b575a882a08, | PA2024CH | 2023Q4 | EN | pension_fund | Report | transitionmonitordockerregistry.azurecr.io/rmi_pacta_2023q4_pa2024ch:20240515T171341Z |
2024-05-15T17:07:46Z | acf272011af271037074cc9c3f408b575a882a08, | PA2024CH | 2023Q4 | EN | politician | Report | transitionmonitordockerregistry.azurecr.io/rmi_pacta_2023q4_pa2024ch:20240515T171341Z |
2024-05-15T17:07:46Z | acf272011af271037074cc9c3f408b575a882a08, | PA2024CH | 2023Q4 | EN | researcher | Report | transitionmonitordockerregistry.azurecr.io/rmi_pacta_2023q4_pa2024ch:20240515T171341Z |
2024-05-15T17:07:46Z | acf272011af271037074cc9c3f408b575a882a08, | PA2024CH | 2023Q4 | FR | asset_manager | Report | transitionmonitordockerregistry.azurecr.io/rmi_pacta_2023q4_pa2024ch:20240515T171341Z |
2024-05-15T17:07:46Z | acf272011af271037074cc9c3f408b575a882a08, | PA2024CH | 2023Q4 | FR | civil_society | Report | transitionmonitordockerregistry.azurecr.io/rmi_pacta_2023q4_pa2024ch:20240515T171341Z |
2024-05-15T17:07:46Z | acf272011af271037074cc9c3f408b575a882a08, | PA2024CH | 2023Q4 | FR | other | Report | transitionmonitordockerregistry.azurecr.io/rmi_pacta_2023q4_pa2024ch:20240515T171341Z |
2024-05-15T17:07:46Z | acf272011af271037074cc9c3f408b575a882a08, | PA2024CH | 2023Q4 | FR | pension_fund | Report | transitionmonitordockerregistry.azurecr.io/rmi_pacta_2023q4_pa2024ch:20240515T171341Z |
2024-05-15T17:07:46Z | acf272011af271037074cc9c3f408b575a882a08, | PA2024CH | 2023Q4 | FR | politician | Report | transitionmonitordockerregistry.azurecr.io/rmi_pacta_2023q4_pa2024ch:20240515T171341Z |
2024-05-15T17:07:46Z | acf272011af271037074cc9c3f408b575a882a08, | PA2024CH | 2023Q4 | FR | researcher | Report | transitionmonitordockerregistry.azurecr.io/rmi_pacta_2023q4_pa2024ch:20240515T171341Z |
2024-05-15T17:07:46Z | acf272011af271037074cc9c3f408b575a882a08 | ghcr.io/rmi-pacta/workflow.transition.monitor:pr313 |
Actually, those build args were added when I added the {pak} installation in #199, and they should work now unless something broke and I didn't realize.
I'm not totally opposed to this change, but the difference is:
Is that desired or at least understood?
Actually, those build args were added when I added the {pak} installation in #199, and they should work now unless something broke and I didn't realize.
I'm not totally opposed to this change, but the difference is:
- in the current state, one can build the Docker image from main using alternate versions of our packages by specifying the specific sha/branch/tag desired in the CLI docker build command
- after this PR, one will not be able to build a Docker image from a clean version of main, they will have to create a PR branch with a modified Dockerfile (or use main with a modified Dockerfile)
Is that desired or at least understood?
what you have here is an accurate description of the situation after this PR.
I think it's preferable to be able to connect a given docker build to the codebase that it connects to, rather than the build args that were passed in.
Additionally, this makes it easier to integrate updated dependencies (the pacta.* packages) on development branches here, and have the CI pipeline pick them up.
Removes uneeded build args from Dockerfile.
The build args are holdovers from before we used
pak
to manage and install dependencies, butpak
allows us to specify non-main
branches in theRemotes:
ofDESCRIPTION
, meaning that the docker image is dependent upon what is committed in code, rather than arguments passed tot he process at build time.As an example of how to use a non-
main
(technically, non-default GH branch) dependency:we might change this example DESCRIPTION:
to read as:
which would use
some-new-branch
as the ref forpacta.executive.summary
, but still use the default branch forpacta.portfolio.allocate
See also: https://pak.r-lib.org/reference/pak_package_sources.html#github-packages-github-