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

influxdb,lexicon: fix builds #91180

Merged
merged 11 commits into from Jun 22, 2020
Merged

influxdb,lexicon: fix builds #91180

merged 11 commits into from Jun 22, 2020

Conversation

@jonringer
Copy link
Contributor

jonringer commented Jun 20, 2020

Motivation for this change

fix lexicon and influxdb builds, which were failing on a different review

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.
@jonringer
Copy link
Contributor Author

jonringer commented Jun 20, 2020

https://github.com/NixOS/nixpkgs/pull/91180
6 packages failed to build:
ape pantalaimon python37Packages.nvchecker python37Packages.sunpy python38Packages.nvchecker python38Packages.sunpy

22 packages built:
lexicon python27Packages.influxdb python27Packages.requests-file python27Packages.tldextract python37Packages.aioresponses python37Packages.graphite_api python37Packages.influxdb python37Packages.influxgraph python37Packages.requests-file python37Packages.structlog python37Packages.tldextract python37Packages.zeep python38Packages.aioresponses python38Packages.graphite_api python38Packages.influxdb python38Packages.influxgraph python38Packages.requests-file python38Packages.structlog python38Packages.tldextract python38Packages.zeep qutebrowser sewer

a few of the packages are fixed by #89694 , going to merge that PR first

@jonringer jonringer force-pushed the jonringer:fix-influx branch from 8fa8bfd to 65db06e Jun 20, 2020
@ofborg ofborg bot requested a review from costrouc Jun 20, 2020
@gloaming
Copy link
Contributor

gloaming commented Jun 20, 2020

LGTM

https://github.com/NixOS/nixpkgs/pull/91180
6 package failed to build:
ape python37Packages.nvchecker python37Packages.sunpy python38Packages.nvchecker python38Packages.sunpy qutebrowser

