Skip to content

Commit

Permalink
rename fields to camelCase (#2712)
Browse files Browse the repository at this point in the history
  • Loading branch information
t4lz authored Sep 2, 2024
1 parent 69e801b commit 5788f90
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
1 change: 1 addition & 0 deletions changelog.d/+sqs-crd-case.changed.md
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
SQS CRD field names changed to camelCase.
5 changes: 4 additions & 1 deletion mirrord/operator/src/crd.rs
Original file line number Diff line number Diff line change
Expand Up @@ -381,6 +381,7 @@ pub enum SplitQueue {

/// A workload that is a consumer of a queue that is being split.
#[derive(Clone, Debug, Deserialize, Eq, PartialEq, Serialize, JsonSchema)]
#[serde(rename_all = "camelCase")] // workload_type -> workloadType
pub struct QueueConsumer {
pub name: String,
/// If a container is not specified, the workload queue registry will apply to every run that
Expand Down Expand Up @@ -440,6 +441,7 @@ impl Display for QueueConsumer {
}

#[derive(Clone, Debug, Default, Deserialize, Serialize, JsonSchema)]
#[serde(rename_all = "camelCase")] // original_name -> originalName
pub struct QueueNameUpdate {
pub original_name: String,
pub output_name: String,
Expand All @@ -452,6 +454,7 @@ pub struct QueueNameUpdate {
// controller's code a bit simpler.
// Some information is present in the spec, but it is organized differently.
#[derive(Clone, Debug, Default, Deserialize, Serialize, JsonSchema)]
#[serde(rename_all = "camelCase")] // workload_type -> workloadType
pub struct ActiveSqsSplits {
/// For each queue_id, the actual queue name as retrieved from the target's pod spec or config
/// map, together with the name of its temporary output queue.
Expand All @@ -474,7 +477,7 @@ impl ActiveSqsSplits {
}

#[derive(Clone, Debug, Default, Deserialize, Serialize, JsonSchema)]
#[serde(rename_all = "camelCase")] // active_filters -> activeFilters
#[serde(rename_all = "camelCase")] // sqs_details -> sqsDetails
pub struct WorkloadQueueRegistryStatus {
/// Optional even though it's currently the only field, because in the future there will be
/// fields for other queue types.
Expand Down

0 comments on commit 5788f90

Please sign in to comment.