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

[release/1.1] Add CodeQL3000 to microsoft-reverse-proxy-official #1917

Merged
merged 2 commits into from
Nov 2, 2022

Conversation

dougbu
Copy link
Contributor

@dougbu dougbu commented Nov 2, 2022

  • backport of [main] Add CodeQL3000 to microsoft-reverse-proxy-official #1914
  • add a new weekly schedule to keep database and tools up to database
  • add top-level parameter enabling CodeQL3000 in manual builds
  • add CodeQL3000 variables
  • add CodeQL3000 tasks
    • not auto-injected because that happens only in builds of the default branch
  • add tag indicating CodeQL3000 tasks did useful work
    • ignore no-op task executions
  • add a tsaoptions.json file
    • cribbed values from eng/sdl-tsa-vars.config file

also:

nits:

  • disable auto-injection of build steps we or Arcade do explicitly
  • remove irrelevant variables in test jobs

- backport of microsoft#1914
- add a new weekly schedule to keep database and tools up to database
- add top-level parameter enabling CodeQL3000 in manual builds
- add CodeQL3000 variables
- add CodeQL3000 tasks
  - not auto-injected because that happens only in builds of the default branch
- add tag indicating CodeQL3000 tasks did useful work
  - ignore no-op task executions
- add a tsaoptions.json file
  - cribbed values from eng/sdl-tsa-vars.config file

also:
- backport microsoft#1843
- backport microsoft#1848
- switch to `-Svc` pools since this is a servicing branch

nits:
- disable auto-injection of build steps we or Arcade do explicitly
- remove irrelevant variables in test jobs
@dougbu
Copy link
Contributor Author

dougbu commented Nov 2, 2022

Changes here match #1916 except that public pool name had not yet changed here. See #1848.

Comment on lines +49 to 53
name: NetCore-Svc-Public
demands: ImageOverride -equals Build.Ubuntu.1804.Amd64.Open
${{ if eq(variables['System.TeamProject'], 'internal') }}:
name: NetCore1ESPool-Internal
name: NetCore1ESPool-Svc-Internal
demands: ImageOverride -equals Build.Ubuntu.1804.Amd64
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jonfortescue @alexperovich @MattGal are these the correct pool and image names these days (for a servicing branch)❔

Copy link
Member

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@MattGal MattGal left a comment

Choose a reason for hiding this comment

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

Looks good from a pool provider standpoint.

Comment on lines +49 to 53
name: NetCore-Svc-Public
demands: ImageOverride -equals Build.Ubuntu.1804.Amd64.Open
${{ if eq(variables['System.TeamProject'], 'internal') }}:
name: NetCore1ESPool-Internal
name: NetCore1ESPool-Svc-Internal
demands: ImageOverride -equals Build.Ubuntu.1804.Amd64
Copy link
Member

Choose a reason for hiding this comment

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

LGTM

@Tratcher Tratcher merged commit 5c2af7f into microsoft:release/1.1 Nov 2, 2022
@dougbu dougbu deleted the dougbu/CodeQL3000/1.1 branch November 30, 2022 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants