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

1ES Transition Items - "Extras" #7941

Open
40 of 41 tasks
scbedd opened this issue Mar 22, 2024 · 4 comments
Open
40 of 41 tasks

1ES Transition Items - "Extras" #7941

scbedd opened this issue Mar 22, 2024 · 4 comments
Assignees
Labels
Central-EngSys This issue is owned by the Engineering System team.

Comments

@scbedd
Copy link
Member

scbedd commented Mar 22, 2024

Discovered long tail items

  • Transition azure-sdk-for-android
  • Scan azure-sdk-build-tools exes with 1es-templates -> Remaining to be resolved:
    • tools/sdk-generation-pipeline/ci.yml
    • tools/oav-traffic-converter/ci.yml
    • tools/mock-service-host/ci.yml
    • tools/js-sdk-release-tools/ci.yml
    • tools/apiview/parsers/js-api-parser/ci.yml
    • tools/apiview/emitters/typespec-apiview/ci.yml
    • eng/pipelines/templates/stages/archetype-autorest-preview.yml
    • tools/tsp-client/ci.yml
  • Remove download-credscan-suppression.yml in favor of disabling the SDL step on specific jobs or the like
    • We can account for the places where the clone is in a different location than the default SDL config by utilizing a templateContext argument on the deployment. Otherwise I'm not certain we can bypass this.
    • We CAN disable sdl injected steps when there is no checkout. Unfortunately during our signing stage there absolutely is checkout. Second section on this page This is for the entire SDL stage, not the injected steps during artifact publish.
    • No way to disable this. Only possible thing we could change is update to generate a blank credscan suppression file, then simply reference that via templateContext override.
      • templateContext:
          sdl:
            credscan:
              suppressionsFile: path/to/generated/file.json
        
  • Eliminate warning for 1es artifact of that name already exists for matrix-ed test jobs This issue has drug on for 3 weeks with new discovered work. Going to file this as its own thing and pursue it.

yml Builds

Designer Builds

The following builds are designer, and will not benefit from the yml updates above.

  • mgmt-netsdk-sign. Not used since 2019, can't clean up due to release. Disabled.
  • mgmt-NetCore-SDK-Publish
  • mgmt-netsdk-multiapi-publish
  • mgmt-netsdk-sdkcommon-publish
  • mgmt-netsdk-publish
  • Specs Repo - Update readme.md all-api-versions tags.
  • Autorest Npm Admin Task Supplanted by npm admin build
  • tools - Codex - vscode.dev

Anything crossed out is deleted. Checkmark otherwise indicates merged PR.

Release Builds

Need to mark as non-production or otherwise transition each of the above builds. At the very minimum the release builds need to be transitioned to yml.

@scbedd scbedd self-assigned this Mar 22, 2024
@github-actions github-actions bot added the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Mar 22, 2024
@scbedd scbedd moved this from 🤔 Triage to 🔬 Dev in PR in Azure SDK EngSys 🤖🧠 Mar 22, 2024
@scbedd scbedd moved this from 🔬 Dev in PR to 🐝 Dev in Azure SDK EngSys 🤖🧠 Mar 22, 2024
@scbedd scbedd moved this from 🐝 Dev to 📋 Backlog in Azure SDK EngSys 🤖🧠 Mar 22, 2024
@kurtzeborn kurtzeborn added Central-EngSys This issue is owned by the Engineering System team. and removed needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. labels Apr 1, 2024
@kurtzeborn
Copy link
Member

kurtzeborn commented Apr 1, 2024

These designer builds are lo-pri... they existed before and they continue to exist after the transition. We should not worry about these until we get dinged on them (and then we'll probably just delete them).

@alzimmermsft
Copy link
Member

At least in Java we're finding that in some CI runs virus scanning can take 10 to 15 minutes to run. Looking at the step running virus scanning, this could be that we're using '$(System.DefaultWorkingDirectory)' as the output parent directory, which may mean we're scanning all source code, build artifacts, and test recordings downloaded and generated during CI.

We should investigate if moving these build artifacts to a subfolder reduces the number of files that need to be virus scanned.

@scbedd
Copy link
Member Author

scbedd commented May 30, 2024

These designer builds are lo-pri... they existed before and they continue to exist after the transition. We should not worry about these until we get dinged on them (and then we'll probably just delete them).

They were all deleted.

@scbedd
Copy link
Member Author

scbedd commented Jun 27, 2024

This is all done except for one item that I am blocked on. Publishing the ADO extension from openapi-alps-publish. Have a support ticket out with the 1es folks. Holding pattern until response.

@scbedd scbedd moved this from 🔬 Dev in PR to 📋 Backlog in Azure SDK EngSys 🤖🧠 Jul 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Central-EngSys This issue is owned by the Engineering System team.
Projects
Status: 📋 Backlog
Development

No branches or pull requests

3 participants