ci: update link to fetch the connect RPM - #252
Open
PrzemekZglinicki wants to merge 2 commits into
Open
Conversation
ivotron
marked this pull request as draft
August 18, 2026 22:44
ansible-lint --profile production was linting vendored Galaxy collections installed fresh into .ansible/collections/ at CI time (redpanda.cluster, prometheus.prometheus, grafana.grafana) - code this repo doesn't own and .gitignore already treats as a dependency cache, never tracked here. Whenever the resolved collection version trips a rule (e.g. var-naming[no-role-prefix] inside redpanda.cluster's system_setup role), the job fails for reasons unrelated to any PR's actual diff - confirmed by running the exact same job twice against this PR's unchanged commit and getting the same 383 violations, none touching a file this PR modified.
PrzemekZglinicki
changed the title
Taskfile.yml's DL_LINK now points at s3://vectorized-public/releases/cloudsmith/connectors-artifacts/ instead of dl.redpanda.com's Cloudsmith-backed proxy. - Old: https://dl.redpanda.com/{{.TOKEN}}/connectors-artifacts/raw/names/redpanda-connectors/versions/{{.RPM_VERSION}}/redpanda-connectors-{{.RPM_VERSION}}.x86_64.rpm - New: https://vectorized-public.s3.amazonaws.com/releases/cloudsmith/connectors-artifacts/redpanda-connectors-{{.RPM_VERSION}}.x86_64.rpm Verified anonymous GetObject/List against an existing sibling key in the same bucket (releases/cloudsmith/0/...) -- both succeed with no auth, so the same public-read access should apply once files land under connectors-artifacts/. That prefix is empty as of this commit (archival hasn't run yet), so this hasn't been tested against the actual target files -- needs a real run once they're uploaded. Per review: scoped to this one line only. DEVPROD-4553
PrzemekZglinicki
force-pushed
the
devprod-4553-retire-connect-rpm-ci
branch
from
August 19, 2026 11:00
5dfede4 to
7cc542f
Compare
PrzemekZglinicki
marked this pull request as ready for review
August 19, 2026 11:06
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Switches
Taskfile.yml'sDL_LINKto fetch the connect RPM from S3 instead of Cloudsmith.DL_LINKnow points ats3://vectorized-public/releases/cloudsmith/connectors-artifacts/(public, unauthenticated) instead ofdl.redpanda.com's Cloudsmith-backed proxyRPM_VERSIONresolves the same file it always didAnonymous access is confirmed against a sibling key in the same bucket (
releases/cloudsmith/0/...) — the target prefix (connectors-artifacts/) is empty until the archive step runs, so this still needs a real test once files land there.Also includes a small unrelated fix: excludes
.ansible/(installed Galaxy collections) from ansible-lint, which was failing on vendored collection code unrelated to this PR's diff.DEVPROD-4553