Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

[EngSys] The storage CI pipeline runs on every pull request

Aperta Adatta ai principianti
  • Qualcosa non va?
#7,347 2 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
68/100
Tipo di issue
Funzionalità
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Stack tecnologico
cpp, yaml
Ambito
build-system, ci-cd

Direzione di ricerca

Leggi prima sdk/storage/ci.yml, quindi confronta i filtri dei percorsi di trigger e PR in sdk/eventhubs/ci.yml e sdk/core/ci.yml. Conferma con un maintainer se la pipeline senza filtri è un canary intenzionale prima di modificarla. Il lavoro è completato quando i percorsi di Storage e di build condivisi previsti sono coperti e le modifiche non correlate non avviano più la pipeline.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Client needs-team-attention Service Attention Storage

Summary

sdk/storage/ci.yml declares no paths filter on its trigger block or its pr block, so the cpp - storage - ci pipeline starts on every pull request to main. Every other service pipeline in the repository filters by path.

Motivation

A pull request that changes no storage file still starts the full storage build matrix across Windows, Linux, and macOS. Pull request 7339 changes only doc/DistributedTracing.md and files under sdk/eventhubs/, with no file under sdk/storage/ and no file under sdk/core/, and it still reports 41 cpp - storage - ci checks. The storage pipeline also does not run storage tests on a pull request, and its own comment records "Not running tests on CI but only on LiveTest mode". The cost is build time with no added signal. A count of the paths keys puts storage alone at zero:

sdk/storage/ci.yml:0
sdk/eventhubs/ci.yml:2
sdk/core/ci.yml:2
sdk/keyvault/ci.yml:2
sdk/identity/ci.yml:2
sdk/tables/ci.yml:2

Proposal

Add a paths block to the trigger block and to the pr block of sdk/storage/ci.yml, in the form the other service pipelines already use. The include list must hold sdk/storage and the shared paths that change a storage build, for example eng/, cmake-modules/, and CMakeLists.txt.

Make sure the present state is not deliberate before a change lands. An unfiltered storage pipeline can act as a wide canary that catches a core change which breaks a dependent package. A maintainer must confirm the intent, because the filter removes that coverage.

Lingua principale
C++
Stelle
207
Fork
173
Merge medio
1g 10h
PR unite (30g)
30

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di Azure/azure-sdk-for-cpp

Tutte le issue di Azure/azure-sdk-for-cpp

Issue simili

Altre issue su C++

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.