-
Notifications
You must be signed in to change notification settings - Fork 280
feat(@xen-orchestra/rest-api): expose get networks and get network #8497
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
base: master
Are you sure you want to change the base?
Conversation
* @example fields "name_label,id" | ||
* @example filter "nbd?" | ||
* @example limit 42 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See this #8490 (comment)
import { Example, Get, Path, Query, Response, Request, Route, Security, Tags } from 'tsoa' | ||
import { inject } from 'inversify' | ||
import { provide } from 'inversify-binding-decorators' | ||
import { Request as ExRequest } from 'express' | ||
import type { XoNetwork } from '@vates/types' | ||
import { RestApi } from '../../rest-api/rest-api.mjs' | ||
import { notFoundResp, unauthorizedResp, Unbrand } from '../common/response.common.mjs' | ||
import { WithHref } from '../../helpers/helper.type.mjs' | ||
import { XapiXoController } from '../../abstract-classes/xapi-xo-controller.mjs' | ||
import { network, networkIds, partialNetworks } from '../oa-examples/network.oa-example.mjs' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Try to keep import sorted
messages_href: '/rest/v0/networks/9fe12ca3-d75d-cfb0-492e-cfd2bc6c568f/messages', | ||
alarms_href: '/rest/v0/networks/9fe12ca3-d75d-cfb0-492e-cfd2bc6c568f/alarms', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
?
id: Branded<'network'> | ||
insecureNbd: boolean |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
insecureNbd: boolean | |
insecureNbd?: boolean |
MTU: number | ||
name_description: string | ||
name_label: string | ||
nbd: boolean |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nbd: boolean | |
nbd?: boolean |
Description
Short explanation of this PR (feel free to re-use commit message)
Checklist
Fixes #007
,See xoa-support#42
,See https://...
)Introduced by
CHANGELOG.unreleased.md
Review process
Notes: