seeweb-updateyml#149
Conversation
| - name: Run fetch_seeweb | ||
| id: fetch_catalogs | ||
| env: | ||
| SEEWEB_API_KEY: ${{ secrets.SEEWEB_API_KEY }} |
There was a problem hiding this comment.
Is the SEEWEB_API_KEY already configured in this repo?
There was a problem hiding this comment.
+1. @massaindustries could I reach out to you on slack?
There was a problem hiding this comment.
For sure, I gave the seeweb api key to Yelei Wu. I'm on the skypilot channel on slack
|
cc @romilbhardwaj for taking a look |
kyuds
left a comment
kyuds
left a comment
There was a problem hiding this comment.
LGTM on my end except for the two comments. Would love to get this merged @massaindustries !
| version: "latest" | ||
| python-version: "3.10" | ||
|
|
||
| - name: Install yq |
There was a problem hiding this comment.
looking at other implementations of the update yml that are working, we don't need to install yq. its already available. refer to aws and gcp catalogs for instance
| - name: Run fetch_seeweb | ||
| id: fetch_catalogs | ||
| env: | ||
| SEEWEB_API_KEY: ${{ secrets.SEEWEB_API_KEY }} |
There was a problem hiding this comment.
+1. @massaindustries could I reach out to you on slack?
massaindustries
force-pushed
the
skypilot-catalog-seeweb-01
branch
from
961769e to
5f61352
Compare
3 participants
ALREADY MERGED
[x] Added update-catalog.yml