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

1309 commits

This branch
This branch
All branches
Author SHA1 Message Date
Daiderd Jordan 40c759e027
add build for ofborg-simple-build 2020-04-25 14:32:42 +02:00
Daiderd Jordan 96639e9d7e
work around carnix workspace issue 
Carnix doesn't properly handle includes for workspace members, this
works around the issue by specifying what should be generated directly
in Cargo.toml.

https://nest.pijul.com/pmeunier/carnix/discussions/46
 2020-04-25 14:28:29 +02:00
Daiderd Jordan 0139cd5621
move carnix files to workspace 2020-04-25 13:54:57 +02:00
Graham Christensen 041d96d890
Merge pull request #470 from cole-h/missing-variants 
ghevent: add missing variants
 2020-04-24 17:33:06 -04:00
Graham Christensen b4afb127df
Merge pull request #469 from LnL7/cargo-workspace 
introduce cargo workspace
 2020-04-24 17:24:42 -04:00
Cole Helbling db165abf99
ghevent: add missing variants 
So we don't see log messages that are literal pages long. Pulled from
here: https://developer.github.com/v3/activity/events/types/#pullrequestevent
 2020-04-24 14:18:21 -07:00
Daiderd Jordan 7ccb4943a1
move simple-build to a separate project 
This is mostly intended as an example and since this is only useful for
testing it should be relatively safe.
 2020-04-24 22:17:12 +02:00
Daiderd Jordan 1c0c5e304c
init cargo workspace 
Enables splittig up the code a bit more, and hopefully helps keep the
compilation time down.
 2020-04-24 21:27:33 +02:00
Graham Christensen 329a08f53d
Merge pull request #468 from emilazy/trust-emily 
config.public.json: add emilazy to trusted_users
 2020-04-23 15:19:08 -04:00
Graham Christensen d823531dea
Merge pull request #425 from marsam/marsam-trusted-users 
config.public.json: add marsam to trusted_users
 2020-04-23 15:18:56 -04:00
Emily 650f0912d8 config.public.json: add emilazy to trusted_users 2020-04-23 20:17:11 +01:00
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