Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[CoE Starter Kit - QUESTION] Can we reduce calls to HELPER - Maker Check #7485

Closed
1 task done
WilliamJYoung opened this issue Jan 18, 2024 · 10 comments
Closed
1 task done
Assignees
Labels
coe-starter-kit CoE Starter Kit issues enhancement New feature or request

Comments

@WilliamJYoung
Copy link

WilliamJYoung commented Jan 18, 2024

Does this question already exist in our backlog?

  • I have checked and confirm this is a new question.

What is your question?

Our tenancy analytics indicate that the "HELPER - Maker Check" flow ran over 50k times in the last 7 days. This appears to be the main cause of an excessive number of request calls in the dedicated environment.
a/ is this normal?
b/ are there options to reduce the number of calls?
Thanks

What solution are you experiencing the issue with?

None

What solution version are you using?

3.41

What app or flow are you having the issue with?

HELPER - Maker Check

What method are you using to get inventory and telemetry?

None

AB#2072

@WilliamJYoung WilliamJYoung added coe-starter-kit CoE Starter Kit issues question Further information is requested labels Jan 18, 2024
@WilliamJYoung
Copy link
Author

Hi Sorry, instead of:
b/ are there options to reduce the number of calls?
...I should perhaps have asked:
b/ Is there any change I can make to reduce the number of flow runs, without notably impacting the Maker records?
Thanks again

@Jenefer-Monroe
Copy link
Collaborator

We dont have a good way to get user information in a truly low API way until the AAD virtual table is available in flow.

But we did, as part of the Jan release ship a change here to reduce calls to this flow for all the objects besides envt, canvas apps, and cloud flows. I know they are the highest hitting but I wanted to get feedback before porting to those.

Please see #7016

Note that it requires you use the v4 versions of the sync flows, see #7146 for how to do that.

@Jenefer-Monroe Jenefer-Monroe changed the title [CoE Starter Kit - QUESTION] QUESTION [CoE Starter Kit - QUESTION] Can we reduce calls to HELPER - Maker Check Jan 18, 2024
@Jenefer-Monroe Jenefer-Monroe self-assigned this Jan 18, 2024
@Jenefer-Monroe
Copy link
Collaborator

If you take a look and get me feedback on this being helpful API wise, then I can use this issue to track the backport of the change to envts / canvas apps / cloud flows.

@Jenefer-Monroe
Copy link
Collaborator

Jenefer-Monroe commented Jan 18, 2024

ok actually just got a pointer. I think I can do this for these 3 for Feb too. Will use your bug to track.

@Jenefer-Monroe Jenefer-Monroe added enhancement New feature or request and removed question Further information is requested labels Jan 18, 2024
@WilliamJYoung
Copy link
Author

Hi Jenefer
Yes, that looks helpful, although it's not clear to me how significantly it would reduce the number of runs/calls.
It looks like the first step is to update our CoE solution.
Thanks for the quick response

@Jenefer-Monroe
Copy link
Collaborator

Yes you can wait till the Feb release when this will be the default behavior.

@WilliamJYoung
Copy link
Author

Hi Jenefer
Just to be sure that I understand, the Feb release should make a significant impact on the flow runs?
I'll do this and report back, as presumably everyone else is/was getting the same
Thanks again

@Jenefer-Monroe
Copy link
Collaborator

yes once we ship Feb release which is targetted for Feb 7th

@CoEStarterKitBot
Copy link
Collaborator

@WilliamJYoung This has been fixed in the latest release. Please install the latest version of the toolkit following the instructions for installing updates. Note that if you do not remove the unmanaged layers as described there you will not receive updates from us.

1 similar comment
@CoEStarterKitBot
Copy link
Collaborator

@WilliamJYoung This has been fixed in the latest release. Please install the latest version of the toolkit following the instructions for installing updates. Note that if you do not remove the unmanaged layers as described there you will not receive updates from us.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
coe-starter-kit CoE Starter Kit issues enhancement New feature or request
Projects
Status: Done
Development

No branches or pull requests

4 participants