Skip to content

SC-130842: Fix support bundle goldpinger namespace #3125

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 34 commits into
base: main
Choose a base branch
from
Draft

SC-130842: Fix support bundle goldpinger namespace #3125

wants to merge 34 commits into from

Conversation

@banjoh
Copy link
Member

@banjoh banjoh commented Oct 27, 2025

What this PR does / why we need it:

Correct namespace used to create goldpinger pods

Which issue(s) this PR fixes:

https://app.shortcut.com/replicated/story/130842/support-bundle-goldpinger-pods-scheduled-in-non-existent-namespace

Does this PR require a test?

NONE

Does this PR require a release note?

NONE

Does this PR require documentation?

NONE

@github-actions
Copy link

github-actions bot commented Oct 27, 2025 •
edited
Loading

This PR has been released (on staging) and is available for download with a embedded-cluster-smoke-test-staging-app license ID.

Online Installer:

curl "https://staging.replicated.app/embedded/embedded-cluster-smoke-test-staging-app/ci/appver-dev-6800933" -H "Authorization: $EC_SMOKE_TEST_LICENSE_ID" -o embedded-cluster-smoke-test-staging-app-ci.tgz

Airgap Installer (may take a few minutes before the airgap bundle is built):

curl "https://staging.replicated.app/embedded/embedded-cluster-smoke-test-staging-app/ci-airgap/appver-dev-6800933?airgap=true" -H "Authorization: $EC_SMOKE_TEST_LICENSE_ID" -o embedded-cluster-smoke-test-staging-app-ci.tgz

Happy debugging!

@banjoh banjoh marked this pull request as draft October 27, 2025 19:21
@banjoh banjoh marked this pull request as ready for review October 27, 2025 19:57
replicated-ci-ec and others added 6 commits October 28, 2025 06:05
@replicated-ci-ec @sgalsaleh
chore: update velero images (#3128)
47a1691 
Update image versions

Co-authored-by: sgalsaleh <[email protected]>
@replicated-ci-ec @sgalsaleh
chore: update openebs images (#3129)
04c8d82 
Update image versions

Co-authored-by: sgalsaleh <[email protected]>
@replicated-ci-ec @sgalsaleh
chore: update k0s images (#3130)
8ab290f 
Update image versions

Co-authored-by: sgalsaleh <[email protected]>
@replicated-ci-ec @diamonwiggins
feat: update adminconsole version (#3131)
565cd22 
updated adminconsole version

Co-authored-by: diamonwiggins <[email protected]>
@JGAntunes
chore(api): improve API logging (#3124)
5a8ff7d 
* chore(api): improve API logging

* chore: fix rebase issue

* chore: cursor feedback
@banjoh
Empty commit to trigger build
35caeb2 
Signed-off-by: Evans Mungai <[email protected]>
@banjoh banjoh marked this pull request as draft October 29, 2025 17:21
replicated-ci-ec and others added 8 commits October 29, 2025 18:05
@replicated-ci-ec @sgalsaleh
chore: update k0s images (#3132)
47ec9d3 
Update image versions

Co-authored-by: sgalsaleh <[email protected]>
@replicated-ci-ec @sgalsaleh
chore: update k0s images (#3137)
a32ea29 
Update image versions

Co-authored-by: sgalsaleh <[email protected]>
@dependabot
build(deps-dev): bump tar from 7.5.1 to 7.5.2 in /web in the npm_and_…
8ade3e4 
…yarn group across 1 directory (#3138)

build(deps-dev): bump tar

Bumps the npm_and_yarn group with 1 update in the /web directory: [tar](https://github.com/isaacs/node-tar).


Updates `tar` from 7.5.1 to 7.5.2
- [Release notes](https://github.com/isaacs/node-tar/releases)
- [Changelog](https://github.com/isaacs/node-tar/blob/main/CHANGELOG.md)
- [Commits](isaacs/node-tar@v7.5.1...v7.5.2)

---
updated-dependencies:
- dependency-name: tar
  dependency-version: 7.5.2
  dependency-type: indirect
  dependency-group: npm_and_yarn
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@sgalsaleh
Move custom CIDR e2e test to unit & dryrun tests (#3126)
1d50ffb 
* Move custom CIDR e2e test to unit & integration tests
@emosbaugh
feat(api): sync license before templating (#3109)
b35ce5b
@banjoh
Empty commit
2790701 
Signed-off-by: Evans Mungai <[email protected]>
@screspod
Add logic to warm template engine cache (#3135)
6edfe65 
* Add logic to warm template engine cache

Signed-off-by: Steven Crespo <[email protected]>

* f

Signed-off-by: Steven Crespo <[email protected]>

* f

Signed-off-by: Steven Crespo <[email protected]>

---------

Signed-off-by: Steven Crespo <[email protected]>
@sgalsaleh
Validate custom cidrs for airgap / proxy (#3142)
9645263 
* Validate custom cidrs for airgap / proxy

* validate no proxy in charts as well
@banjoh
Copy link
Member Author

banjoh commented Oct 31, 2025 

Failed to query goldpinger endpoint in cluster: Get "http://ts-goldpinger.embedded-cluster.svc.cluster.local:80/check_all": dial tcp ***HIDDEN***:80: i/o timeout

The pods is get deployed, but it does not look like pings are succeeding

banjoh and others added 8 commits October 31, 2025 18:30
@banjoh
Add namespace to goldpinger pod pinging goldpinger
5959572 
Signed-off-by: Evans Mungai <[email protected]>
@sgalsaleh
Remove duplicate unnecessary tests (#3143)
1599091 
* Remove duplicate / unnecessary tests
@NoaheCampbell
Linter (#3140)
2a4bea9 
* added embedded cluster lint command for replicated cli

* added basic yaml syntax checking

* Create README.md

* addressing bugbot concerns

* linter failure fix

* added json output format

* refactored linter for new custom domain unit tests

* sanitize error
@dependabot
build(deps): bump the security group across 2 directories with 3 upda…
8b1e55b 
…tes (#3145)

Bumps the security group with 1 update in the /e2e/playwright directory: [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node).
Bumps the security group with 3 updates in the /web directory: [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node), [react-router-dom](https://github.com/remix-run/react-router/tree/HEAD/packages/react-router-dom) and [@netlify/vite-plugin](https://github.com/netlify/primitives).


Updates `@types/node` from 24.9.1 to 24.9.2
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/node)

Updates `@types/node` from 24.9.1 to 24.9.2
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/node)

Updates `react-router-dom` from 7.9.4 to 7.9.5
- [Release notes](https://github.com/remix-run/react-router/releases)
- [Changelog](https://github.com/remix-run/react-router/blob/main/packages/react-router-dom/CHANGELOG.md)
- [Commits](https://github.com/remix-run/react-router/commits/[email protected]/packages/react-router-dom)

Updates `@netlify/vite-plugin` from 2.7.6 to 2.7.8
- [Release notes](https://github.com/netlify/primitives/releases)
- [Changelog](https://github.com/netlify/primitives/blob/main/release-please-config.json)
- [Commits](netlify/primitives@vite-plugin-v2.7.6...vite-plugin-v2.7.8)

---
updated-dependencies:
- dependency-name: "@types/node"
  dependency-version: 24.9.2
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: security
- dependency-name: "@types/node"
  dependency-version: 24.9.2
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: security
- dependency-name: react-router-dom
  dependency-version: 7.9.5
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: security
- dependency-name: "@netlify/vite-plugin"
  dependency-version: 2.7.8
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: security
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
build(deps-dev): bump eslint from 9.38.0 to 9.39.0 in /web (#3146)
6bc166f 
Bumps [eslint](https://github.com/eslint/eslint) from 9.38.0 to 9.39.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Commits](eslint/eslint@v9.38.0...v9.39.0)

---
updated-dependencies:
- dependency-name: eslint
  dependency-version: 9.39.0
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
build(deps-dev): bump @eslint/js from 9.38.0 to 9.39.0 in /web (#3148)
943e3bf 
Bumps [@eslint/js](https://github.com/eslint/eslint/tree/HEAD/packages/js) from 9.38.0 to 9.39.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Commits](https://github.com/eslint/eslint/commits/v9.39.0/packages/js)

---
updated-dependencies:
- dependency-name: "@eslint/js"
  dependency-version: 9.39.0
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
build(deps-dev): bump jsdom from 27.0.1 to 27.1.0 in /web (#3149)
ea38bb1 
Bumps [jsdom](https://github.com/jsdom/jsdom) from 27.0.1 to 27.1.0.
- [Release notes](https://github.com/jsdom/jsdom/releases)
- [Changelog](https://github.com/jsdom/jsdom/blob/main/Changelog.md)
- [Commits](jsdom/jsdom@27.0.1...27.1.0)

---
updated-dependencies:
- dependency-name: jsdom
  dependency-version: 27.1.0
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
build(deps): bump lucide-react from 0.548.0 to 0.552.0 in /web (#3150)
a4d538f 
Bumps [lucide-react](https://github.com/lucide-icons/lucide/tree/HEAD/packages/lucide-react) from 0.548.0 to 0.552.0.
- [Release notes](https://github.com/lucide-icons/lucide/releases)
- [Commits](https://github.com/lucide-icons/lucide/commits/0.552.0/packages/lucide-react)

---
updated-dependencies:
- dependency-name: lucide-react
  dependency-version: 0.552.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
sgalsaleh and others added 11 commits November 1, 2025 09:04
@sgalsaleh
Remove duplicate unnecessary tests - second pass (#3144)
9dcce8e
@banjoh
Set collectDelay to 15s for goldpinger support bundle
df7e939 
Signed-off-by: Evans Mungai <[email protected]>
@dependabot
build(deps): bump github.com/onsi/ginkgo/v2 from 2.26.0 to 2.27.2 (#3152
ab2670a 
)

Bumps [github.com/onsi/ginkgo/v2](https://github.com/onsi/ginkgo) from 2.26.0 to 2.27.2.
- [Release notes](https://github.com/onsi/ginkgo/releases)
- [Changelog](https://github.com/onsi/ginkgo/blob/master/CHANGELOG.md)
- [Commits](onsi/ginkgo@v2.26.0...v2.27.2)

---
updated-dependencies:
- dependency-name: github.com/onsi/ginkgo/v2
  dependency-version: 2.27.2
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@banjoh
Fix goldpinger namespace in cluster-config.yaml files
617a253 
Signed-off-by: Evans Mungai <[email protected]>
@banjoh
SC-130842: Fix support bundle goldpinger namespace
bb5ca93 
Signed-off-by: Evans Mungai <[email protected]>
@banjoh
Empty commit to trigger build
b652695 
Signed-off-by: Evans Mungai <[email protected]>
@banjoh
Empty commit
ffdcf03 
Signed-off-by: Evans Mungai <[email protected]>
@banjoh
Add namespace to goldpinger pod pinging goldpinger
e0b3beb 
Signed-off-by: Evans Mungai <[email protected]>
@banjoh
Set collectDelay to 15s for goldpinger support bundle
f95ac79 
Signed-off-by: Evans Mungai <[email protected]>
@banjoh
Fix goldpinger namespace in cluster-config.yaml files
70c8cdd 
Signed-off-by: Evans Mungai <[email protected]>
@banjoh
Merge branch 'evansmungai/sc-130842/support-bundle-goldpinger-pods-sc…
6800933 
…heduled' of github.com:replicatedhq/embedded-cluster into evansmungai/sc-130842/support-bundle-goldpinger-pods-scheduled
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

bug::normal type::bug

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@banjoh @replicated-ci-ec @JGAntunes @sgalsaleh @emosbaugh @screspod @NoaheCampbell