Skip to content
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

fix(multichain-testing): update starship helm chart repo URL #10863

Merged
merged 1 commit into from
Jan 20, 2025

Conversation

mujahidkay
Copy link
Member

@mujahidkay mujahidkay commented Jan 20, 2025

closes: #10862

Description

Cosmology has rebranded to Hyberweb which now leads https://cosmology-tech.github.io/starship to a 404. This PR updates the helm chart repo url for starship to unbreak dapp-orch-basics and agoric-sdk's CI (failing multichain-e2e run). The change is fairly recent - around 2 days ago - so they haven't gotten to update all their docs yet. I stumbled upon the correct URL intuitively - which means I can't provide some docs or issue as proof other than the fact that it works now.
Edit: updated here

Before

 % make setup start         
/Users/mujahid/Documents/GitHub/agoric-sdk/multichain-testing/scripts/dev-setup.sh
All binaries are installed
kind create cluster --name agship
Creating cluster "agship" ...
 ✓ Ensuring node image (kindest/node:v1.32.0) 🖼
 ✓ Preparing nodes 📦  
 ✓ Writing configuration 📜 
 ✓ Starting control-plane 🕹️ 
 ✓ Installing CNI 🔌 
 ✓ Installing StorageClass 💾 
Set kubectl context to "kind-agship"
You can now use your cluster with:

kubectl cluster-info --context kind-agship

Have a nice day! 👋
/Users/mujahid/Documents/GitHub/agoric-sdk/multichain-testing/scripts/install.sh --config config.yaml --name agoric-multichain-testing --version v0.2.20
Script dir: /Users/mujahid/Documents/GitHub/agoric-sdk/multichain-testing/scripts
Error: looks like "https://cosmology-tech.github.io/starship/" is not a valid chart repository or cannot be reached: failed to fetch https://cosmology-tech.github.io/starship/index.yaml : 404 Not Found
make: *** [install] Error 1

After

/Users/mujahid/Documents/GitHub/agoric-sdk/multichain-testing/scripts/dev-setup.sh
All binaries are installed
kind create cluster --name agship
Creating cluster "agship" ...
 ✓ Ensuring node image (kindest/node:v1.32.0) 🖼
 ✓ Preparing nodes 📦  
 ✓ Writing configuration 📜 
 ✓ Starting control-plane 🕹️ 
 ✓ Installing CNI 🔌 
 ✓ Installing StorageClass 💾 
Set kubectl context to "kind-agship"
You can now use your cluster with:

kubectl cluster-info --context kind-agship

Not sure what to do next? 😅  Check out https://kind.sigs.k8s.io/docs/user/quick-start/
/Users/mujahid/Documents/GitHub/agoric-sdk/multichain-testing/scripts/install.sh --config config.yaml --name agoric-multichain-testing --version v0.2.20
Script dir: /Users/mujahid/Documents/GitHub/agoric-sdk/multichain-testing/scripts
"starship" has been added to your repositories
Hang tight while we grab the latest from your chart repositories...

Security Considerations

None

Scaling Considerations

None

Documentation Considerations

None

Testing Considerations

Fixes multichain-e2e tests

Upgrade Considerations

None

@mujahidkay mujahidkay added the force:integration Force integration tests to run on PR label Jan 20, 2025
@mujahidkay mujahidkay self-assigned this Jan 20, 2025
Copy link

cloudflare-workers-and-pages bot commented Jan 20, 2025

Deploying agoric-sdk with  Cloudflare Pages  Cloudflare Pages

Latest commit: 52cd081
Status: ✅  Deploy successful!
Preview URL: https://1503819d.agoric-sdk.pages.dev
Branch Preview URL: https://mk-update-startship-helm-url.agoric-sdk.pages.dev

View logs

@mujahidkay mujahidkay force-pushed the mk/update-startship-helm-url branch from bca9648 to 52cd081 Compare January 20, 2025 11:56
@mujahidkay mujahidkay marked this pull request as ready for review January 20, 2025 12:02
@mujahidkay mujahidkay requested a review from a team as a code owner January 20, 2025 12:02
@mujahidkay mujahidkay added the automerge:rebase Automatically rebase updates, then merge label Jan 20, 2025
@mergify mergify bot merged commit b1998a7 into master Jan 20, 2025
101 of 110 checks passed
@mergify mergify bot deleted the mk/update-startship-helm-url branch January 20, 2025 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge:rebase Automatically rebase updates, then merge force:integration Force integration tests to run on PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

starship cannot be reached - multichain-testing
2 participants