-
Notifications
You must be signed in to change notification settings - Fork 34
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
Keep updating v5 #106
Open
Michaelvll
wants to merge
14
commits into
master
Choose a base branch
from
keep-updating-v5
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Keep updating v5 #106
Changes from all commits
Commits
Show all changes
14 commits
Select commit
Hold shift + click to select a range
68a044b
Keep updating v5 for catalogs
Michaelvll 5d21c9d
fix latest commit
Michaelvll b14c85e
Add venv
Michaelvll 11c574b
test with ref
Michaelvll 46bd99a
fix
Michaelvll e09d9e0
authorize with github action for GCP
Michaelvll 31dd9f5
Add commit hash
Michaelvll fdc72df
fix commit hash fetcher
Michaelvll 9f6120d
add history for sky
Michaelvll ed3bf1d
cd sky
Michaelvll 0d29879
move sky
Michaelvll 97e6ce4
fix
Michaelvll 989a462
fix versions
Michaelvll 84eb1a2
Address comment
Michaelvll File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,9 @@ | ||
# Mapping between version and SkyPilot commit. The CI will update the catalogs | ||
# for all versions listed here. We keep older versions for backward | ||
# compatibility. | ||
# TODO(zhwu): Remove v5 support after 0.10.0 release. | ||
version_commit: | ||
# This commit should be fixed, as any newer commit will be fetching v6 | ||
# catalogs | ||
v5: 1ed40e3174646ba835423b9308d8d6489f83b6bc | ||
v6: latest | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Should we also add a quick update to README.md for this repo that v6 is now a supported catalog version? Can be another PR too. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Good catch! Updated. |
This file contains 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
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just so I understand: when we update catalog fetchers in our main skypilot repo, this hash would also need to be updated?
If so, we should probably make a note of this in README.md