Container resources readonly incosistency

[oleghind]

Description of the bug:

I can adjust cpu resources, but not memory

export class MyChart extends MyBaseChart  {
  constructor(scope: Construct, id: string, props: ChartProps = {}) {
    super(scope, id, props)
    this.someContainer.resources.cpu.request.amount = '3000m'; // Works
    this.someContainer.resources.memory.request.limit = '4096Mi' // Doesn't work, value readonly
});

Please make both adjustable, so child charts can adjust the value without patching yaml

---

This is 🐛 Bug Report

[iliapolo]

@oleghind Apologies for the inconsistency, but the immutable nature of these properties is intentional. You'll note that none of the other properties in the library are mutable. The fact that amount is mutable is actually a mistake/bug:

https://github.com/cdk8s-team/cdk8s-plus/blob/a3afd28ecd403a048f984143e6c2cc7a225120b6/src/container.ts#L1344

[github-actions]

This issue has not received a response in a while and will be closed soon. If you want to keep it open, please leave a comment below @mentioning a maintainer.