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

chore: automated PR to update ReleaseNotes for 202411.03.0 VHD #5208

Closed
wants to merge 10 commits into from
Closed
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
5 changes: 3 additions & 2 deletions .devcontainer/devcontainer.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,10 @@
{
"name": "Go",
// Or use a Dockerfile or Docker Compose file. More info: https://containers.dev/guide/dockerfile
"image": "mcr.microsoft.com/devcontainers/go:1-1.20-bullseye",
"image": "mcr.microsoft.com/devcontainers/go:1-1.22-bullseye",
"features": {
"ghcr.io/devcontainers-contrib/features/protoc:1": {}
"ghcr.io/devcontainers-contrib/features/protoc:1": {},
"ghcr.io/devcontainers/features/azure-cli:1": {}
},

// Features to add to the dev container. More info: https://containers.dev/features.
Expand Down
7 changes: 7 additions & 0 deletions .github/renovate.json
Original file line number Diff line number Diff line change
Expand Up @@ -87,6 +87,13 @@
{
"matchPackageNames": ["moby-runc", "moby-containerd"],
"extractVersion": "^v?(?<version>.+)$"
},
{
"matchPackageNames": ["aks/aks-gpu-cuda", "aks/aks-gpu-grid"],
"versioning": "regex:^(?<major>\\d+)\\.(?<minor>\\d+)\\.(?<patch>\\d+)-(?<prerelease>\\d{14})$",
"automerge": false,
"enabled": true,
"ignoreUnstable": false
}
],
"customManagers": [
Expand Down
45 changes: 0 additions & 45 deletions .github/workflows/golangci-lint-pr.yml

This file was deleted.

4 changes: 4 additions & 0 deletions .github/workflows/golangci-lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,10 @@ on:
branches:
- master
- main
pull_request:
branches:
- master
- main

permissions:
contents: read
Expand Down
6 changes: 6 additions & 0 deletions .pipelines/.vsts-vhd-automation.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,10 @@ parameters:
- name: VHD_BUILD_ID
displayName: VHD Build ID
type: string
- name: PRTargetBranch
displayName: PR Target Branch
type: string
default: master
- name: ImageBump
displayName: Image Bump + Create Official Branch
type: boolean
Expand Down Expand Up @@ -43,6 +47,7 @@ stages:
GITHUB_PAT: $(GITHUB_PAT)
IMAGE_VERSION: $(IMAGE_VERSION)
VHD_BUILD_ID: ${{ parameters.VHD_BUILD_ID }}
PR_TARGET_BRANCH: ${{ parameters.PRTargetBranch }}
displayName: 'Bump image version and create official branch'
- stage: generate_release_notes
dependsOn: []
Expand All @@ -66,6 +71,7 @@ stages:
IMAGE_VERSION: $(IMAGE_VERSION)
VHD_BUILD_ID: ${{ parameters.VHD_BUILD_ID }}
SKIP_LATEST: $(SKIP_LATEST_RELEASE_NOTES_UPDATE)
PR_TARGET_BRANCH: ${{ parameters.PRTargetBranch }}
displayName: 'Generate release notes'
- stage: create_release
dependsOn: []
Expand Down
1 change: 1 addition & 0 deletions .pipelines/.vsts-vhd-builder.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ pr:
branches:
include:
- master
- dev
paths:
include:
- schemas
Expand Down
2 changes: 2 additions & 0 deletions .pipelines/e2e.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,13 @@ trigger:
branches:
include:
- master
- dev
pr:
branches:
include:
- master
- official/*
- dev
paths:
include:
- .pipelines/e2e.yaml
Expand Down
7 changes: 6 additions & 1 deletion .pipelines/templates/.set-image-version-template.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,12 @@ parameters:

steps:
- bash: |
[ -n "$IMAGE_VERSION_OVERRIDE" ] && echo "IMAGE_VERSION_OVERRIDE is set to $IMAGE_VERSION_OVERRIDE" && exit 0
if [ -n "$IMAGE_VERSION_OVERRIDE" ]; then
echo "IMAGE_VERSION_OVERRIDE is set to $IMAGE_VERSION_OVERRIDE"
echo "##vso[task.setvariable variable=IMAGE_VERSION]$IMAGE_VERSION_OVERRIDE"
exit 0
fi

source vhdbuilder/scripts/automate_generate_version.sh
echo "setting image version to $GENERATED_IMAGE_VERSION"
echo "##vso[task.setvariable variable=IMAGE_VERSION]$GENERATED_IMAGE_VERSION"
Expand Down
6 changes: 3 additions & 3 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -180,11 +180,11 @@ endif
ginkgoBuild: generate
make -C ./test/e2e ginkgo-build

test: test-node-bootstrapper
test: test-aks-node-controller
go test ./...

test-node-bootstrapper:
pushd node-bootstrapper && go test ./... && popd
test-aks-node-controller:
pushd aks-node-controller && go test ./... && popd

.PHONY: test-style
test-style: validate-go validate-shell validate-copyright-headers
Expand Down
1 change: 1 addition & 0 deletions aks-node-controller/.gitignore
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
aks-node-controller
12 changes: 6 additions & 6 deletions node-bootstrapper/app.go → aks-node-controller/app.go
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,8 @@ import (
"os"
"os/exec"

"github.com/Azure/agentbaker/node-bootstrapper/parser"
nbcontractv1 "github.com/Azure/agentbaker/pkg/proto/nbcontract/v1"
"github.com/Azure/agentbaker/aks-node-controller/parser"
aksnodeconfigv1 "github.com/Azure/agentbaker/pkg/proto/aksnodeconfig/v1"
)

type App struct {
Expand All @@ -31,13 +31,13 @@ type ProvisionFlags struct {
}

func (a *App) Run(ctx context.Context, args []string) int {
slog.Info("node-bootstrapper started")
slog.Info("aks-node-controller started")
err := a.run(ctx, args)
exitCode := errToExitCode(err)
if exitCode == 0 {
slog.Info("node-bootstrapper finished successfully")
slog.Info("aks-node-controller finished successfully")
} else {
slog.Error("node-bootstrapper failed", "error", err)
slog.Error("aks-node-controller failed", "error", err)
}
return exitCode
}
Expand Down Expand Up @@ -69,7 +69,7 @@ func (a *App) Provision(ctx context.Context, flags ProvisionFlags) error {
return fmt.Errorf("open proision file %s: %w", flags.ProvisionConfig, err)
}

config := &nbcontractv1.Configuration{}
config := &aksnodeconfigv1.Configuration{}
err = json.Unmarshal(inputJSON, config)
if err != nil {
return fmt.Errorf("unmarshal provision config: %w", err)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -43,12 +43,12 @@ func TestApp_Run(t *testing.T) {
}{
{
name: "missing command argument",
args: []string{"node-bootstrapper"},
args: []string{"aks-node-controller"},
wantExit: 1,
},
{
name: "unknown command",
args: []string{"node-bootstrapper", "unknown"},
args: []string{"aks-node-controller", "unknown"},
wantExit: 1,
},
{
Expand All @@ -58,7 +58,7 @@ func TestApp_Run(t *testing.T) {
},
{
name: "provision command with valid flag",
args: []string{"node-bootstrapper", "provision", "--provision-config=parser/testdata/test_nbc.json"},
args: []string{"aks-node-controller", "provision", "--provision-config=parser/testdata/test_nbc.json"},
setupMocks: func(mc *MockCmdRunner) {
mc.RunFunc = func(cmd *exec.Cmd) error {
return nil
Expand All @@ -68,7 +68,7 @@ func TestApp_Run(t *testing.T) {
},
{
name: "provision command with command runner error",
args: []string{"node-bootstrapper", "provision", "--provision-config=parser/testdata/test_nbc.json"},
args: []string{"aks-node-controller", "provision", "--provision-config=parser/testdata/test_nbc.json"},
setupMocks: func(mc *MockCmdRunner) {
mc.RunFunc = func(cmd *exec.Cmd) error {
return &ExitError{Code: 666}
Expand Down
2 changes: 1 addition & 1 deletion node-bootstrapper/go.mod → aks-node-controller/go.mod
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
module github.com/Azure/agentbaker/node-bootstrapper
module github.com/Azure/agentbaker/aks-node-controller

go 1.22.2

Expand Down
File renamed without changes.
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ import (
// Some options are intentionally non-configurable to avoid customization by users
// it will help us to avoid introducing any breaking changes in the future.
const (
LogFile = "/var/log/azure/node-bootstrapper.log"
LogFile = "/var/log/azure/aks-node-controller.log"
)

func main() {
Expand Down
Loading
Loading