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

storage: applied(%d) is out of range [prevApplied(%d), committed(%d)] #36592

Closed
cockroach-teamcity opened this issue Apr 6, 2019 · 2 comments
Labels
C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. O-sentry Originated from an in-the-wild panic report.

Comments

@cockroach-teamcity
Copy link
Member

This issue was autofiled by Sentry. It represents a crash or reported error on a live cluster with telemetry enabled.

Sentry link: https://sentry.io/organizations/cockroach-labs/issues/974738775/?project=164528&referrer=webhooks_plugin

Panic message:

stopper.go:182: string

Stacktrace (expand for inline code snippets):

/usr/local/go/src/runtime/asm_amd64.s#L572-L574 in runtime.call32
/usr/local/go/src/runtime/panic.go#L501-L503 in runtime.gopanic

log.ErrorfDepth(r.ctx, 1, format, v...)
panic(fmt.Sprintf(format, v...))
}
in pkg/storage.(*raftLogger).Panicf
https://github.com/cockroachdb/cockroach/blob/4523f9819d2dde9b054fc8faf1568e107f130986/vendor/go.etcd.io/etcd/raft/log.go#L211-L213 in vendor/go.etcd.io/etcd/raft.(*raftLog).appliedTo
https://github.com/cockroachdb/cockroach/blob/4523f9819d2dde9b054fc8faf1568e107f130986/vendor/go.etcd.io/etcd/raft/raft.go#L355-L357 in vendor/go.etcd.io/etcd/raft.newRaft
https://github.com/cockroachdb/cockroach/blob/4523f9819d2dde9b054fc8faf1568e107f130986/vendor/go.etcd.io/etcd/raft/rawnode.go#L78-L80 in vendor/go.etcd.io/etcd/raft.NewRawNode
ctx := r.AnnotateCtx(context.TODO())
raftGroup, err := raft.NewRawNode(newRaftConfig(
raft.Storage((*replicaRaftStorage)(r)),
in pkg/storage.(*Replica).withRaftGroupLocked
if err := r.withRaftGroupLocked(true, func(raftGroup *raft.RawNode) (bool, error) {
return true, nil
in pkg/storage.(*Replica).maybeInitializeRaftGroup
// If the internal Raft group is not initialized, create it and wake the leader.
r.maybeInitializeRaftGroup(ctx)
in pkg/storage.(*Replica).sendWithRangeID
) (*roachpb.BatchResponse, *roachpb.Error) {
return r.sendWithRangeID(ctx, r.RangeID, ba)
}
in pkg/storage.(*Replica).Send
ba.Add(leaseReq)
_, pErr = p.repl.Send(ctx, ba)
}
in pkg/storage.(*pendingLeaseRequest).requestLeaseAsync.func2
f(ctx)
}()
in pkg/util/stop.(*Stopper).RunAsyncTask.func1

/usr/local/go/src/runtime/asm_amd64.s in runtime.call32 at line 573
/usr/local/go/src/runtime/panic.go in runtime.gopanic at line 502
pkg/storage/raft.go in pkg/storage.(*raftLogger).Panicf at line 106
vendor/go.etcd.io/etcd/raft/log.go in vendor/go.etcd.io/etcd/raft.(*raftLog).appliedTo at line 212
vendor/go.etcd.io/etcd/raft/raft.go in vendor/go.etcd.io/etcd/raft.newRaft at line 356
vendor/go.etcd.io/etcd/raft/rawnode.go in vendor/go.etcd.io/etcd/raft.NewRawNode at line 79
pkg/storage/replica.go in pkg/storage.(*Replica).withRaftGroupLocked at line 545
pkg/storage/replica.go in pkg/storage.(*Replica).maybeInitializeRaftGroup at line 1956
pkg/storage/replica.go in pkg/storage.(*Replica).sendWithRangeID at line 1997
pkg/storage/replica.go in pkg/storage.(*Replica).Send at line 1969
pkg/storage/replica_range_lease.go in pkg/storage.(*pendingLeaseRequest).requestLeaseAsync.func2 at line 317
pkg/util/stop/stopper.go in pkg/util/stop.(*Stopper).RunAsyncTask.func1 at line 324
Tag Value
Cockroach Release v2.1.1
Cockroach SHA: 4523f98
Platform linux amd64
Distribution CCL
Environment v2.1.1
Command server
Go Version go1.10.3
# of CPUs 4
# of Goroutines 152
@cockroach-teamcity cockroach-teamcity added C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. O-sentry Originated from an in-the-wild panic report. labels Apr 6, 2019
@tbg
Copy link
Member

tbg commented Apr 17, 2019

applied(%d) is out of range [prevApplied(%d), committed(%d)]

@tbg tbg changed the title sentry: stopper.go:182: string storage: applied(%d) is out of range [prevApplied(%d), committed(%d)] Apr 17, 2019
@nvanbenschoten
Copy link
Member

Merging into #41144, as this still looks possible as of v19.1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. O-sentry Originated from an in-the-wild panic report.
Projects
None yet
Development

No branches or pull requests

3 participants