Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

ref(routes): ListTrafficTargetPermutation using TrafficTarget fields … #1865

Merged
merged 1 commit into from
Oct 19, 2020

Conversation

ksubrmnn
Copy link
Contributor

…directly

Description:
Refactors ListTrafficTargetPermutation to include all of the logic for getting services from service accounts listed in TrafficTarget. This makes the code a bit more modular and easier to read.

Affected area:

  • New Functionality [ ]
  • Documentation [ ]
  • Install [ ]
  • Control Plane [X]
  • CLI Tool [ ]
  • Certificate Management [ ]
  • Networking [ ]
  • Metrics [ ]
  • SMI Policy [ ]
  • Security [ ]
  • Tests [ ]
  • CI System [ ]
  • Performance [ ]
  • Other [ ]

Please answer the following questions with yes/no.

  • Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution?

No

@ksubrmnn ksubrmnn requested a review from a team as a code owner October 16, 2020 23:49
@ksubrmnn ksubrmnn merged commit 1f25d11 into openservicemesh:main Oct 19, 2020
@ksubrmnn ksubrmnn deleted the traffic_target_permutations branch October 19, 2020 18:19
draychev pushed a commit to draychev/osm that referenced this pull request Oct 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants