Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(11)

Unified Diff: appengine/logdog/coordinator/endpoints/logs/util.go

Issue 1863973002: LogDog: Update to archival V2. (Closed) Base URL: https://github.com/luci/luci-go@grpcutil-errors
Patch Set: Code review comments, use Pub/Sub, archival staging, quality of life. Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: appengine/logdog/coordinator/endpoints/logs/util.go
diff --git a/appengine/logdog/coordinator/endpoints/logs/util.go b/appengine/logdog/coordinator/endpoints/logs/util.go
index 5ae7d73a5a28ac2e8f8df48251e07d08d454b44c..65694ce4e685c4ed863b6ace46b445ae36dc3a43 100644
--- a/appengine/logdog/coordinator/endpoints/logs/util.go
+++ b/appengine/logdog/coordinator/endpoints/logs/util.go
@@ -14,16 +14,17 @@ func loadLogStreamState(ls *coordinator.LogStream) *logdog.LogStreamState {
lss := logdog.LogStreamState{
ProtoVersion: ls.ProtoVersion,
Created: google.NewTimestamp(ls.Created),
- Updated: google.NewTimestamp(ls.Updated),
TerminalIndex: ls.TerminalIndex,
Purged: ls.Purged,
}
+
if ls.Archived() {
lss.Archive = &logdog.LogStreamState_ArchiveInfo{
- IndexUrl: ls.ArchiveIndexURL,
- StreamUrl: ls.ArchiveStreamURL,
- DataUrl: ls.ArchiveDataURL,
- Whole: ls.ArchiveWhole,
+ IndexUrl: ls.ArchiveIndexURL,
+ StreamUrl: ls.ArchiveStreamURL,
+ DataUrl: ls.ArchiveDataURL,
+ Complete: ls.ArchiveComplete(),
+ LogEntryCount: ls.ArchiveLogEntryCount,
}
}

Powered by Google App Engine
This is Rietveld 408576698