Skip to content

Commit e14b29b

Browse files
committed
Set fields to None instead of Some(empty map) (#398)
1 parent ac3f79b commit e14b29b

File tree

2 files changed

+16
-2
lines changed

2 files changed

+16
-2
lines changed

CHANGELOG.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,12 @@ All notable changes to this project will be documented in this file.
44

55
## [Unreleased]
66

7+
### Changed
8+
9+
- `impl Into<Resourcerequirements> for Resources` set's fields to `None` instead of `Some(<empty map>)` when nothing is defined.([#398]).
10+
11+
[#398]: https://github.com/stackabletech/operator-rs/pull/398
12+
713
## [0.20.0] - 2022-05-13
814

915
### Added

src/commons/resources.rs

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -196,8 +196,16 @@ where
196196
}
197197

198198
ResourceRequirements {
199-
limits: Some(limits),
200-
requests: Some(requests),
199+
limits: if limits.is_empty() {
200+
None
201+
} else {
202+
Some(limits)
203+
},
204+
requests: if requests.is_empty() {
205+
None
206+
} else {
207+
Some(requests)
208+
},
201209
}
202210
}
203211
}

0 commit comments

Comments
 (0)