Skip to content

Commit d906190

Browse files
authored
minor: update affinity docs link, cut a few lines (#2795)
1 parent 8e509d9 commit d906190

File tree

4 files changed

+12
-26
lines changed

4 files changed

+12
-26
lines changed

app/pages/project/instances/AntiAffinityCard.tsx

Lines changed: 5 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ import { Affinity24Icon } from '@oxide/design-system/icons/react'
2525
import { AffinityPolicyHeader } from '~/components/AffinityDocsPopover'
2626
import { ComboboxField } from '~/components/form/fields/ComboboxField'
2727
import { HL } from '~/components/HL'
28+
import { antiAffinityGroupList } from '~/forms/affinity-util'
2829
import { useInstanceSelector } from '~/hooks/use-params'
2930
import { AffinityGroupPolicyBadge } from '~/pages/project/affinity/AffinityPage'
3031
import { confirmAction } from '~/stores/confirm-action'
@@ -49,17 +50,6 @@ export const instanceAntiAffinityGroups = ({ project, instance }: PP.Instance) =
4950
query: { project, limit: ALL_ISH },
5051
})
5152

52-
export const allAntiAffinityGroups = ({ project }: PP.Project) =>
53-
apiq('antiAffinityGroupList', {
54-
query: { project, limit: ALL_ISH },
55-
})
56-
57-
const instanceView = ({ project, instance }: PP.Instance) =>
58-
apiq('instanceView', {
59-
path: { instance },
60-
query: { project },
61-
})
62-
6353
const colHelper = createColumnHelper<AffinityGroup | AntiAffinityGroup>()
6454
const staticCols = [
6555
colHelper.accessor('description', Columns.description),
@@ -76,8 +66,10 @@ export function AntiAffinityCard() {
7666
const { data: memberGroups } = usePrefetchedQuery(
7767
instanceAntiAffinityGroups(instanceSelector)
7868
)
79-
const { data: allGroups } = usePrefetchedQuery(allAntiAffinityGroups(instanceSelector))
80-
const { data: instanceData } = usePrefetchedQuery(instanceView(instanceSelector))
69+
const { data: allGroups } = usePrefetchedQuery(antiAffinityGroupList(instanceSelector))
70+
const { data: instanceData } = usePrefetchedQuery(
71+
apiq('instanceView', { path: { instance }, query: { project } })
72+
)
8173

8274
const nonMemberGroups = useMemo(
8375
() => R.differenceWith(allGroups.items, memberGroups.items, (a, b) => a.id === b.id),

app/pages/project/instances/InstancePage.tsx

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -297,11 +297,9 @@ export function ResizeInstanceModal({
297297
const { project } = useProjectSelector()
298298
const instanceUpdate = useApiMutation('instanceUpdate', {
299299
onSuccess(_updatedInstance) {
300-
if (onListView) {
301-
apiQueryClient.invalidateQueries('instanceList')
302-
} else {
303-
apiQueryClient.invalidateQueries('instanceView')
304-
}
300+
apiQueryClient.invalidateQueries('instanceList')
301+
apiQueryClient.invalidateQueries('instanceView')
302+
305303
onDismiss()
306304
addToast({
307305
content: (

app/pages/project/instances/SettingsTab.tsx

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -10,13 +10,10 @@ import { type LoaderFunctionArgs } from 'react-router'
1010

1111
import { queryClient } from '@oxide/api'
1212

13+
import { antiAffinityGroupList } from '~/forms/affinity-util'
1314
import { getInstanceSelector } from '~/hooks/use-params'
1415

15-
import {
16-
allAntiAffinityGroups,
17-
AntiAffinityCard,
18-
instanceAntiAffinityGroups,
19-
} from './AntiAffinityCard'
16+
import { AntiAffinityCard, instanceAntiAffinityGroups } from './AntiAffinityCard'
2017
import { AutoRestartCard } from './AutoRestartCard'
2118

2219
export const handle = { crumb: 'Settings' }
@@ -25,7 +22,7 @@ export async function clientLoader({ params }: LoaderFunctionArgs) {
2522
const { project, instance } = getInstanceSelector(params)
2623
await Promise.all([
2724
queryClient.prefetchQuery(instanceAntiAffinityGroups({ project, instance })),
28-
queryClient.prefetchQuery(allAntiAffinityGroups({ project })),
25+
queryClient.prefetchQuery(antiAffinityGroupList({ project })),
2926
])
3027
return null
3128
}

app/util/links.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,8 @@ const remoteAccess = 'https://docs.oxide.computer/guides/remote-access'
1010

1111
export const links = {
1212
accessDocs: 'https://docs.oxide.computer/guides/configuring-access',
13-
// TODO: make sure this is right before merging
1413
affinityDocs:
15-
'https://docs.oxide.computer/guides/deploying-workloads#_anti_affinity_groups',
14+
'https://docs.oxide.computer/guides/deploying-workloads#_affinity_and_anti_affinity',
1615
cloudInitFormat: 'https://cloudinit.readthedocs.io/en/latest/explanation/format.html',
1716
cloudInitExamples: 'https://cloudinit.readthedocs.io/en/latest/reference/examples.html',
1817
disksDocs: 'https://docs.oxide.computer/guides/managing-disks-and-snapshots',

0 commit comments

Comments
 (0)