Skip to content

Commit 896f674

Browse files
committed
feat(mgmt): rename VpcExposeManifest->VpcManifest
Signed-off-by: Fredi Raspall <[email protected]>
1 parent 83e3066 commit 896f674

File tree

3 files changed

+21
-21
lines changed

3 files changed

+21
-21
lines changed

mgmt/src/rpc/overlay/mod.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ pub mod vpc;
88
pub mod vpcpeering;
99

1010
use crate::rpc::overlay::vpc::VpcTable;
11-
use crate::rpc::overlay::vpcpeering::VpcExposeManifest;
11+
use crate::rpc::overlay::vpcpeering::VpcManifest;
1212
use crate::rpc::overlay::vpcpeering::VpcPeeringTable;
1313
use tracing::error;
1414

@@ -27,7 +27,7 @@ impl Overlay {
2727
peering_table,
2828
}
2929
}
30-
fn check_peering_vpc(&self, peering: &str, manifest: &Option<VpcExposeManifest>) -> ApiResult {
30+
fn check_peering_vpc(&self, peering: &str, manifest: &Option<VpcManifest>) -> ApiResult {
3131
if let Some(vpc) = manifest {
3232
if self.vpc_table.get_vpc(&vpc.name).is_none() {
3333
error!("peering '{}': unknown VPC '{}'", peering, vpc.name);

mgmt/src/rpc/overlay/tests.rs

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -8,16 +8,16 @@
88
pub mod test {
99
use crate::rpc::overlay::vpc::{Vpc, VpcTable};
1010
use crate::rpc::overlay::vpcpeering::VpcExpose;
11-
use crate::rpc::overlay::vpcpeering::VpcExposeManifest;
11+
use crate::rpc::overlay::vpcpeering::VpcManifest;
1212
use crate::rpc::overlay::vpcpeering::{VpcPeering, VpcPeeringTable};
1313
use crate::rpc::overlay::Overlay;
1414
use crate::rpc::ApiError;
1515

1616
use routing::prefix::Prefix;
1717

1818
/* Build sample manifests for a peering */
19-
fn build_manifest_vpc1() -> VpcExposeManifest {
20-
let mut vpc1 = VpcExposeManifest::new("VPC-1");
19+
fn build_manifest_vpc1() -> VpcManifest {
20+
let mut vpc1 = VpcManifest::new("VPC-1");
2121
let expose = VpcExpose::empty()
2222
.ip(Prefix::from(("10.0.0.0", 25)))
2323
.ip(Prefix::from(("10.0.2.128", 25)))
@@ -28,8 +28,8 @@ pub mod test {
2828
vpc1.add_expose(expose).expect("Should succeed");
2929
vpc1
3030
}
31-
fn build_manifest_vpc2() -> VpcExposeManifest {
32-
let mut vpc1 = VpcExposeManifest::new("VPC-2");
31+
fn build_manifest_vpc2() -> VpcManifest {
32+
let mut vpc1 = VpcManifest::new("VPC-2");
3333
let expose = VpcExpose::empty()
3434
.ip(Prefix::from(("10.0.0.0", 24)))
3535
.as_range(Prefix::from(("100.64.2.0", 24)));
@@ -146,29 +146,29 @@ pub mod test {
146146
fn test_peering_iter() {
147147
let mut peering_table = VpcPeeringTable::new();
148148

149-
let m1 = VpcExposeManifest::new("VPC-1");
150-
let m2 = VpcExposeManifest::new("VPC-2");
149+
let m1 = VpcManifest::new("VPC-1");
150+
let m2 = VpcManifest::new("VPC-2");
151151
let mut peering = VpcPeering::new("Peering-1");
152152
peering.set_one(m1);
153153
peering.set_two(m2);
154154
peering_table.add(peering).unwrap();
155155

156-
let m1 = VpcExposeManifest::new("VPC-1");
157-
let m2 = VpcExposeManifest::new("VPC-3");
156+
let m1 = VpcManifest::new("VPC-1");
157+
let m2 = VpcManifest::new("VPC-3");
158158
let mut peering = VpcPeering::new("Peering-2");
159159
peering.set_one(m1);
160160
peering.set_two(m2);
161161
peering_table.add(peering).unwrap();
162162

163-
let m1 = VpcExposeManifest::new("VPC-2");
164-
let m2 = VpcExposeManifest::new("VPC-4");
163+
let m1 = VpcManifest::new("VPC-2");
164+
let m2 = VpcManifest::new("VPC-4");
165165
let mut peering = VpcPeering::new("Peering-3");
166166
peering.set_one(m1);
167167
peering.set_two(m2);
168168
peering_table.add(peering).unwrap();
169169

170-
let m1 = VpcExposeManifest::new("VPC-1");
171-
let m2 = VpcExposeManifest::new("VPC-4");
170+
let m1 = VpcManifest::new("VPC-1");
171+
let m2 = VpcManifest::new("VPC-4");
172172
let mut peering = VpcPeering::new("Peering-4");
173173
peering.set_one(m1);
174174
peering.set_two(m2);

mgmt/src/rpc/overlay/vpcpeering.rs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -43,11 +43,11 @@ impl VpcExpose {
4343
}
4444

4545
#[derive(Debug, Default)]
46-
pub struct VpcExposeManifest {
46+
pub struct VpcManifest {
4747
pub name: String, /* key: name of vpc */
4848
pub exposes: Vec<VpcExpose>,
4949
}
50-
impl VpcExposeManifest {
50+
impl VpcManifest {
5151
pub fn new(vpc_name: &str) -> Self {
5252
Self {
5353
name: vpc_name.to_owned(),
@@ -64,8 +64,8 @@ impl VpcExposeManifest {
6464
#[derive(Debug)]
6565
pub struct VpcPeering {
6666
pub name: String, /* key: name of peering */
67-
pub vpc1: Option<VpcExposeManifest>,
68-
pub vpc2: Option<VpcExposeManifest>,
67+
pub vpc1: Option<VpcManifest>,
68+
pub vpc2: Option<VpcManifest>,
6969
}
7070
impl VpcPeering {
7171
pub fn new(name: &str) -> Self {
@@ -85,10 +85,10 @@ impl VpcPeering {
8585
Ok(())
8686
}
8787
}
88-
pub fn set_one(&mut self, exp_manifest: VpcExposeManifest) {
88+
pub fn set_one(&mut self, exp_manifest: VpcManifest) {
8989
self.vpc1 = Some(exp_manifest);
9090
}
91-
pub fn set_two(&mut self, exp_manifest: VpcExposeManifest) {
91+
pub fn set_two(&mut self, exp_manifest: VpcManifest) {
9292
self.vpc2 = Some(exp_manifest);
9393
}
9494

0 commit comments

Comments
 (0)