Skip to content

Conversation

@sgalsaleh
Copy link
Member

What this PR does / why we need it:

Adds the customer ID to the license so that vendors can then send it from on-prem instances back to their envs.

Which issue(s) this PR fixes:

SC-131529

Does this PR require a test?

Yes

Does this PR require a release note?

Adds the customer ID to the license and exposes it via the `repl LicenseFieldValue "customerID"` template function.

Does this PR require documentation?

NONe

@sgalsaleh sgalsaleh added the type::feature New feature or request label Nov 25, 2025
Comment on lines 2027 to +2028
endpoint: https://example.com
customerID: customer-id
Copy link
Member

Choose a reason for hiding this comment

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

indentation

Suggested change
endpoint: https://example.com
customerID: customer-id
endpoint: https://example.com
customerID: customer-id

Comment on lines 2089 to +2090
Endpoint: "https://example.com",
CustomerID: "customer-id",
Copy link
Member

Choose a reason for hiding this comment

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

how would this pass with wrong indentation?

Copy link
Member

Choose a reason for hiding this comment

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

or does yaml parse just not care about tabs vs spaces

Copy link
Member Author

Choose a reason for hiding this comment

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

it doesn't :), was trying to figure out why it's failing. thanks

Copy link
Member

Choose a reason for hiding this comment

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

Hah!

@sgalsaleh sgalsaleh changed the title Salah/sc 131529/add customer id to the license Add customer ID to the license Nov 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type::feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants