danderson/ofborg
0
0
Fork 0
forked from the-distro/ofborg
Code Pull requests Activity
1298 commits 1 branch 0 tags 2.6 MiB
Commit graph

1298 commits

This branch
This branch
All branches
Author SHA1 Message Date
Graham Christensen b3a3b4b0ee
Merge pull request #467 from cole-h/unbump 
Revert "Bump amqp"
 2020-04-23 11:39:21 -04:00
Cole Helbling a3475d1c0d
Revert "Bump amqp" 
This reverts commit fa299dedaa.
 2020-04-23 08:21:34 -07:00
Graham Christensen 98455feb18
Merge pull request #464 from cole-h/bump 
Bump amqp
 2020-04-22 16:24:24 -04:00
Cole Helbling fa299dedaa
Bump amqp 
Hopefully this will deal with (or at least help address) our
`SendError(..)` panics and log collector unavailability.
 2020-04-22 09:53:53 -07:00
Graham Christensen 0581cc59ed
Merge pull request #462 from doronbehar/patch-1 
config.public.json: add @doronbehar
 2020-04-20 07:49:02 -04:00
Graham Christensen b8e39f7d22
Merge pull request #460 from cole-h/nix1-removal 
Nix1 removal
 2020-04-20 07:48:51 -04:00
Doron Behar bd80f1fe58
config.public.json: add @doronbehar 
Per @domenkozar 's comment:
https://github.com/NixOS/nixpkgs/issues/50105#issuecomment-614886167
 2020-04-17 12:35:24 +00:00
Cole Helbling f5e0318f9f
nix: fix failing test 
Nix 2.3.4 now shows the assertion that failed since NixOS/nix@307bcb9, changing
the message we need to check against.
 2020-04-12 11:06:16 -07:00
Cole Helbling 35bb3a8975
Remove nix1 
As of NixOS/nixpkgs@7121160, nix1 has been removed from Nixpkgs. Thus, we
remove it here.
 2020-04-12 11:04:40 -07:00
Graham Christensen a095c36668
Merge pull request #458 from cole-h/logging 
{checkout,clone}: redirect output to /dev/null
 2020-04-09 17:23:20 -04:00
Cole Helbling b33d3de6cf
tasks/evaluate: show issue number not id 
The `id` field is not the GitHub PR or issue number; the `number` field
is.
 2020-04-09 14:06:58 -07:00
Cole Helbling db190207a2
{checkout,clone}: redirect output to /dev/null 
It fills the logs with typically-useless information. We don't really
care about the novels people write in their commit messages.

The only `git` commands that were left unmodified are the ones where we
use their output in some way (and thus, it isn't printed to stdout/err
anyways).
 2020-04-09 13:47:10 -07:00
Graham Christensen 83d748169c
Merge pull request #457 from cole-h/logging 
More logging
 2020-04-07 16:39:28 -04:00
Cole Helbling 5de114c151
tasks/build: don't use \n to split log line 
Otherwise, we have one line that has the log prefix, and one line
immediately following that doesn't.
 2020-04-07 13:27:37 -07:00
Cole Helbling 5fd48be23d
tasks/eval/nixpkgs: don't request review from PR author 
GitHub doesn't let us, so we silently skip them.
 2020-04-07 13:27:21 -07:00
Graham Christensen 284a7bfeaa
Merge pull request #456 from NixOS/retryon-bad-creds 
Retry on bad creds
 2020-04-07 08:29:02 -04:00
Graham Christensen 3dc7498211
retry a job if we get an expired creds error 2020-04-07 08:16:08 -04:00
Graham Christensen e65a24aa89
commitstatuserror: handle each error case separately, not based on 'internal' error 2020-04-07 08:14:02 -04:00
Graham Christensen b4989e44f8
Merge pull request #455 from cole-h/logging 
Cargo.toml: enable debug symbols in release profile
 2020-04-06 16:18:53 -04:00
Cole Helbling 8e284c5b6b
Cargo.toml: enable debug symbols in release profile 
Maybe now we'll be able to read the backtraces generated by
`error-chain` via `hubcaps`.
 2020-04-06 13:01:40 -07:00
Graham Christensen dd38e6e2ec
Merge pull request #454 from NixOS/label-log 
labeling: make log messages one-line
 2020-04-06 12:42:00 -04:00
Graham Christensen 3bc29be475
labeling: make log messages one-line 2020-04-06 10:52:32 -04:00
Graham Christensen 8e6f78607b
Merge pull request #453 from NixOS/handle-errs-better 
Handle errs better
 2020-04-06 07:20:57 -04:00
Graham Christensen a1e8dcc1e5
evaluate: handle evaluation errors, and posting errors while posting eval errors better 2020-04-06 07:05:56 -04:00
Graham Christensen 00478e9dee
update_status: also return a CommitStatusError 2020-04-06 07:02:49 -04:00
Graham Christensen f02c7382cc
commitstatus: distinguish internal and external error factors 2020-04-06 07:02:45 -04:00
Graham Christensen 2541d183bf
Merge pull request #452 from cole-h/logging 
Logging cleanup
 2020-04-06 06:00:48 -04:00
Cole Helbling c9d4bb1fd8
bin/*: replace {e,}println!s with log macros 
May as well be consistent.
 2020-04-05 23:52:21 -07:00
Cole Helbling 1b8cec63f2
treewide: replace {e,}println!s with log macros 
Currently, our logs are a bit inconsistent, with some being
`{e,}println!`s and others utilizing macros from the `log` crate. One
line might look like `INFO:ofborg::tasks::evaluate: Removing labels:
[]`, while the next might look like `Already: []`.
 2020-04-05 23:52:21 -07:00
Cole Helbling 5c1a85ca31
treewide: replace {:#?} with {:?} 
Cleans up logging output so that things that belong together stay
together.
 2020-04-05 12:32:19 -07:00
Graham Christensen e26497e49d
Merge pull request #451 from LnL7/eval-no-nixpkgs 
rename nixpkgs entry in NIX_PATH
 2020-04-05 15:06:55 -04:00
Daiderd Jordan f6c22bfc4f
rename nixpkgs entry in NIX_PATH 
This ensures the local sources are still allowed in restricted mode, but
referencing <nixpkgs> in expressions inside nixpkgs is not.
 2020-04-05 20:09:34 +02:00
Graham Christensen b55535a608
Merge pull request #450 from cole-h/silence-diffs 
checkout: silence diffs when checking validity of a commit
 2020-04-05 13:27:30 -04:00
Cole Helbling a907ffb488
checkout: silence diffs when checking validity of a commit 
Otherwise, when people submit treewide PRs to nixpkgs, the logs get
cluttered with thousands of useless diffs.
 2020-04-05 09:44:19 -07:00
Graham Christensen 91f6aee4d5
Merge pull request #449 from cole-h/ignore-remove-file 
nixenv: warn when removing the stats file fails
 2020-04-03 20:45:55 -04:00
Cole Helbling 9736cf08fe
nixenv: remove unnecessary single quotes 
The debug representation of a `PathBuf` provides double quotes already.
 2020-04-03 17:37:21 -07:00
Cole Helbling 42b4bb2118
nixenv: warn when removing the stats file fails 
If the `nix` invocation fails, a stats file won't get created. We log a
warning, but it is generally safe to ignore this failure.
 2020-04-03 17:25:58 -07:00
Graham Christensen 0db6f70714
Merge pull request #448 from cole-h/bump 
Bump hubcaps
 2020-04-03 19:55:28 -04:00
Cole Helbling 2649658fe8
Bump hubcaps 
New hubcaps update makes `created_at` and `updated_at` fields optional,
because sometimes they just aren't there.
 2020-04-03 16:39:52 -07:00
Graham Christensen 85988113aa
Merge pull request #447 from cole-h/nix-env-describe 
nixenv: describe what went wrong when io fails
 2020-04-03 14:07:14 -04:00
Cole Helbling 9dc0fd123a
match -> map_err 2020-04-03 10:51:02 -07:00
Cole Helbling 2b2b675011
nixenv: describe what went wrong when io fails 
Instead of just propagating a raw `std::io::Error`, wrap that in our own
`Error` enum to provide more context for what actually went wrong.
 2020-04-03 10:34:01 -07:00
Graham Christensen f9086a62ec
upgrade hubcaps for nullable target URLs 2020-04-02 11:37:23 -04:00
Graham Christensen fa5d6425fe
Build release=false 2020-04-02 10:27:07 -04:00
Graham Christensen d4d4da9eee
Debug a bit more status errors 2020-04-02 09:31:10 -04:00
Graham Christensen 1c1ccc8c7e
Provide some debug info for if posting a status update fails 2020-04-02 09:12:40 -04:00
Graham Christensen 9548a495d5
Merge pull request #395 from ryantm/leaf 
add label for exactly 1 rebuild
 2020-04-01 20:44:25 -04:00
Graham Christensen bb7ca7e6ea
Merge pull request #433 from fgaz/move-fgaz-to-trusted-users 
Move fgaz to trusted users
 2020-04-01 20:43:43 -04:00
Graham Christensen c5239bcc2f
Merge pull request #424 from nh2/patch-1 
config.public.json: add nh2 to trusted_users
 2020-04-01 20:43:23 -04:00
Graham Christensen ba020b6383
Merge pull request #430 from worldofpeace/tag-cinnamon 
Tag cinnamon pkgs
 2020-04-01 20:42:30 -04:00