Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(orca-clouddriver): Fix exception in mapping of trafficManagement when null #4845

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

christosarvanitis
Copy link
Member

Fixes the mapping exception when trafficManagement is not defined in the Deploy stage definition.

Exception:
Screenshot 2025-02-21 at 14 29 24

Resulted from changes in #4823

@dbyron-sf
Copy link
Contributor

We're now 2 PRs deep with no tests....This needs to change. Any chance you can add one here?

@christosarvanitis
Copy link
Member Author

We're now 2 PRs deep with no tests....This needs to change. Any chance you can add one here?

Yeah sure, will do until Monday. Lets hold 1.37.x for this fix though

@dbyron-sf
Copy link
Contributor

We're now 2 PRs deep with no tests....This needs to change. Any chance you can add one here?

Yeah sure, will do until Monday. Lets hold 1.37.x for this fix though

Agreed. Thank you!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants