Skip to content

feat: Add support for update checks and notifications #4810

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.

Sign up for GitHub

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

Merged
merged 14 commits into from
Dec 1, 2022
Merged

feat: Add support for update checks and notifications #4810

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
cc28c1b
feat: Add support for checking for updates
mafredri Oct 31, 2022
c163af8
wip(site): Add support for informing of updates
mafredri Oct 31, 2022
e9ed0b7
Add /api/v2/updatecheck to noauthorize list
mafredri Oct 31, 2022
64ad0f5
Fix edge case where update check has never succeeded
mafredri Oct 31, 2022
0238d5a
Fix logging
mafredri Oct 31, 2022
9e1ea9e
Use AlertBanner for coder version notice
mafredri Nov 29, 2022
979ff80
Update golden files
mafredri Nov 29, 2022
b80de3c
Fix margins, improve alert banner and add stories
mafredri Nov 30, 2022
81d11dd
Add authorization, dismissal and fetch after login
mafredri Nov 30, 2022
392d06e
Apply suggestions from code review
mafredri Dec 1, 2022
ad38c3b
Add forgotten UpdateCheckBanner.stories.tsx
mafredri Nov 30, 2022
1c9dbc1
fix(api): Amend PR comments
mafredri Dec 1, 2022
49a7bfc
fix(site): Always show prefix for update check errors, remove as const
mafredri Dec 1, 2022
44f6475
fix(site): Add jest tests for UpdateCheckBanner
mafredri Dec 1, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix logging
  • Loading branch information
@mafredri
mafredri committed Nov 29, 2022
commit 0238d5a4a01c7b7f14c3f19bf53456e9a83d6006
38 changes: 17 additions & 21 deletions coderd/updatecheck/updatecheck.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -136,33 +136,29 @@ func (c *Checker) start() {
t := time.NewTicker(c.opts.Interval)
defer t.Stop()

doCheck := func() {
rr, err := c.update()
if err != nil {
if xerrors.Is(err, context.Canceled) {
return
}
c.log.Error(c.ctx, "update check failed", slog.Error(err))
} else {
c.notifyIfNewer(r, rr)
r = rr
}
diff := time.Until(r.Checked.Add(c.opts.Interval))
if diff > 0 {
c.log.Info(c.ctx, "time until next update check", slog.F("duration", diff))
t.Reset(diff)
} else {
c.log.Info(c.ctx, "time until next update check", slog.F("duration", c.opts.Interval))
t.Reset(c.opts.Interval)
}

if !r.Checked.IsZero() {
diff := time.Until(r.Checked.Add(c.opts.Interval))
if diff > 0 {
c.log.Info(c.ctx, "time until next update check", slog.F("duration", diff))
t.Reset(diff)
}
}

for {
select {
case <-t.C:
doCheck()
rr, err := c.update()
if err != nil {
if xerrors.Is(err, context.Canceled) {
return
}
c.log.Error(c.ctx, "update check failed", slog.Error(err))
} else {
c.notifyIfNewer(r, rr)
r = rr
}
c.log.Info(c.ctx, "time until next update check", slog.F("duration", c.opts.Interval))
t.Reset(c.opts.Interval)
case <-c.ctx.Done():
return
}
Expand Down