Skip to content

Commit

Permalink
remove unused name_or_id query param on bgp announce set list
Browse files Browse the repository at this point in the history
  • Loading branch information
david-crespo committed Aug 31, 2024
1 parent 61e1b38 commit 5fab4be
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 21 deletions.
4 changes: 1 addition & 3 deletions nexus/external-api/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1565,9 +1565,7 @@ pub trait NexusExternalApi {
}]
async fn networking_bgp_announce_set_list(
rqctx: RequestContext<Self::Context>,
query_params: Query<
PaginatedByNameOrId<params::OptionalBgpAnnounceSetSelector>,
>,
query_params: Query<PaginatedByNameOrId>,
) -> Result<HttpResponseOk<Vec<BgpAnnounceSet>>, HttpError>;

/// Delete BGP announce set
Expand Down
4 changes: 1 addition & 3 deletions nexus/src/external_api/http_entrypoints.rs
Original file line number Diff line number Diff line change
Expand Up @@ -3139,9 +3139,7 @@ impl NexusExternalApi for NexusExternalApiImpl {

async fn networking_bgp_announce_set_list(
rqctx: RequestContext<ApiContext>,
query_params: Query<
PaginatedByNameOrId<params::OptionalBgpAnnounceSetSelector>,
>,
query_params: Query<PaginatedByNameOrId>,
) -> Result<HttpResponseOk<Vec<BgpAnnounceSet>>, HttpError> {
let apictx = rqctx.context();
let handler = async {
Expand Down
7 changes: 0 additions & 7 deletions nexus/types/src/external_api/params.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1675,13 +1675,6 @@ pub struct BgpAnnounceSetCreate {
pub announcement: Vec<BgpAnnouncementCreate>,
}

/// Optionally select a BGP announce set by a name or id.
#[derive(Clone, Debug, Deserialize, Serialize, JsonSchema, PartialEq)]
pub struct OptionalBgpAnnounceSetSelector {
/// A name or id to use when s electing BGP port settings
pub name_or_id: Option<NameOrId>,
}

/// Select a BGP announce set by a name or id.
#[derive(Clone, Debug, Deserialize, Serialize, JsonSchema, PartialEq)]
pub struct BgpAnnounceSetSelector {
Expand Down
8 changes: 0 additions & 8 deletions openapi/nexus.json
Original file line number Diff line number Diff line change
Expand Up @@ -6554,14 +6554,6 @@
"minimum": 1
}
},
{
"in": "query",
"name": "name_or_id",
"description": "A name or id to use when s electing BGP port settings",
"schema": {
"$ref": "#/components/schemas/NameOrId"
}
},
{
"in": "query",
"name": "page_token",
Expand Down

0 comments on commit 5fab4be

Please sign in to comment.