21 package were built:
lexicon python27Packages.influxdb python27Packages.requests-file python27Packages.tldextract python37Packages.aioftp python37Packages.graphite_api python37Packages.influxdb python37Packages.influxgraph python37Packages.parfive python37Packages.requests-file python37Packages.structlog python37Packages.tldextract python38Packages.aioftp python38Packages.graphite_api python38Packages.influxdb python38Packages.influxgraph python38Packages.parfive python38Packages.requests-file python38Packages.structlog python38Packages.tldextract sewer
[nix-shell:~/.cache/nix-review/pr-91180]$ nix run nixpkgs.influxdb
[2 copied (168.6 MiB), 32.7 MiB DL]
[craig@samwise:~/.cache/nix-review/pr-91180]$ influxd &
[...]
[craig@samwise:~/.cache/nix-review/pr-91180]$ python3
Python 3.8.3 (default, May 13 2020, 19:59:26) 
[GCC 9.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import influxdb
>>> c = influxdb.client.InfluxDBClient()
>>> c.ping()
[httpd] ::1 - root [20/Jun/2020:21:37:50 +0100] "GET /ping HTTP/1.1" 204 0 "-" "python-requests/2.23.0" e8837e13-b335-11ea-8002-e0d55ee7fa14 43
'1.8.0'
[nix-shell:~/.cache/nix-review/pr-91180-1]$ lexicon --help
usage: lexicon [-h] [--version] [--delegated DELEGATED] [--config-dir CONFIG_DIR]
@jonringer
Copy link
Contributor Author

jonringer commented Jun 20, 2020

doing a few more package cleanups then I'll merge

@jonringer jonringer force-pushed the jonringer:fix-influx branch from 5ee2a92 to c35340a Jun 20, 2020
@jonringer jonringer force-pushed the jonringer:fix-influx branch from 140accd to 42e508a Jun 20, 2020
@ofborg ofborg bot requested a review from smaret Jun 20, 2020
jonringer added 11 commits Jun 20, 2020
@jonringer jonringer force-pushed the jonringer:fix-influx branch from 42e508a to 10ef4d5 Jun 22, 2020
@jonringer
Copy link
Contributor Author

jonringer commented Jun 22, 2020

@jonringer
Copy link
Contributor Author

jonringer commented Jun 22, 2020

https://github.com/NixOS/nixpkgs/pull/91180
5 packages failed to build:
ape python37Packages.nvchecker python37Packages.sunpy python38Packages.nvchecker python38Packages.sunpy

39 packages built:
lexicon python27Packages.requests-file python27Packages.tldextract python37Packages.aioftp python37Packages.aplpy python37Packages.asdf python37Packages.extension-helpers python37Packages.graphite_api python37Packages.influxgraph python37Packages.parfive python37Packages.pyregion python37Packages.pytest-astropy python37Packages.pytest-astropy-header python37Packages.pytest-filter-subpackage python37Packages.radio_beam python37Packages.reproject python37Packages.requests-file python37Packages.spectral-cube python37Packages.structlog python37Packages.tldextract python38Packages.aioftp python38Packages.aplpy python38Packages.asdf python38Packages.extension-helpers python38Packages.graphite_api python38Packages.influxgraph python38Packages.parfive python38Packages.pyregion python38Packages.pytest-astropy python38Packages.pytest-astropy-header python38Packages.pytest-filter-subpackage python38Packages.radio_beam python38Packages.reproject python38Packages.requests-file python38Packages.spectral-cube python38Packages.structlog python38Packages.tldextract qutebrowser sewer
@jonringer jonringer merged commit b5badcf into NixOS:master Jun 22, 2020
14 checks passed
14 checks passed
Evaluation Performance Report Evaluator Performance Report
Details
grahamcofborg-eval ^.^!
Details
grahamcofborg-eval-check-maintainers matching changed paths to changed attrs...
Details
grahamcofborg-eval-check-meta config.nix: checkMeta = true
Details
grahamcofborg-eval-darwin nix-instantiate --arg nixpkgs { outPath=./.; revCount=999999; shortRev="10ef4d5"; rev="10ef4d508760dbe2ca6f6260d21860ef36f0f338"; } ./pkgs/t
Details
grahamcofborg-eval-lib-tests nix-build --arg pkgs import ./. {} ./lib/tests/release.nix
Details
grahamcofborg-eval-nixos nix-instantiate --arg nixpkgs { outPath=./.; revCount=999999; shortRev="10ef4d5"; rev="10ef4d508760dbe2ca6f6260d21860ef36f0f338"; } ./nixos/
Details
grahamcofborg-eval-nixos-manual nix-instantiate --arg nixpkgs { outPath=./.; revCount=999999; shortRev="10ef4d5"; rev="10ef4d508760dbe2ca6f6260d21860ef36f0f338"; } ./nixos/
Details
grahamcofborg-eval-nixos-options nix-instantiate --arg nixpkgs { outPath=./.; revCount=999999; shortRev="10ef4d5"; rev="10ef4d508760dbe2ca6f6260d21860ef36f0f338"; } ./nixos/
Details
grahamcofborg-eval-nixpkgs-manual nix-instantiate --arg nixpkgs { outPath=./.; revCount=999999; shortRev="10ef4d5"; rev="10ef4d508760dbe2ca6f6260d21860ef36f0f338"; } ./pkgs/t
Details
grahamcofborg-eval-nixpkgs-tarball nix-instantiate --arg nixpkgs { outPath=./.; revCount=999999; shortRev="10ef4d5"; rev="10ef4d508760dbe2ca6f6260d21860ef36f0f338"; } ./pkgs/t
Details
grahamcofborg-eval-nixpkgs-unstable-jobset nix-instantiate --arg nixpkgs { outPath=./.; revCount=999999; shortRev="10ef4d5"; rev="10ef4d508760dbe2ca6f6260d21860ef36f0f338"; } ./pkgs/t
Details
grahamcofborg-eval-package-list nix-env -qa --json --file .
Details
grahamcofborg-eval-package-list-no-aliases nix-env -qa --json --file . --arg config { allowAliases = false; }
Details
@jonringer jonringer deleted the jonringer:fix-influx branch Jun 22, 2020
@jonringer
Copy link
Contributor Author

jonringer commented Jun 22, 2020

seeing as almost all of these files were failures previously, I say this is some progress. Other failures can be addressed in other PRs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.