-
Notifications
You must be signed in to change notification settings - Fork 26
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
service gets stuck and doesn't update anymore after some time of uninterrupted operation #106
Comments
Any ideas on how this can happen? Maybe a caching issue? |
Anything useful logged? Does restarting the deployment help? |
It does for a few weeks and then it happens again. |
Currently, everything is up to date (presumably due to the merge of #107). Let's see, how long it lasts. |
It happened again |
So I guess it makes sense to assume that both #156 and #145 are somehow related to this? As @kid pointed out, the logs show only a few error messages regarding timeouts, but nothing beyond that. Maybe it makes sense to try and locate the problem methodically:
|
If we were hitting the memory limit the process would have been killed with signal 9 and I would expect the thing to restart automatically and to be working again. |
Couldn't it just be the cache getting bigger over time, as we'd expect? |
there it goes again |
I've tried running deps.rs locally with the following patches: deps.rsdiff --git a/Cargo.lock b/Cargo.lock
index 9721623..4aeff6a 100644
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -245,8 +245,6 @@ dependencies = [
[[package]]
name = "crates-index"
version = "2.10.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "8467f424ab6d70e5daf384289fa68247787effc13901dd0ca46bc9b4a62f1474"
dependencies = [
"gix",
"hex",
diff --git a/Cargo.toml b/Cargo.toml
index 746daaf..d8f3589 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -16,7 +16,7 @@ badge = { path = "./libs/badge" }
anyhow = "1"
cadence = "1"
-crates-index = { version = "2", default-features = false, features = ["git"] }
+crates-index = { path = "../rust-crates-index", default-features = false, features = ["git"] }
derive_more = "0.99"
font-awesome-as-a-crate = "0.3"
futures-util = { version = "0.3", default-features = false, features = ["std"] } rust-crates-indexdiff --git a/src/error.rs b/src/error.rs
index 7ae1d27..5f06ee2 100644
--- a/src/error.rs
+++ b/src/error.rs
@@ -7,7 +7,7 @@ pub use toml::de::Error as TomlDeError;
#[derive(Debug, thiserror::Error)]
#[allow(missing_docs)]
pub enum Error {
- #[error("\"gix\" crate failed. If problems persist, consider deleting `~/.cargo/registry/index/github.com-1ecc6299db9ec823/`")]
+ #[error("\"gix\" crate failed. If problems persist, consider deleting `~/.cargo/registry/index/github.com-1ecc6299db9ec823/` {0:?}")]
#[cfg(feature = "git")]
Git(#[from] GixError),
#[error("{0}")] The logs were the following:
|
This reads to me like it's having issues with the repo collapses. I ran current master locally and saw the same kind of errors as you, Paolo. Following the advice of the error message (deleting Possible solutions and workarounds:
|
refactoring to sparse index wasn't too much work: #227 |
After a few weeks of deps.rs running without new deploys or restarts it looks like the updater for popular crates and popular repos gets stuck and doesn't update anymore. This has happened multiple times now.
This is easily noticeable when serde releases a new version, since the homepage keeps pointing to the older version, causing the badge to report 1 outdated dependency (
serde
imports an exact version ofserde-derive
).deps.rs
local instance
The text was updated successfully, but these errors were encountered: