Skip to content

Revert "Improve components infrastructure (#12145)" #12679

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

Merged
merged 2 commits into from
Jul 29, 2019
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
122 changes: 2 additions & 120 deletions .azure/pipelines/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,31 +29,7 @@ variables:
- ${{ if or(eq(variables['System.TeamProject'], 'public'), in(variables['Build.Reason'], 'PullRequest')) }}:
- name: _BuildArgs
value: ''

jobs:
# Build Web.JS
- template: jobs/default-build.yml
parameters:
codeSign: true
jobName: WebJS_build
jobDisplayName: "Build: Web.JS"
agentOs: Linux
steps:
- script: ./build.sh
--ci
--projects $(Build.SourcesDirectory)/src/Components/Web.JS/Microsoft.AspNetCore.Components.Web.JS.npmproj
-bl:artifacts/log/build.linux-x64.binlog
$(_BuildArgs)
displayName: Run build.sh
- publish: src/Components/Web.JS/dist/
artifact: WebJS_Javascript
installJdk: false
artifacts:
- name: WebJS_Logs
path: artifacts/log/
publishOnError: true

# Code check
- template: jobs/default-build.yml
parameters:
jobName: Code_check
Expand All @@ -62,16 +38,10 @@ jobs:
steps:
- powershell: ./eng/scripts/CodeCheck.ps1 -ci
displayName: Run eng/scripts/CodeCheck.ps1
artifacts:
- name: Code_Check_Logs
path: artifacts/log/
publishOnError: true

# Build Windows (x64/x86)
- template: jobs/default-build.yml
parameters:
dependsOn:
- WebJS_build
codeSign: true
jobName: Windows_build
jobDisplayName: "Build: Windows x64/x86"
Expand All @@ -83,10 +53,6 @@ jobs:
- script: "echo ##vso[build.addbuildtag]release-candidate"
condition: and(ne(variables['Build.Reason'], 'PullRequest'), in(variables['DotNetFinalVersionKind'], 'release', 'prerelease'))
displayName: 'Set CI tags'
- task: DownloadPipelineArtifact@2
inputs:
artifact: WebJS_Javascript
path: $(Build.SourcesDirectory)/src/Components/Web.JS/dist/
# !!! NOTE !!! Some of these steps have disabled code signing.
# This is intentional to workaround https://github.com/dotnet/arcade/issues/1957 which always re-submits for code-signing, even
# if they have already been signed. This results in slower builds due to re-submitting the same .nupkg many times for signing.
Expand Down Expand Up @@ -164,17 +130,10 @@ jobs:
# Build Windows ARM
- template: jobs/default-build.yml
parameters:
dependsOn:
- WebJS_build
codeSign: true
jobName: Windows_arm_build
jobDisplayName: "Build: Windows ARM"
agentOs: Windows
beforeBuild:
- task: DownloadPipelineArtifact@2
inputs:
artifact: WebJS_Javascript
path: $(Build.SourcesDirectory)/src/Components/Web.JS/dist/
buildArgs:
-arch arm
-sign
Expand All @@ -200,16 +159,9 @@ jobs:
# Build MacOS
- template: jobs/default-build.yml
parameters:
dependsOn:
- WebJS_build
jobName: MacOs_x64_build
jobDisplayName: "Build: macOS"
agentOs: macOs
beforeBuild:
- task: DownloadPipelineArtifact@2
inputs:
artifact: WebJS_Javascript
path: $(Build.SourcesDirectory)/src/Components/Web.JS/dist/
buildArgs:
--pack
--all
Expand All @@ -219,7 +171,6 @@ jobs:
-bl:artifacts/log/build.macos.binlog
$(_BuildArgs)
installNodeJs: false
installJdk: false
artifacts:
- name: MacOS_x64_Packages
path: artifacts/packages/
Expand All @@ -237,16 +188,11 @@ jobs:
# Build Linux x64
- template: jobs/default-build.yml
parameters:
dependsOn:
- WebJS_build
jobName: Linux_x64_build
jobDisplayName: "Build: Linux x64"
agentOs: Linux
installNodeJs: false
steps:
- task: DownloadPipelineArtifact@2
inputs:
artifact: WebJS_Javascript
path: $(Build.SourcesDirectory)/src/Components/Web.JS/dist/
- script: ./build.sh
--ci
--arch x64
Expand All @@ -265,7 +211,6 @@ jobs:
--arch x64 \
--build-installers \
--no-build-deps \
--no-build-nodejs \
-p:OnlyPackPlatformSpecificPackages=true \
-p:BuildRuntimeArchive=false \
-p:LinuxInstallerType=deb \
Expand All @@ -279,15 +224,12 @@ jobs:
--arch x64 \
--build-installers \
--no-build-deps \
--no-build-nodejs \
-p:OnlyPackPlatformSpecificPackages=true \
-p:BuildRuntimeArchive=false \
-p:LinuxInstallerType=rpm \
-bl:artifacts/log/build.rpm.binlog \
$(_BuildArgs)
displayName: Build RPM installers
installNodeJs: false
installJdk: false
artifacts:
- name: Linux_x64_Packages
path: artifacts/packages/
Expand All @@ -305,16 +247,9 @@ jobs:
# Build Linux ARM
- template: jobs/default-build.yml
parameters:
dependsOn:
- WebJS_build
jobName: Linux_arm_build
jobDisplayName: "Build: Linux ARM"
agentOs: Linux
beforeBuild:
- task: DownloadPipelineArtifact@2
inputs:
artifact: WebJS_Javascript
path: $(Build.SourcesDirectory)/src/Components/Web.JS/dist/
buildArgs:
--arch arm
--pack
Expand All @@ -325,7 +260,6 @@ jobs:
-bl:artifacts/log/build.linux-arm.binlog
$(_BuildArgs)
installNodeJs: false
installJdk: false
artifacts:
- name: Linux_arm_Packages
path: artifacts/packages/
Expand All @@ -343,16 +277,9 @@ jobs:
# Build Linux ARM64
- template: jobs/default-build.yml
parameters:
dependsOn:
- WebJS_build
jobName: Linux_arm64_build
jobDisplayName: "Build: Linux ARM64"
agentOs: Linux
beforeBuild:
- task: DownloadPipelineArtifact@2
inputs:
artifact: WebJS_Javascript
path: $(Build.SourcesDirectory)/src/Components/Web.JS/dist/
buildArgs:
--arch arm64
--all
Expand All @@ -363,7 +290,6 @@ jobs:
-bl:artifacts/log/build.arm64.binlog
$(_BuildArgs)
installNodeJs: false
installJdk: false
artifacts:
- name: Linux_arm64_Packages
path: artifacts/packages/
Expand All @@ -381,16 +307,9 @@ jobs:
# Build Linux Musl x64
- template: jobs/default-build.yml
parameters:
dependsOn:
- WebJS_build
jobName: Linux_musl_x64_build
jobDisplayName: "Build: Linux Musl x64"
agentOs: Linux
beforeBuild:
- task: DownloadPipelineArtifact@2
inputs:
artifact: WebJS_Javascript
path: $(Build.SourcesDirectory)/src/Components/Web.JS/dist/
buildScript: ./dockerbuild.sh alpine
buildArgs:
--ci
Expand All @@ -404,7 +323,6 @@ jobs:
-bl:artifacts/log/build.musl.binlog
$(_BuildArgs)
installNodeJs: false
installJdk: false
artifacts:
- name: Linux_musl_x64_Packages
path: artifacts/packages/
Expand All @@ -419,19 +337,12 @@ jobs:
parameters:
inputName: Linux_musl_x64

# Build Linux Musl ARM64
# Build Linux Musl arm64
- template: jobs/default-build.yml
parameters:
dependsOn:
- WebJS_build
jobName: Linux_musl_arm64_build
jobDisplayName: "Build: Linux Musl ARM64"
agentOs: Linux
beforeBuild:
- task: DownloadPipelineArtifact@2
inputs:
artifact: WebJS_Javascript
path: $(Build.SourcesDirectory)/src/Components/Web.JS/dist/
buildScript: ./dockerbuild.sh ubuntu-alpine37
buildArgs:
--ci
Expand All @@ -445,7 +356,6 @@ jobs:
-bl:artifacts/log/build.musl.binlog
$(_BuildArgs)
installNodeJs: false
installJdk: false
artifacts:
- name: Linux_musl_arm64_Packages
path: artifacts/packages/
Expand All @@ -463,8 +373,6 @@ jobs:
# Test jobs
- template: jobs/default-build.yml
parameters:
dependsOn:
- WebJS_build
condition: ne(variables['SkipTests'], 'true')
jobName: Windows_Test
jobDisplayName: "Test: Windows Server 2016 x64"
Expand All @@ -474,10 +382,6 @@ jobs:
beforeBuild:
- powershell: "& ./src/Servers/IIS/tools/UpdateIISExpressCertificate.ps1; & ./src/Servers/IIS/tools/update_schema.ps1"
displayName: Setup IISExpress test certificates and schema
- task: DownloadPipelineArtifact@2
inputs:
artifact: WebJS_Javascript
path: $(Build.SourcesDirectory)/src/Components/Web.JS/dist/
afterBuild:
- powershell: "& ./build.ps1 -CI -NoBuild -Test /p:RunFlakyTests=true"
displayName: Run Flaky Tests
Expand All @@ -492,18 +396,12 @@ jobs:

- template: jobs/default-build.yml
parameters:
dependsOn:
- WebJS_build
condition: ne(variables['SkipTests'], 'true')
jobName: Windows_Templates_Test
jobDisplayName: "Test: Templates - Windows Server 2016 x64"
agentOs: Windows
isTestingJob: true
steps:
- task: DownloadPipelineArtifact@2
inputs:
artifact: WebJS_Javascript
path: $(Build.SourcesDirectory)/src/Components/Web.JS/dist/
- script: ./build.cmd -ci -all -pack
displayName: Build Repo
- script: ./src/ProjectTemplates/build.cmd -ci -pack -NoRestore -NoBuilddeps "/p:RunTemplateTests=true /bl:artifacts/log/template.pack.binlog"
Expand All @@ -520,8 +418,6 @@ jobs:

- template: jobs/default-build.yml
parameters:
dependsOn:
- WebJS_build
condition: ne(variables['SkipTests'], 'true')
jobName: MacOs_Test
jobDisplayName: "Test: macOS 10.13"
Expand All @@ -531,10 +427,6 @@ jobs:
beforeBuild:
- bash: "./eng/scripts/install-nginx-mac.sh"
displayName: Installing Nginx
- task: DownloadPipelineArtifact@2
inputs:
artifact: WebJS_Javascript
path: $(Build.SourcesDirectory)/src/Components/Web.JS/dist/
afterBuild:
- bash: ./build.sh --ci --pack --no-build --no-restore --no-build-deps "/bl:artifacts/log/packages.pack.binlog"
displayName: Pack Packages (for Template tests)
Expand All @@ -553,8 +445,6 @@ jobs:

- template: jobs/default-build.yml
parameters:
dependsOn:
- WebJS_build
condition: ne(variables['SkipTests'], 'true')
jobName: Linux_Test
jobDisplayName: "Test: Ubuntu 16.04 x64"
Expand All @@ -566,10 +456,6 @@ jobs:
displayName: Installing Nginx
- bash: "echo fs.inotify.max_user_watches=524288 | sudo tee -a /etc/sysctl.conf && sudo sysctl -p"
displayName: Increase inotify limit
- task: DownloadPipelineArtifact@2
inputs:
artifact: WebJS_Javascript
path: $(Build.SourcesDirectory)/src/Components/Web.JS/dist/
afterBuild:
- bash: ./build.sh --ci --pack --no-build --no-restore --no-build-deps "/bl:artifacts/log/packages.pack.binlog"
displayName: Pack Packages (for Template tests)
Expand Down Expand Up @@ -603,10 +489,6 @@ jobs:
chmod +x $HOME/bin/jq
echo "##vso[task.prependpath]$HOME/bin"
displayName: Install jq
- task: NodeTool@0
displayName: Install Node 10.x
inputs:
versionSpec: 10.x
- task: UseDotNet@2
displayName: 'Use .NET Core sdk'
inputs:
Expand Down
3 changes: 2 additions & 1 deletion .azure/pipelines/helix-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,10 +36,11 @@ jobs:
steps:
- script: ./restore.sh -ci
displayName: Restore
- script: ./build.sh -ci --arch arm64 -test -projects $(Build.SourcesDirectory)/eng/helix/helix.proj /p:IsHelixJob=true /p:BuildAllProjects=true /p:BuildNative=true -bl
- script: ./build.sh -ci --arch arm64 -test --no-build-nodejs -projects $(Build.SourcesDirectory)/eng/helix/helix.proj /p:IsHelixJob=true /p:BuildAllProjects=true /p:BuildNative=true -bl
displayName: Run build.sh helix arm64 target
env:
SYSTEM_ACCESSTOKEN: $(System.AccessToken) # We need to set this env var to publish helix results to Azure Dev Ops
installNodeJs: false
artifacts:
- name: Helix_arm64_logs
path: artifacts/logs/
Expand Down
6 changes: 1 addition & 5 deletions build.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -183,18 +183,14 @@ elseif ($Projects) {
}
# When adding new sub-group build flags, add them to this check.
elseif((-not $BuildNative) -and (-not $BuildManaged) -and (-not $BuildNodeJS) -and (-not $BuildInstallers) -and (-not $BuildJava)) {
Write-Warning "No default group of projects was specified, so building the 'managed' and its dependent subsets of projects. Run ``build.cmd -help`` for more details."
Write-Warning "No default group of projects was specified, so building the 'managed' subsets of projects. Run ``build.cmd -help`` for more details."

# This goal of this is to pick a sensible default for `build.cmd` with zero arguments.
# Now that we support subfolder invokations of build.cmd, we will be pushing to have build.cmd build everything (-all) by default

$BuildManaged = $true
}

if ($BuildManaged -and ($NoBuildNodeJS)) {
Write-Warning "Some managed projects that depend on NodeJS projects will be skipped since building NodeJS is disabled."
}

if ($BuildInstallers) { $MSBuildArguments += "/p:BuildInstallers=true" }
if ($BuildManaged) { $MSBuildArguments += "/p:BuildManaged=true" }
if ($BuildNative) { $MSBuildArguments += "/p:BuildNative=true" }
Expand Down
6 changes: 1 addition & 5 deletions build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -213,18 +213,14 @@ elif [ ! -z "$build_projects" ]; then
elif [ -z "$build_managed" ] && [ -z "$build_nodejs" ] && [ -z "$build_java" ] && [ -z "$build_native" ] && [ -z "$build_installers" ]; then
# This goal of this is to pick a sensible default for `build.sh` with zero arguments.
# We believe the most common thing our contributors will work on is C#, so if no other build group was picked, build the C# projects.
__warn "No default group of projects was specified, so building the 'managed' and its dependent subset of projects. Run ``build.sh --help`` for more details."
__warn "No default group of projects was specified, so building the 'managed' subset of projects. Run ``build.sh --help`` for more details."
build_managed=true
fi

if [ "$build_deps" = false ]; then
msbuild_args[${#msbuild_args[*]}]="-p:BuildProjectReferences=false"
fi

if [ "$build_nodejs" = false ] && [ "$build_managed" = true ]; then
__warn "Some managed projects that depend on NodeJS projects will be skipped since building NodeJS is disabled."
fi

# Only set these MSBuild properties if they were explicitly set by build parameters.
[ ! -z "$build_java" ] && msbuild_args[${#msbuild_args[*]}]="-p:BuildJava=$build_java"
[ ! -z "$build_native" ] && msbuild_args[${#msbuild_args[*]}]="-p:BuildNative=$build_native"
Expand Down
Loading