mirror/bgp-stayrtr
1
0
Fork 0
mirror of https://github.com/bgp/stayrtr.git synced 2024-05-06 15:54:54 +00:00
Code Issues Activity
268 Commits 27 Branches 9 Tags
cache_refactor
Commit Graph

268 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
Null User edd5801e1d fuck. 2022-02-09 13:20:14 +00:00
Null User 7000b5fdda Fix more test stuff 2022-02-09 13:16:11 +00:00
Null User afba875272 Remove cruft not even used by test case 2022-02-09 13:14:39 +00:00
Null User d88a28beb2 Delety mc Delete 2022-02-09 12:59:49 +00:00
Null User 3195e542b7 Moving cache more 2022-02-09 12:58:35 +00:00
Null User 179ebdd40a Wire up cache refactor stuff 2022-02-09 12:43:47 +00:00
Null User a9bc9674f5 Well look what the cat drug in. 2022-02-02 05:58:25 +00:00
Null User f2531db1e1 Shuffle shuffle shuffle refactor metrics 2022-02-02 05:15:05 +00:00
Null User a19658de54 Well, metrics are moved and it builds? 2022-02-02 04:48:24 +00:00
Job Snijders c4ac62586a Merge pull request #60 from BenCastricum/deadlock-fix 
Revert defer unlock in StayRTR AddVRPs
 2022-01-26 15:26:49 +01:00
Ben Castricum 4fef7114a3 Revert defer unlock in StayRTR AddVRPs 
vrplock needs to be unlocked before AddVRPsDiff() because AddVRPsDiff needs a full lock.

I added some debug logging found this deadlock

INFO[0000] new cache file: Updating sha256 hash  -> da753c7804d6f386bf303fed6931853eaaca0771ba160ef7fdbebb17e899d78b
INFO[0001] New update (306189 uniques, 306189 total prefixes).
INFO[0001] RLocking vrplock in AddVRPs
INFO[0002] RLocking vrplock in AddVRPsDiff
INFO[0002] RUnlocked vrplock in AddVRPsDiff
INFO[0002] Locking vrplock in AddVRPsDiff
...
 2022-01-26 11:20:46 +01:00
Job Snijders 1bf0956e25 Update README.md 2022-01-24 23:43:16 +00:00
Ties de Kock f43d23e354 Merge pull request #55 from bgp/fix/revert-defer-in-rtrmon 
Revert defer unlock in select/case
 2021-10-31 17:55:43 +01:00
Ties de Kock 1833aceccf Revert defer unlock in select/case 
Hindsight... Agree with lspgn here, it is confusing that there is
a non-local effect (lock being held) that you will only hit when
you add code later in the same function.

I do not like the style of the alternatives (anonymous functions
called immediately) either.
 2021-10-31 17:05:57 +01:00
Ties de Kock f0b864089d Merge pull request #52 from ties/fix/change-defers-in-rtrmon 
defer unlocks in rtrmon as well
 2021-10-31 11:29:48 +01:00
Ties de Kock 3822cf8807 Do not defer unlock 'over' channel communication 2021-10-31 11:26:09 +01:00
Ties de Kock 4ddfd7f215 defer unlocks in rtrmon iff unlock happens before return 2021-10-30 16:16:32 +02:00
Ties de Kock 992471aac2 Merge pull request #51 from mellowdrifter/master 
Use defers for mutexs
 2021-10-30 16:11:24 +02:00
Darren O'Connor c3045da803 Merge branch 'master' of github.com:mellowdrifter/stayrtr 2021-10-30 09:53:13 -04:00
Darren O'Connor 3726782f68 Use defers for locks 2021-10-30 09:52:55 -04:00
Job Snijders 406a710de1 Merge pull request #50 from ties/fix/doc-fix-typos-in-rtrmon-doc 
Fix a typo in the README section on rtrmon
 2021-10-29 08:44:43 +01:00
Ties de Kock 765f8a17d4 Fix a typo in the README section on rtrmon 2021-10-29 09:42:56 +02:00
randomthingsandstuff 5ee7f75622 Move slurm into json file 2021-10-29 04:46:29 +00:00
Darren O'Connor 644fb871a3 Move slurm into json file 2021-10-28 20:39:19 -04:00
randomthingsandstuff a875e54242 Remove unused cruft and add tests 2021-10-28 08:21:38 +00:00
Darren O'Connor e64644ff62 update to 1.16 2021-10-27 21:05:35 -04:00
Darren O'Connor 91228f65e3 remove unused 2021-10-27 20:59:40 -04:00
Job Snijders d5023c1899 Merge pull request #47 from mellowdrifter/master 
GOMAXPROCS
 2021-10-27 23:28:32 +01:00
Darren O'Connor 0b738b8ee2 GOMAXPROCS 2021-10-27 18:16:16 -04:00
randomthingsandstuff 65b170c7f2 Merge pull request #45 from bgp/update_fail 
Fix intial sync stuff
 2021-10-27 04:48:15 +00:00
randomthingsandstuff 1952c19b32 Merge pull request #46 from bgp/disable_cgo_build 
Invert the usual CGO_ENABLED behavior to default to off.
 2021-10-27 04:43:42 +00:00
Null User a9fde96027 Invert the usual CGO_ENABLED behavior to default to off. 2021-10-27 04:42:36 +00:00
Null User 5878317f6e Catch error and simplify the initial sync stuff 2021-10-27 04:36:47 +00:00
Null User 2455723c87 Make the refresh faster when update fails 2021-10-26 15:20:17 +00:00
Job Snijders 70aef8f20c Merge pull request #41 from bgp/cleanup 
Refresh and cleanup junk
 2021-10-26 15:30:45 +01:00
Job Snijders 614d6e027d Merge pull request #40 from bgp/crash 
Crash
 2021-10-26 15:25:52 +01:00
Null User 76eac7450a Refresh faster when the VRPs weren't downloaded on init 2021-10-26 14:14:55 +00:00
Null User 6332b3ee71 Fix the crash in #39 2021-10-26 13:37:56 +00:00
Null User cc539c43a5 Trim paths. Don't need all that cruft. 2021-10-26 11:34:39 +00:00
Null User e5d3cf3d07 Update dependencies 2021-10-26 11:15:29 +00:00
randomthingsandstuff 683f7f2369 Merge pull request #37 from mellowdrifter/master 
Don't abuse main() with too much crap. Just let it be an entrypoint.
 2021-10-26 06:42:17 +00:00
Darren O'Connor c55254b8ff Controversial 2021-10-25 21:26:49 -04:00
Job Snijders 1994189e1b Merge pull request #36 from mellowdrifter/master 
More test code
 2021-10-26 02:04:02 +01:00
Darren O'Connor 783f2eaf9f More test code 2021-10-25 20:52:41 -04:00
Job Snijders fb912e5c18 Merge pull request #35 from bgp/feature/add-codeql-actions 
Add codeql actions
 2021-10-25 20:07:31 +01:00
Ties de Kock 041a1c52f3 Remove ineffectual assign 2021-10-25 20:21:07 +02:00
Ties de Kock e5c5a99a0c check for err 2021-10-25 20:19:28 +02:00
Ties de Kock 0a9217aaad Use local variable for password, not argument 
Use the local variable (possibly containing a password from the
environment) for the SSH password.
 2021-10-25 20:16:16 +02:00
Ties de Kock a8759efa62 add codeql scanning 2021-10-25 19:54:48 +02:00
Ties de Kock fa2fb297c6 golang 1.16 2021-10-25 19:54:15 +02:00