Skip to content

DLPX-95748 cloud-init oracle data source needs to handle a null metadata value#114

Merged
prakashsurya merged 1 commit intodevelopfrom
dlpx/pr/prakashsurya/0165dfbc-da83-47fe-b130-28764c5cd4af
Oct 29, 2025
Merged

DLPX-95748 cloud-init oracle data source needs to handle a null metadata value#114
prakashsurya merged 1 commit intodevelopfrom
dlpx/pr/prakashsurya/0165dfbc-da83-47fe-b130-28764c5cd4af

Conversation

@prakashsurya
Copy link
Contributor

Problem

We've seen the "metadata" value exist but have a "null" value, which the cloud-init code doesn't handle.

Solution

Fix the logic to properly handle a "null" value (i.e. treat it the same as if the field doesn't exist).

@prakashsurya prakashsurya force-pushed the dlpx/pr/prakashsurya/0165dfbc-da83-47fe-b130-28764c5cd4af branch from f119360 to 26c4f04 Compare October 29, 2025 17:34
@prakashsurya prakashsurya marked this pull request as ready for review October 29, 2025 17:34
Copy link
Contributor

@sebroy sebroy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we upstream this?

@prakashsurya
Copy link
Contributor Author

Can we upstream this?

presumably we can.. off hand, idk what upstream we want to contribute too, or how ubuntu ties into that (since we pull from their source package), but I'll poke around and see what makes sense..

@prakashsurya
Copy link
Contributor Author

I opened upstream: canonical/cloud-init#6549

@prakashsurya prakashsurya merged commit da2976d into develop Oct 29, 2025
16 of 21 checks passed
@prakashsurya prakashsurya deleted the dlpx/pr/prakashsurya/0165dfbc-da83-47fe-b130-28764c5cd4af branch October 29, 2025 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants