Skip to content

Identity perf pipeline run triggers#5892

Merged
gearama merged 5 commits intoAzure:mainfrom
gearama:Identitypipeline
Aug 12, 2024
Merged

Identity perf pipeline run triggers#5892
gearama merged 5 commits intoAzure:mainfrom
gearama:Identitypipeline

Conversation

@gearama
Copy link
Copy Markdown
Contributor

@gearama gearama commented Aug 12, 2024

closes #5423

Pull Request Checklist

Please leverage this checklist as a reminder to address commonly occurring feedback when submitting a pull request to make sure your PR can be reviewed quickly:

See the detailed list in the contributing guide.

  • C++ Guidelines
  • Doxygen docs
  • Unit tests
  • No unwanted commits/changes
  • Descriptive title/description
    • PR is single purpose
    • Related issue listed
  • Comments in source
  • No typos
  • Update changelog
  • Not work-in-progress
  • External references or docs updated
  • Self review of PR done
  • Any breaking changes?

@gearama gearama marked this pull request as ready for review August 12, 2024 21:34
@gearama gearama requested review from billwert, chlowell, g2vinay, joshfree and maorleger and removed request for billwert, chlowell, g2vinay, joshfree and maorleger August 12, 2024 21:34
Copy link
Copy Markdown
Member

@LarryOsterman LarryOsterman left a comment

Choose a reason for hiding this comment

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

Could you please improve the title and/or comment so that it appears better when this is committed

Copy link
Copy Markdown
Contributor

@ahsonkhan ahsonkhan left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this.

Do we need a similar change in other perf.yml files in the repo, such as the one in core?

I am not sure if the following is setup to run the perf pipelines for core, whenever Azure.Core files are changed:

@ahsonkhan
Copy link
Copy Markdown
Contributor

I am not sure if the following is setup to run the perf pipelines for core, whenever Azure.Core files are changed

For example, see this, where only cpp - core - ci run.
#5908

When do we expect cpp - core - perf to get triggered and run?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

The azure-identity-perf pipeline should not run when a code change is made to Azure::Core.

3 participants