rel: https://github.com/fleetdm/fleet/issues/8768 thanks to @gillespi314 for the idea, the test consistently fails with:
```
=== RUN TestHosts/TestHostOrder
hosts_test.go:5392:
Error Trace: /home/runner/work/fleet/fleet/server/datastore/mysql/hosts_test.go:5392
/home/runner/work/fleet/fleet/server/datastore/mysql/hosts_test.go:5416
/home/runner/work/fleet/fleet/server/datastore/mysql/hosts_test.go:139
Error: "[]" should have 3 item(s), but has 0
Test: TestHosts/TestHostOrder
```
and the relevant code around line `5416`:
```go
_, err = ds.writer.Exec(`UPDATE hosts SET created_at = created_at + id`)
require.NoError(t, err)
hosts, err = ds.ListHosts(ctx, fleet.TeamFilter{User: test.UserAdmin}, fleet.HostListOptions{
ListOptions: fleet.ListOptions{
OrderKey: "created_at",
After: "2010-10-22T20:22:03Z",
OrderDirection: fleet.OrderAscending,
},
})
require.NoError(t, err)
chk(hosts, "0001", "0004", "0003")
```
This PR changes it to be `UPDATE hosts SET created_at = DATE_ADD(created_at, INTERVAL id DAY)` instead, which seems to fix the issue (so far 3 runs without issues)
This test was failing in Dec 2022, because in this line:
```go
_, err := sut.GetFeed((now.AddDate(0, 1, 0)).Month(), now.Year())
```
`(now.AddDate(0, 1, 0)).Month()` returns `"January"` , and `now.Year()` returns `2022` , so we were sending a date in the past.
* Add negation pseuodo-class to button hover to prevent bg color change when disabled
* change file
Co-authored-by: Jacob Shandling <jacob@fleetdm.com>
* Handbook: Openness value
adding the Q&A session to the "why this way" page
* Update handbook/company/why-this-way.md
Co-authored-by: Chris McGillicuddy <108031970+chris-mcgillicuddy@users.noreply.github.com>
* Update why-this-way.md
I revised the copy Charlie provided for two main reasons.
First, to keep formatting on the page consistent by removing the Q&A framework.
Second, to apply Mike's rationale from the Slack conversation to speak about Openness more broadly. People who find this section won't have the Slack conversation for context. Stepping back a bit will make this more approachable. However, I'm happy to add more detail if necessary.
Co-authored-by: Chris McGillicuddy <108031970+chris-mcgillicuddy@users.noreply.github.com>
* Update "No 1Password recovery kits..." policy
- Update policy in standard query library shown on fleetdm.com
- Update policy in standard templates shown in Fleet UI
* Update policy template shown in UI
* Commit Sharvil's exits to standard query library
* Commit Sharvil's edits to policy templates
* Codeowners
Before we changed to Nate being the code owner for the page, I received notifications for reviewing changes to the ceo-handbook subpage. Now that defaults to Nate.
Not sure if this is the fix for that or not. But wanted to put it on someone's radar.
* Update CODEOWNERS
Co-authored-by: Roberto Dip <me@roperzh.com>
Co-authored-by: Roberto Dip <me@roperzh.com>
We discussed in the @fleetdm/g-platform weekly meeting that it would be good to have a read replica enabled by default in load testing, as it matches what we would recommend to customers for deployments of the size we use to load test with.
This is just what I did to get it working in the past, but I'm happy to adjust as you consider appropriate.
gillespi314