jalil/webnsupdate
1
0
Fork 0
Code Issues 2 Pull requests 3 Releases Activity Actions

refactor(nsupdate): send all commands at once #100

Merged
jalil merged 1 commit from push-otlmuzolrztu into main 2025-02-05 23:17:17 +01:00
Conversation 2 Commits 1 Files changed 3 +96 -50
jalil commented 2025-02-05 22:40:44 +01:00
Owner
Copy link

This ensures nsupdate is only called once per IP update (even for both IPv4 and IPv6 in a single call).

This ensures `nsupdate` is only called once per IP update (even for both IPv4 and IPv6 in a single call).
jalil self-assigned this 2025-02-05 22:40:45 +01:00
jalil added 1 commit 2025-02-05 22:40:45 +01:00
refactor(nsupdate): send all commands at once
Some checks failed
/ build (push) Successful in 1m6s
Details
/ check (clippy) (push) Failing after 11s
Details
/ check (module-ipv4-only-test) (push) Successful in 30s
Details
/ check (module-ipv4-test) (push) Successful in 29s
Details
/ check (module-ipv6-only-test) (push) Successful in 29s
Details
/ check (module-ipv6-test) (push) Successful in 28s
Details
/ check (module-nginx-test) (push) Successful in 30s
Details
/ check (nextest) (push) Successful in 3s
Details
/ check (treefmt) (push) Successful in 3s
Details
/ report-size (push) Successful in 7s
Details
6ca7348391 
This ensures `nsupdate` is only called once per IP update (even for both
IPv4 and IPv6 in a single call).
forgejo-actions commented 2025-02-05 22:44:30 +01:00
First-time contributor
Copy link

Flake output sizes

Definitions:

  • Name: the name of the package/configuration.
  • Size: the closure size (size on disk/NAR size + all transitive dependencies).
  • NAR Size: the size of the build output (package without the dependencies).
  • [NAR] Size Change: the amount changed compared to the main branch.

Tips on reading this data:

  • For NixOS configurations you generally care only about the Size (closure size/size on disk).
    • Reduce the Size by disabling unneeded services/default packages.
  • For Packages you care about both the Size and the NAR Size.
    • Reduce the NAR Size by reducing the size of the build outputs, e.g. don't copy unnecessary data to the $out dir, optimize binaries for size, etc.
    • Reduce the Size by reducing the dependencies (e.g. buildInputs).
    • Don't worry too much about size, some dependencies are deduplicated, e.g. glibc adds ~40MiB to the Size, but is generally shared by ~every binary on the system, so, chances are, you are already including it from somewhere else and statically linking with e.g. musl is not gonna improve things.

Packages

Name Size Size Change NAR Size NAR Size Change
default 35MiB 6.6KiB 2.8MiB 6.6KiB
git-cliff 60MiB 0B 19MiB 0B
webnsupdate 35MiB 6.6KiB 2.8MiB 6.6KiB
# Flake output sizes **Definitions:** - `Name`: the name of the package/configuration. - `Size`: the closure size (size on disk/NAR size + all transitive dependencies). - `NAR Size`: the size of the build output (package without the dependencies). - `[NAR] Size Change`: the amount changed compared to the main branch. **Tips on reading this data:** - For NixOS configurations you generally care only about the `Size` (closure size/size on disk). - Reduce the `Size` by disabling unneeded services/default packages. - For Packages you care about both the `Size` and the `NAR Size`. - Reduce the `NAR Size` by reducing the size of the build outputs, e.g. don't copy unnecessary data to the $out dir, optimize binaries for size, etc. - Reduce the `Size` by reducing the dependencies (e.g. `buildInputs`). - Don't worry too much about size, some dependencies are deduplicated, e.g. `glibc` adds ~40MiB to the `Size`, but is generally shared by ~every binary on the system, so, chances are, you are already including it from somewhere else and statically linking with e.g. `musl` is not gonna improve things. # Packages | Name | Size | Size Change | NAR Size | NAR Size Change | |------|-----:|------------:|---------:|----------------:| | `default` | 35MiB | 6.6KiB | 2.8MiB | 6.6KiB | | `git-cliff` | 60MiB | 0B | 19MiB | 0B | | `webnsupdate` | 35MiB | 6.6KiB | 2.8MiB | 6.6KiB |
jalil force-pushed push-otlmuzolrztu from 6ca7348391 to b775f8e811 2025-02-05 22:47:18 +01:00 Compare
forgejo-actions commented 2025-02-05 22:50:46 +01:00
First-time contributor
Copy link

Flake output sizes

Definitions:

  • Name: the name of the package/configuration.
  • Size: the closure size (size on disk/NAR size + all transitive dependencies).
  • NAR Size: the size of the build output (package without the dependencies).
  • [NAR] Size Change: the amount changed compared to the main branch.

Tips on reading this data:

  • For NixOS configurations you generally care only about the Size (closure size/size on disk).
    • Reduce the Size by disabling unneeded services/default packages.
  • For Packages you care about both the Size and the NAR Size.
    • Reduce the NAR Size by reducing the size of the build outputs, e.g. don't copy unnecessary data to the $out dir, optimize binaries for size, etc.
    • Reduce the Size by reducing the dependencies (e.g. buildInputs).
    • Don't worry too much about size, some dependencies are deduplicated, e.g. glibc adds ~40MiB to the Size, but is generally shared by ~every binary on the system, so, chances are, you are already including it from somewhere else and statically linking with e.g. musl is not gonna improve things.

Packages

Name Size Size Change NAR Size NAR Size Change
default 35MiB 6.6KiB 2.8MiB 6.6KiB
git-cliff 60MiB 0B 19MiB 0B
webnsupdate 35MiB 6.6KiB 2.8MiB 6.6KiB
# Flake output sizes **Definitions:** - `Name`: the name of the package/configuration. - `Size`: the closure size (size on disk/NAR size + all transitive dependencies). - `NAR Size`: the size of the build output (package without the dependencies). - `[NAR] Size Change`: the amount changed compared to the main branch. **Tips on reading this data:** - For NixOS configurations you generally care only about the `Size` (closure size/size on disk). - Reduce the `Size` by disabling unneeded services/default packages. - For Packages you care about both the `Size` and the `NAR Size`. - Reduce the `NAR Size` by reducing the size of the build outputs, e.g. don't copy unnecessary data to the $out dir, optimize binaries for size, etc. - Reduce the `Size` by reducing the dependencies (e.g. `buildInputs`). - Don't worry too much about size, some dependencies are deduplicated, e.g. `glibc` adds ~40MiB to the `Size`, but is generally shared by ~every binary on the system, so, chances are, you are already including it from somewhere else and statically linking with e.g. `musl` is not gonna improve things. # Packages | Name | Size | Size Change | NAR Size | NAR Size Change | |------|-----:|------------:|---------:|----------------:| | `default` | 35MiB | 6.6KiB | 2.8MiB | 6.6KiB | | `git-cliff` | 60MiB | 0B | 19MiB | 0B | | `webnsupdate` | 35MiB | 6.6KiB | 2.8MiB | 6.6KiB |
jalil merged commit b775f8e811 into main 2025-02-05 23:17:17 +01:00
jalil deleted branch push-otlmuzolrztu 2025-02-05 23:17:17 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
jalil
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: jalil/webnsupdate#100
No description provided.
Delete branch "push-otlmuzolrztu"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?