Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

nixops: update the PrettyTable package pin to accomodate the currently available version #281879

Closed
wants to merge 1 commit into from

Conversation

deepfire
Copy link
Contributor

@deepfire deepfire commented Jan 18, 2024

Description of changes

This allows a plugin-less nixops_unstable to be built on master. Tested with a SSH target.

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

@roberth
Copy link
Member

roberth commented Jan 18, 2024

Did nixops check still work? IIRC that command renders a table.

@deepfire
Copy link
Contributor Author

Did nixops check still work? IIRC that command renders a table.

Yep, just checked, does render a table.

@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/prs-ready-for-review/3032/3301

Copy link
Contributor

@datafoo datafoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried to test your PR but I could not get it to work.

I added a new flake input nixpkgsPr = { url = github:deepfire/nixpkgs/0-gh-nixops-fix; }; and used it to get nixpkgsPr.legacyPackages.${system}.nixops_unstable in my dev shell.

Unfortunately, my dev shell did not build:

$ nix develop 
warning: Git tree '/home/me/dev/mydeployments-dev' is dirty
error: builder for '/nix/store/vxphbk5v50spmw9kbw2wmmz5w0j561zj-python3.11-nixops-hetznercloud-unstable-2023-02-19.drv' failed with exit code 1;
       last 10 log lines:
       > * Getting build dependencies for wheel...
       > * Building wheel...
       > Successfully built nixops_hetznercloud-0.1.3-py3-none-any.whl
       > Finished creating a wheel...
       > Finished executing pypaBuildPhase
       > Running phase: pythonRuntimeDepsCheckHook
       > Executing pythonRuntimeDepsCheck
       > Checking runtime dependencies for nixops_hetznercloud-0.1.3-py3-none-any.whl
       >   - hcloud==1.18.2 not satisfied by version 1.33.2
       >   - typing-extensions<4.0.0,>=3.7.4 not satisfied by version 4.8.0
       For full logs, run 'nix log /nix/store/vxphbk5v50spmw9kbw2wmmz5w0j561zj-python3.11-nixops-hetznercloud-unstable-2023-02-19.drv'.
error: 1 dependencies of derivation '/nix/store/3w64q0ximicm8dlcbdhjyhh4l3jrgqxh-nixops-unstable-2023-12-17.drv' failed to build
error: 1 dependencies of derivation '/nix/store/cjj8akrn9lbrr8a1s3b4m0ar0s3b1fkk-mydeployment-repository-shell-env.drv' failed to build

$ nix log /nix/store/vxphbk5v50spmw9kbw2wmmz5w0j561zj-python3.11-nixops-hetznercloud-unstable-2023-02-19.drv
warning: The interpretation of store paths arguments ending in `.drv` recently changed. If this command is now failing try again with '/nix/store/vxphbk5v50spmw9kbw2wmmz5w0j561zj-python3.11-nixops-hetznercloud-unstable-2023-02-19.drv^*'
Sourcing python-remove-tests-dir-hook
Sourcing python-catch-conflicts-hook.sh
Sourcing python-remove-bin-bytecode-hook.sh
Sourcing pypa-build-hook
Using pypaBuildPhase
Sourcing python-runtime-deps-check-hook
Using pythonRuntimeDepsCheckHook
Sourcing pypa-install-hook
Using pypaInstallPhase
Sourcing python-imports-check-hook.sh
Using pythonImportsCheckPhase
Sourcing python-namespaces-hook
Sourcing python-catch-conflicts-hook.sh
@nix { "action": "setPhase", "phase": "unpackPhase" }
Running phase: unpackPhase
unpacking source archive /nix/store/yi4xv8h1dlyq7j5cffqssjih22r4kcb0-source
source root is source
setting SOURCE_DATE_EPOCH to timestamp 315619200 of file source/tests/functional/test_stop_stops.py
@nix { "action": "setPhase", "phase": "patchPhase" }
Running phase: patchPhase
@nix { "action": "setPhase", "phase": "updateAutotoolsGnuConfigScriptsPhase" }
Running phase: updateAutotoolsGnuConfigScriptsPhase
@nix { "action": "setPhase", "phase": "configurePhase" }
Running phase: configurePhase
no configure script, doing nothing
@nix { "action": "setPhase", "phase": "buildPhase" }
Running phase: buildPhase
Executing pypaBuildPhase
Creating a wheel...
* Getting build dependencies for wheel...
* Building wheel...
Successfully built nixops_hetznercloud-0.1.3-py3-none-any.whl
Finished creating a wheel...
Finished executing pypaBuildPhase
@nix { "action": "setPhase", "phase": "pythonRuntimeDepsCheckHook" }
Running phase: pythonRuntimeDepsCheckHook
Executing pythonRuntimeDepsCheck
Checking runtime dependencies for nixops_hetznercloud-0.1.3-py3-none-any.whl
  - hcloud==1.18.2 not satisfied by version 1.33.2
  - typing-extensions<4.0.0,>=3.7.4 not satisfied by version 4.8.0

@deepfire
Copy link
Contributor Author

@datafoo, there are, unfortunately, multiple problems with nixops_unstable -- indeed hetznercloud remains broken.

This is why I explicitly said:

This allows a plugin-less nixops_unstable to be built on master

@deepfire
Copy link
Contributor Author

@datafoo, also, if you want a solution to the problem -- #281883 allows you to easily request a less plugin-encumbered build of nixops.

@datafoo
Copy link
Contributor

datafoo commented Jan 22, 2024

You requested my review of this PR, how am I supposed to test it then?

@deepfire
Copy link
Contributor Author

You requested my review of this PR, how am I supposed to test it then?

@datafoo, one option is to wait until #281883 is merged.

Another option would be to cherry-pick this commit into #281883. @roberth what do you think?

@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/prs-ready-for-review/3032/3429

@roberth
Copy link
Member

roberth commented Feb 18, 2024

Let's do it upstream.

Sorry for the abysmal response time.

@deepfire
Copy link
Contributor Author

deepfire commented Feb 21, 2024

Let's do it upstream.

* [PrettyTable: ^0.7.2 -> ^3 nixops#1575](https://github.com/NixOS/nixops/pull/1575)

Sorry for the abysmal response time.

Sure, makes sense, closing.

@deepfire deepfire closed this Feb 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants