-
Notifications
You must be signed in to change notification settings - Fork 15
Permalink
Loading
Choose a base ref
{{ refName }}
default
Loading
Choose a head ref
{{ refName }}
default
Comparing changes
Choose two branches to see what’s changed or to start a new pull request.
If you need to, you can also or
learn more about diff comparisons.
Open a pull request
Create a new pull request by comparing changes across two branches. If you need to, you can also .
Learn more about diff comparisons here.
base repository: jetify-com/devbox-install-action
base: v0.5.0
Could not load branches
Nothing to show
Loading
Could not load tags
Nothing to show
{{ refName }}
default
Loading
...
head repository: jetify-com/devbox-install-action
compare: v0.7.0
Could not load branches
Nothing to show
Loading
Could not load tags
Nothing to show
{{ refName }}
default
Loading
- 11 commits
- 4 files changed
- 4 contributors
Commits on Jul 27, 2023
-
Configuration menu - View commit details
-
Copy full SHA for 1be8e96 - Browse repository at this point
Copy the full SHA 1be8e96View commit details
Commits on Aug 23, 2023
-
Configuration menu - View commit details
-
Copy full SHA for a739e51 - Browse repository at this point
Copy the full SHA a739e51View commit details
Commits on Aug 24, 2023
-
Configuration menu - View commit details
-
Copy full SHA for 5798c46 - Browse repository at this point
Copy the full SHA 5798c46View commit details
Commits on Sep 5, 2023
-
Temporary fix until jetify-com/devbox#1453 is released
Configuration menu - View commit details
-
Copy full SHA for a0bd99f - Browse repository at this point
Copy the full SHA a0bd99fView commit details
Commits on Sep 19, 2023
-
### Description macOS caching is broken. This PR fixes it. This is related to `gtar` not having `sudo` permission, but `/nix/store` requires sudo. There are many many related issues out there. eg. actions/cache#749 I also switched to using DeterminateSystems nix installer. However, I noticed that even though cache is successfully restored in the latest round of CICD, we are still fetching the binaries from remote? (at least based on the log lines) Why is that? I suspect that I'm missing some sqlite data in the cached files. cc @savil @gcurtis if any of you have insights into this. EDIT: SOLVED. Added xdg paths in the cache and it reduces the time by half. 🎉 🎉 🎉 closes #13
Configuration menu - View commit details
-
Copy full SHA for 572189b - Browse repository at this point
Copy the full SHA 572189bView commit details -
Configuration menu - View commit details
-
Copy full SHA for 1998394 - Browse repository at this point
Copy the full SHA 1998394View commit details -
Narrow the cache of xdg directory (#20)
Narrow the cache of xdg directory per request from @savil
Configuration menu - View commit details
-
Copy full SHA for 76d8a64 - Browse repository at this point
Copy the full SHA 76d8a64View commit details
Commits on Sep 21, 2023
-
Configuration menu - View commit details
-
Copy full SHA for c1d7077 - Browse repository at this point
Copy the full SHA c1d7077View commit details
Commits on Sep 22, 2023
-
Restores the tar command to its original state (#21)
### What This PR tries to reduce side effects by: - Restores the `tar` command to its original state - Narrow the ~/.cache directory to just nix and devbox ### Why We are using `chmod u+s "$(command -v tar)"` to setuid on tar (This is needed because we are caching `/nix/` and other directories outside of `~`, which is owned by `root`) This has a problem where it changes the parent directory of `~/.cache/nix` to be owned by `root` instead of `runner`. I thought #22 had fixed it, but we ran into similar issues when we use a go cache on top of the devbox cache. ### Caveat Cache enabled on self hosted Linux may stop working
Configuration menu - View commit details
-
Copy full SHA for f013611 - Browse repository at this point
Copy the full SHA f013611View commit details
Commits on Sep 27, 2023
-
We have some problems in our custom runner with this action so I've tried it with skipping the installation via the included action and it works in our runners (we do not use the GitHub provided ones, we are using custom runners). In case you are wondering which action we are using, we use: `cachix/install-nix-action@v23` - I guess it has something to do with the usage of `sudo` in the other action but I'm not 100% sure on that. I'm also not sure about the differences between these two actions so I've opted for the possibility to skip the installation rather than changing the action as I guess you've put a lot of thought into which action to use. Edit: I need to correct myself, it's not because of `sudo` but because of `systemd`. I tried to run the action without `systemd` by passing in `init: none` and `planner: linux-multi` but I got other errors. So I would prefer to be able to just skip the installation and use my own / a different action to install `nix`.
Configuration menu - View commit details
-
Copy full SHA for 6b09349 - Browse repository at this point
Copy the full SHA 6b09349View commit details
Commits on Oct 4, 2023
-
Replace github cache action with restore and save (#24)
### What Instead of letting `action/cache` do the restoring and saving cache for us, we choose when to restore and when to save cache. In this case, we will be saving cache immediately after the devbox package installation, instead of waiting for the action post run. This way, the `tar` sudo permission stays intact until the cache is saved. More specifically, `action/cache/save` runs before the last step of restoring `tar` to its original permission (as oppose to after)
Configuration menu - View commit details
-
Copy full SHA for e523b1c - Browse repository at this point
Copy the full SHA e523b1cView commit details
Loading
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff v0.5.0...v0.7.0