Skip to content

Pin dependencies in requirements.txt - #3

Merged
tarekziade merged 1 commit into
mainfrom
artem/pin-requirements
Jul 21, 2022
Merged

tarekziade merged 1 commit into
mainfrom
artem/pin-requirements

Conversation

@artem-shelkovnikov

Copy link
Copy Markdown
Member

Suggestion when reading CONTRIBUTING.rst - pin requirements of currently used dependencies declared in requirements.txt.

@tarekziade
tarekziade merged commit 00893fd into main Jul 21, 2022
@tarekziade
tarekziade deleted the artem/pin-requirements branch July 21, 2022 15:07
Apmats added a commit that referenced this pull request May 6, 2026
## Summary

Adds `release-eng: BUILD_AND_READ` to the `connectors` pipeline's teams
block so release-eng team members can trigger builds.

## Why

The `connectors-version-bump` pipeline (introduced in #3986) executes a
`trigger:` step that starts a DRA build on the target release branch on
the main `connectors` pipeline. Buildkite propagates the original
triggering user's identity through trigger chains, so the downstream
build on `connectors` is created as that user -- and the user needs
Build access on `connectors` for the trigger to succeed.

We already granted release-eng `BUILD_AND_READ` on
`connectors-version-bump` (so they can trigger our pipeline at all), but
missed adding it on the main `connectors` pipeline. This caused failures
in [build
#3](https://buildkite.com/elastic/connectors-version-bump/builds/3)
(9.3.5) and [build
#4](https://buildkite.com/elastic/connectors-version-bump/builds/4)
(8.19.16) when triggered by Nina Lee, while [build
#2](https://buildkite.com/elastic/connectors-version-bump/builds/2)
(9.4.1) triggered by Julien Mailleret succeeded -- presumably via
individual grants.

This grants team-level access so any release-eng member can complete the
version bump end-to-end.

## Test plan

- [ ] After merge and Backstage sync (~few minutes), re-trigger build #3
or #4 from the centralized pipeline and confirm the `Trigger DRA build
on ${BRANCH}` step succeeds

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
waelgazal added a commit to HamoudeFakhoury01/connectors that referenced this pull request Jun 2, 2026
…ic#3/elastic#7 résolus, §10 déploiement)

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants