forked from kubernetes-sigs/cluster-api-provider-openstack
-
Notifications
You must be signed in to change notification settings - Fork 42
UPSTREAM-SYNC: Sync release-5.0 with upstream release-0.15 #442
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
Open
stephenfin
wants to merge
406
commits into
openshift:release-5.0
Choose a base branch
from
shiftstack:sync-release-5.0-with-release-0.15
base: release-5.0
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
406 commits
Select commit
Hold shift + click to select a range
622104d
Merge pull request #3105 from Nordix/bump-go-1.25.9
k8s-ci-robot 37eeb30
Merge pull request #3096 from bnallapeta/v1beta2-tests
k8s-ci-robot fba5437
Switch light GitHub workflows to ubuntu-slim runner
larainema 1266f9e
Merge pull request #3103 from larainema/fix/switch-light-workflows-ub…
k8s-ci-robot a0da76e
Bump the go_modules group across 2 directories with 1 update
dependabot[bot] ea8f397
Merge pull request #3108 from kubernetes-sigs/dependabot/go_modules/g…
k8s-ci-robot 76ab594
Add missing conditions on network error paths in OpenStackCluster rec…
larainema 8aa37b7
Merge pull request #3098 from bnallapeta/migration_docs
k8s-ci-robot a8abcb8
Clarify providerID initialization approaches in documentation
larainema 85e352f
:seedling:(deps): Bump github.com/itchyny/gojq
dependabot[bot] 688ca89
:seedling:(deps): Bump sigs.k8s.io/cluster-api
dependabot[bot] 74a90d5
Update generated code
dependabot[bot] 10e5009
Merge pull request #3112 from kubernetes-sigs/dependabot/go_modules/h…
k8s-ci-robot 4ed6860
Merge pull request #3104 from larainema/fix/test-sg-failure-clears-re…
k8s-ci-robot 0c1a69b
Merge pull request #3107 from larainema/fix/add-missing-conditions-on…
k8s-ci-robot 34c6ce5
Merge pull request #3113 from kubernetes-sigs/dependabot/go_modules/m…
k8s-ci-robot 0f25c9f
Merge pull request #3056 from jfpucheu/feat/allowedaddresspairs-mutable
k8s-ci-robot 89ad437
Use port_trusted_vif extension for trusted VF when available
larainema b3b1b35
Merge pull request #3111 from larainema/docs/clarify-providerid-initi…
k8s-ci-robot 3e049fc
Standardize flavor in OpenStackMachine
nikParasyr 0f0b378
Merge pull request #3087 from nikParasyr/v1beta2_flavor
k8s-ci-robot 55bab43
Replace embedded quick-start with link to upstream CAPI docs
larainema 24a0fa1
Bump github.com/go-git/go-git/v5
dependabot[bot] 2e5c906
Replace gofuzz with randfill
lentzi90 4e5488f
Add fuzzy conversion tests for v1beta1 API types
lentzi90 dcfa896
:seedling:(deps): Bump the all-github-actions group with 3 updates
dependabot[bot] 351b411
:seedling:(deps): Bump the all-go-mod-patch-and-minor group across 1 …
dependabot[bot] eb26fd4
:seedling:(deps): Bump sigs.k8s.io/cluster-api
dependabot[bot] f94f493
Update generated code
dependabot[bot] b329029
Merge pull request #3124 from kubernetes-sigs/dependabot/github_actio…
k8s-ci-robot f32be81
Merge pull request #3125 from kubernetes-sigs/dependabot/go_modules/m…
k8s-ci-robot 18f6ce6
Merge pull request #3126 from kubernetes-sigs/dependabot/go_modules/m…
k8s-ci-robot d7975df
Merge pull request #3119 from Nordix/lentzi90/fuzz-conversion-v1beta1…
k8s-ci-robot ecd616d
Add Kube API Linter with everything excluded
lentzi90 c53c196
Add managedNetworks on OpenStackCluster
nikParasyr 4a9606b
Merge pull request #3123 from kubernetes-sigs/dependabot/go_modules/h…
k8s-ci-robot 22394b9
Merge pull request #3110 from larainema/ci/periodic-docs-build
k8s-ci-robot b3cb144
Merge pull request #3132 from Nordix/lentzi90/kal
k8s-ci-robot c412e69
:rocket: Release v0.13.6
smoshiur1237 dc486c0
:rocket: Release v0.14.3
smoshiur1237 014e056
Merge pull request #3135 from Nordix/release-notes-0.14.3
k8s-ci-robot e346d18
Merge pull request #3136 from Nordix/release-notes-0.13.6
k8s-ci-robot aa6815b
Merge pull request #3122 from nikParasyr/v1beta2_network
k8s-ci-robot 28f223d
:rocket: Release v0.14.4
smoshiur1237 aa50773
:rocket: Release v0.13.7
smoshiur1237 6a93ca6
Merge pull request #3142 from Nordix/release-notes-0.14.4
k8s-ci-robot 67d65d4
Merge pull request #3141 from Nordix/release-notes-0.13.7
k8s-ci-robot bd9f904
:seedling:(deps): Bump tj-actions/changed-files
dependabot[bot] 7480121
:seedling:(deps): Bump the all-go-mod-patch-and-minor group across 1 …
dependabot[bot] 610644e
:seedling:(deps): Bump sigs.k8s.io/cluster-api
dependabot[bot] f160281
Update generated code
dependabot[bot] c647844
Merge pull request #3144 from kubernetes-sigs/dependabot/github_actio…
k8s-ci-robot e2fa9f8
Dependabot: Ignore ORC on release-0.14
lentzi90 24b4cbb
Merge pull request #3145 from kubernetes-sigs/dependabot/go_modules/m…
k8s-ci-robot 73a966e
Merge pull request #3146 from kubernetes-sigs/dependabot/go_modules/m…
k8s-ci-robot 53815e2
Merge pull request #3150 from Nordix/lentzi90/dependabot-ignore-orc-0.14
k8s-ci-robot 7b5841e
Add managedRouter on OpenStackCluster
nikParasyr a558fcf
trivial: Use lazy evaluation for shell calls
stephenfin aa3b922
Merge pull request #3143 from nikParasyr/v1beta2_router
k8s-ci-robot 79f7dcd
:seedling:(deps): Bump the all-go-mod-patch-and-minor group across 2 …
dependabot[bot] d794634
Merge pull request #3156 from kubernetes-sigs/dependabot/go_modules/m…
k8s-ci-robot 15631a2
hack: Bump github.com/ahmetb/gen-crd-api-reference-docs
stephenfin 31504b1
Merge pull request #3154 from shiftstack/faster-makefile
k8s-ci-robot 163fb87
Add metadata for v0.15 and update e2e
lentzi90 dd3bc79
Merge pull request #3159 from shiftstack/bump-gen-crd-api-reference-docs
k8s-ci-robot 21d7d42
Bump Go version to 1.25.10
smoshiur1237 72e6f32
Rename managedSecurityGroups.allNodesSecurityGroupRules
nikParasyr 8cf4c3c
Merge pull request #3120 from larainema/fix/port-trusted-vif-extension
k8s-ci-robot 3ded8f9
Merge pull request #3153 from Nordix/lentzi90/e2e-v1beta2
k8s-ci-robot 4c7f8c4
Merge pull request #3162 from Nordix/bump-go-1.25.10
k8s-ci-robot d1f2af2
Bump github.com/go-git/go-git/v5
dependabot[bot] 9ea3aac
Merge pull request #3165 from kubernetes-sigs/dependabot/go_modules/h…
k8s-ci-robot b0c867d
Merge pull request #3155 from nikParasyr/v1beta2_sg
k8s-ci-robot 3037521
KAL: Enable linters that require no fixes
lentzi90 28387b4
KAL: Enable commentstart and apply autofix
lentzi90 e3ed513
KAL: Manual fixes for commentstart
lentzi90 9d1c8c8
KAL: Enabled conditions linter
lentzi90 878225f
KAL: Enable statusoptional and fix
lentzi90 79731cc
Merge pull request #3161 from Nordix/lentzi90/kal-autofix
k8s-ci-robot c516c72
Ignore ginkgo and setup-envtest bumps
lentzi90 21ed2be
Bump github.com/go-git/go-git/v5
dependabot[bot] 7693014
Merge pull request #3174 from kubernetes-sigs/dependabot/go_modules/h…
k8s-ci-robot 2aa82b6
Merge pull request #3173 from Nordix/lentzi90/dependabot-config
k8s-ci-robot ee5652c
:seedling:(deps): Bump zizmorcore/zizmor-action
dependabot[bot] 9e98870
Bump CI go version to 1.26.3
lentzi90 f5508c1
Merge pull request #3176 from kubernetes-sigs/dependabot/github_actio…
k8s-ci-robot f885b04
Group API server fields under spec.apiServer
nikParasyr 2225c1f
Rename status.apiServerLoadBalancer to status.apiServerManagedLoadBal…
nikParasyr cd71112
Merge pull request #3180 from Nordix/lentzi90/go-1.26
k8s-ci-robot 82cb500
Bump golanci-lint to v2.12.2
lentzi90 744afd5
Merge pull request #3181 from Nordix/lentzi90/golangci-lint
k8s-ci-robot 5bea9ec
:seedling:(deps): Bump the all-go-mod-patch-and-minor group across 2 …
dependabot[bot] c4a287a
✏️ Fix rules not being propagated to all nodes.
Whisper40 b48a75c
Merge pull request #3182 from kubernetes-sigs/dependabot/go_modules/m…
k8s-ci-robot 25d093e
Patch govulncheck to support ignoring findings
lentzi90 5c4ec2e
Merge pull request #3185 from Nordix/lentzi90/govulncheck-patch
k8s-ci-robot a71c03c
Enable optionalorrequired linter and add required/optional markers
lentzi90 55f090c
Merge pull request #2660 from sebltm/multi-az-lb-proposal
k8s-ci-robot 46bb4b6
Merge pull request #3169 from nikParasyr/v1beta2_lb
k8s-ci-robot 438b0e4
:warning: Rename OSC.ManagedNetwork.DisablePortSecurity to EnablePort…
nikParasyr d60baf5
:warning: Rename OSC.DisableExternalNetwork to EnableExternalNetwork
nikParasyr 6037fb9
:warning: Rename OSC.DisableFloatingIP to EnableFloatingIP in APIServer
nikParasyr 52e5ae3
chore: standardize logging and regenerate generated files
AryanSharma9917 aa4d690
Merge pull request #3183 from Nordix/lentzi90/kal-optionalrequired
k8s-ci-robot eb758d5
:seedling:(deps): Bump the all-go-mod-patch-and-minor group across 1 …
dependabot[bot] d223cc4
Add nil guard for LoadBalancerNetwork
lentzi90 c1793eb
Merge pull request #3194 from kubernetes-sigs/dependabot/go_modules/m…
k8s-ci-robot c989586
Merge pull request #3192 from Nordix/lentzi90/nil-guard-loadbalancer-…
k8s-ci-robot 19ba2eb
bump go version to 1.26.4
bnallapeta e8e6fe4
bump gomodguard to v2
bnallapeta f7de663
Merge pull request #3198 from bnallapeta/gomodguard
k8s-ci-robot 4cc5d34
Bump x/net to v0.55.0
lentzi90 84d7427
Merge pull request #3201 from Nordix/lentzi90/x-net-bump
k8s-ci-robot bcacf2a
Merge pull request #3197 from bnallapeta/bump-go
k8s-ci-robot b5670f9
:warning: Rename OSM.ports[*].DisablePortSecurity to EnablePortSecurity
nikParasyr 1cd81a1
Merge pull request #3184 from Whisper40/fix/issue-3175
k8s-ci-robot 3951890
Merge pull request #3189 from nikParasyr/v1beta2_positive
k8s-ci-robot b5a72cd
Migrate ci to stable/2026.1
nikParasyr e48f3ef
Enable PriorityQueue per default
nikParasyr 8d545b2
Merge pull request #3186 from AryanSharma9917/aryan/issue-1314-capi-l…
k8s-ci-robot 31804c4
KAL: Enable intergers linter
lentzi90 6502d30
Bump cloudbuild image to support go 1.26
nikParasyr a6f9102
Merge pull request #3203 from Nordix/lentzi90/kal-integers
k8s-ci-robot d681831
Replace requeue delay with watch
stephenfin 072162d
trivial: Use lazy evaluation for shell calls (redux)
stephenfin 171eade
trivial: Simplify GOPROXY configuration
stephenfin 7442f61
trivial: Use native builds on arm64
stephenfin 42aa7f7
Merge pull request #3207 from nikParasyr/cloudbuild_1_26
k8s-ci-robot 5ab86b6
Merge pull request #3211 from shiftstack/delete-informer
k8s-ci-robot 2544c9a
Merge pull request #3212 from shiftstack/faster-makefile
k8s-ci-robot c7774a3
:seedling:(deps): Bump actions/checkout in the all-github-actions group
dependabot[bot] 5d772d5
:seedling:(deps): Bump the all-go-mod-patch-and-minor group across 1 …
dependabot[bot] edad488
Merge pull request #3213 from kubernetes-sigs/dependabot/github_actio…
k8s-ci-robot deab156
CI: Correctly pin to v1.13 latest
nikParasyr 3ebf4ad
Merge pull request #3205 from nikParasyr/ci_bump
k8s-ci-robot 48f8efc
Merge pull request #3214 from kubernetes-sigs/dependabot/go_modules/m…
k8s-ci-robot 17b97c5
Update cluster-api/test to v1.13.2
nikParasyr 979d421
Merge pull request #3220 from nikParasyr/capi-test-mod-update
k8s-ci-robot 1f222df
Merge pull request #3219 from nikParasyr/capi-test-update
k8s-ci-robot 46da4df
Merge pull request #3206 from nikParasyr/priority_queue_beta
k8s-ci-robot 3eac57b
feat: support multiple subnets via primarySubnet field
memorais 834083f
Merge pull request #3210 from memorais/feature/multiple-subnets
k8s-ci-robot 41c933d
Enable requiredfields linter
lentzi90 a9e6c9e
Merge pull request #3221 from Nordix/lentzi90/kal-requiredfields
k8s-ci-robot 658edb0
Enable conflictingmarkers linter and fix issues
lentzi90 83dfdb2
E2E: Use Kubernetes v1.36.1 in tests
nikParasyr d2d6a13
Enable ssatags linter and fix findings
lentzi90 20a1532
Merge pull request #3223 from Nordix/lentzi90/kal-conflictingmarkers
k8s-ci-robot 8fa4ce2
Merge pull request #3224 from Nordix/lentzi90/kal-ssatags
k8s-ci-robot 2299e18
Merge pull request #3208 from nikParasyr/k8s_1.36
kubernetes-prow[bot] 753b38b
:seedling:(deps): Bump the all-go-mod-patch-and-minor group across 2 …
dependabot[bot] db6a800
Merge pull request #3229 from kubernetes-sigs/dependabot/go_modules/m…
kubernetes-prow[bot] 5a79014
:rocket: Release v0.14.5
smoshiur1237 9394678
Merge pull request #3232 from Nordix/release-notes-0.14.5
kubernetes-prow[bot] a5af417
:rocket: Release v0.13.8
smoshiur1237 7cff110
:rocket: Release v0.14.6
smoshiur1237 4980d8d
Merge pull request #3238 from Nordix/release-notes-0.14.6
kubernetes-prow[bot] 5d31334
Merge pull request #3233 from Nordix/release-notes-0.13.8
kubernetes-prow[bot] b315089
🚀 Release v0.15.0-alpha.0
lentzi90 a2f4e8c
Merge pull request #3240 from Nordix/lentzi90/release-0.15-alpha
kubernetes-prow[bot] 76a9dc5
Bump ORC to v2.6.0 in E2E
tanayarun 20e884d
Merge pull request #3242 from tanayarun/bump-orc-v2.6.0
kubernetes-prow[bot] b82f40f
:seedling:(deps): Bump the all-github-actions group with 2 updates
dependabot[bot] acaefd4
:seedling:(deps): Bump the all-go-mod-patch-and-minor group across 2 …
dependabot[bot] 876b5e2
Merge pull request #3243 from kubernetes-sigs/dependabot/github_actio…
kubernetes-prow[bot] 71a2a62
Merge pull request #3247 from kubernetes-sigs/dependabot/go_modules/m…
kubernetes-prow[bot] 67dffb9
:seedling:(deps): Bump the all-github-actions group with 2 updates
dependabot[bot] 85a4397
Merge pull request #3249 from kubernetes-sigs/dependabot/github_actio…
kubernetes-prow[bot] 1027bbf
Bump go version to 1.26.5
smoshiur1237 34b7418
Merge pull request #3257 from Nordix/bump-go-1-26-5
kubernetes-prow[bot] 2abf96b
:seedling:(deps): Bump lycheeverse/lychee-action
dependabot[bot] 53b1b74
allow unconditional providerID updates in OpenStackMachine
simkam 5ba0458
docs: remove reference to removed kustomize installation method for ORC
Atomsoldat d8c7876
:seedling:(deps): Bump the all-go-mod-patch-and-minor group across 1 …
dependabot[bot] 7f4d1cd
Fix badges on README.md
nikParasyr 3e55a18
Merge pull request #3261 from kubernetes-sigs/dependabot/github_actio…
kubernetes-prow[bot] ff38282
Configure dependabot to avoid golang bumps
lentzi90 391ec39
Merge pull request #3265 from simkam/providerid-webhook
kubernetes-prow[bot] 0cb9951
Merge pull request #3269 from kubernetes-sigs/dependabot/go_modules/m…
kubernetes-prow[bot] d4ad527
fix(networking): infer IP version from subnet CIDR on creation
Jamstah 66a7761
Merge pull request #3273 from nikParasyr/go_report
kubernetes-prow[bot] 6ea3bd4
Merge pull request #3280 from Nordix/lentzi90/dependabot-ignores
kubernetes-prow[bot] a195979
Merge pull request #3267 from Atomsoldat/docs/remove-reference-to-orc…
kubernetes-prow[bot] 1f360f1
:seedling:(deps): Bump the all-go-mod-patch-and-minor group across 1 …
dependabot[bot] f4eae3a
Update generated code
dependabot[bot] 560e296
Prevent go requirement changes
lentzi90 a64bf7b
Merge pull request #3289 from kubernetes-sigs/dependabot/go_modules/m…
kubernetes-prow[bot] 533b1af
Bump the go_modules group across 2 directories with 2 updates
dependabot[bot] 98efea3
Merge pull request #3291 from kubernetes-sigs/dependabot/go_modules/g…
kubernetes-prow[bot] adb3a2f
Merge pull request #3290 from Nordix/lentzi90/dependabot-prometheus-g…
kubernetes-prow[bot] 81526c5
:seedling:(deps): Bump the all-github-actions group across 1 director…
dependabot[bot] 1cb372b
Merge pull request #3285 from kubernetes-sigs/dependabot/github_actio…
kubernetes-prow[bot] b20b961
Add metadata to OpenStackClusterTemplate.spec
nikParasyr 910348d
Merge pull request #3276 from Jamstah/ipv6-create-subnet
kubernetes-prow[bot] 2eb6968
Add ClusterClass topology tests for ubuntu and bastion
larainema 1602863
:seedling:(deps): Bump zizmorcore/zizmor-action
dependabot[bot] f883745
:seedling:(deps): Bump the all-go-mod-patch-and-minor group across 1 …
dependabot[bot] 197c0b7
Merge pull request #3296 from kubernetes-sigs/dependabot/github_actio…
kubernetes-prow[bot] c870f27
Merge pull request #3297 from kubernetes-sigs/dependabot/go_modules/m…
kubernetes-prow[bot] 7793bd1
fix(openstackserver): recover from Nova ERROR state
mxm-tr 6640c68
:rocket: Release v0.14.7
smoshiur1237 d0cc636
:rocket: Release v0.13.9
smoshiur1237 bab1458
🚀 Release v0.15.0-beta.0
lentzi90 67ac8ac
Update gcb-docker-gcloud image registry and digest
lentzi90 afd402a
Merge pull request #3301 from Nordix/release-notes-0.13.9
kubernetes-prow[bot] d664653
Merge pull request #3302 from Nordix/release-notes-0.14.7
kubernetes-prow[bot] 872fbdd
Bump github.com/go-git/go-git/v5
dependabot[bot] d2d50bf
Merge pull request #3303 from Nordix/lentzi90/release-notes-v0.15.0-b…
kubernetes-prow[bot] 9315e6f
Merge pull request #3305 from Nordix/lentzi90/cloudbuild-image
kubernetes-prow[bot] 2ef1dde
:seedling:(deps): Bump zizmorcore/zizmor-action
dependabot[bot] e91cb99
Merge pull request #3308 from kubernetes-sigs/dependabot/github_actio…
kubernetes-prow[bot] f196215
Docs: Include gcrane example in releaseing doc
lentzi90 ae4e495
Merge pull request #3307 from kubernetes-sigs/dependabot/go_modules/h…
kubernetes-prow[bot] 832645c
Bump CAPI to v1.14.0
lentzi90 abd54a9
Bump Go version to 1.26.6
smoshiur1237 6071301
:seedling:(deps): Bump EndBug/add-and-commit
dependabot[bot] 815a9a4
Merge pull request #3309 from Nordix/lentzi90/bump-capi-v1.14.0
kubernetes-prow[bot] 5f51db4
Merge pull request #3310 from Nordix/lentzi90/release-docs-gcrane
kubernetes-prow[bot] 27b64f2
Merge pull request #3311 from Nordix/uplift-go-1.26.6
kubernetes-prow[bot] 78ef68d
Merge pull request #3300 from mxm-tr/main
kubernetes-prow[bot] 1811c83
Guard Neutron tagging by extension support
dlanov 2bbb306
Merge pull request #3121 from larainema/feat/clusterclass-topology-tests
kubernetes-prow[bot] 52a4d6e
Merge pull request #3278 from nikParasyr/clusterclass_metadata
kubernetes-prow[bot] 6e915e5
Merge pull request #3315 from kubernetes-sigs/dependabot/github_actio…
kubernetes-prow[bot] e52de58
Merge pull request #3319 from dlanov/fix-standard-attr-tag-extension
kubernetes-prow[bot] 0ca81e3
🚀 Release v0.13.10
lentzi90 b814fc6
🚀 Release v0.14.8
lentzi90 7bfa2f6
🚀 Release v0.15.0-rc.0
lentzi90 8d6be04
Merge pull request #3323 from Nordix/lentzi90/release-notes-v0.13.10
kubernetes-prow[bot] 0d8b0af
Merge pull request #3324 from Nordix/lentzi90/release-notes-v0.14.8
kubernetes-prow[bot] 10dfeed
Merge pull request #3325 from Nordix/lentzi90/release-notes-v0.15.0-rc.0
kubernetes-prow[bot] f0bdcf0
:seedling:(deps): Bump the all-go-mod-patch-and-minor group across 2 …
dependabot[bot] 04ba3a0
🐛 Fix lint failures surfaced by golangci-lint v2.13.1 bump
lentzi90 ff93d73
:seedling:(deps): Bump the all-github-actions group across 1 director…
dependabot[bot] 7a81528
Merge pull request #3341 from kubernetes-sigs/dependabot/github_actio…
kubernetes-prow[bot] 0ce872c
Avoid (re)setting zswap
lentzi90 7d4fcbd
Merge pull request #3346 from k8s-infra-cherrypick-robot/cherry-pick-…
kubernetes-prow[bot] e6ce95d
Merge pull request #3336 from Nordix/lentzi90/fix-pr-3333-lint
kubernetes-prow[bot] 0a1ed12
:seedling:(deps): Bump the all-go-mod-patch-and-minor group across 2 …
dependabot[bot] f2e78cd
Update generated code
dependabot[bot] fb70874
Merge pull request #3348 from kubernetes-sigs/dependabot/go_modules/r…
kubernetes-prow[bot] 52af106
E2E: Pin glance to working commit
lentzi90 8e8f6c4
Merge pull request #3354 from k8s-infra-cherrypick-robot/cherry-pick-…
kubernetes-prow[bot] 7cccace
Revert "E2E: Pin glance to working commit"
lentzi90 940c439
Merge pull request #3359 from k8s-infra-cherrypick-robot/cherry-pick-…
kubernetes-prow[bot] c2001d7
Merge remote-tracking branch 'upstream/release-0.15' into sync-releas…
stephenfin d1c277a
CARRY: Run `make full-vendoring` post merge
stephenfin 41cae1a
CARRY: Drop unused makefile targets
stephenfin 9643b66
UPSTREAM: <drop>: make ocp-manifests
stephenfin 5744861
make: Re-add missing variable
stephenfin File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
The diff you're trying to view is too large. We only load the first 3000 changed files.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🔒 Security & Privacy | 🛡️ Analyzed with Security Review | 🟡 Minor | ⚡ Quick win
🧩 Analysis chain
🏁 Script executed:
Repository: openshift/cluster-api-provider-openstack
Length of output: 2498
🌐 Web query:
site:docs.github.com GitHub Actions pull_request branches filter base branch fork pull request GITHUB_TOKEN read-only permissions💡 Result:
<search_synthesis>
In GitHub Actions, you can control which pull requests trigger a workflow by using the branches filter within the pull_request event configuration [1][2]. This filter operates against the base branch—the branch that the pull request is targeting [1][2]. Workflow Filter Configuration: You can specify the target base branches using the branches key, which supports glob patterns [2][3]. on: pull_request: branches: - main - &
#39;releases/' If you need to both include and exclude branches, use the branches filter with the! character to denote exclusions, as you cannot use branches and branches-ignore simultaneously for the same event [2][3]. Security and GITHUB_TOKEN Permissions: When a pull request originates from a fork, GitHub applies security restrictions to the GITHUB_TOKEN to protect the repository from unauthorized code execution [4]. 1. pull_request Event: Workflows triggered by the pull_request event run using the code from the pull request's merge commit [4]. Because this code is untrusted, GitHub restricts the GITHUB_TOKEN to read-only permissions and denies access to repository secrets [4]. 2. pull_request_target Event: If you require elevated permissions (such as access to secrets) for pull requests from forks, the pull_request_target event is used [4]. This event executes the workflow file from the base repository's default branch rather than the fork's branch, making it safer to grant read/write access and secrets [4]. Regardless of the event, you should always explicitly define the minimum required permissions for your GITHUB_TOKEN using the permissions key at the workflow or job level to follow security best practices [5]. permissions: contents: read # Add other necessary permissions here You can also use the GITHUB_BASE_REF environment variable within your workflow steps to programmatically identify the target base branch of the pull request [6].</search_synthesis>
<source_evidence>
Citations:
Sensitive Data Exposure
Reachability: External
Exploitability: Moderate
CWE: CWE-522 — Insufficiently Protected Credentials
Restrict this credentialed job to Dependabot events.
pull_request.branchesmatches the base branch, not the pull-request head, so it does not restrict this workflow to Dependabot. A fork pull request receives a read-onlyGITHUB_TOKEN, but repository-controlled Make targets can still read and exfiltrate that token. Thepushandworkflow_dispatchtriggers also lack a Dependabot guard. Add a job condition for Dependabot-authored pull requests and Dependabot pushes, and keepworkflow_dispatchexcluded or equally guarded. Keep only the permissions required for the commit.🤖 Prompt for AI Agents
Source: Path instructions