-
Notifications
You must be signed in to change notification settings - Fork 514
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 for bigtable.sh - Debian buster repo issue #1164
Fix for bigtable.sh - Debian buster repo issue #1164
Conversation
…/initialization-actions into bigtable-debian-buster-fix
/gcbrun |
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.
LGTM
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.
I think the conditional around remove_old_backports could use a tweak...
Added condition to consider 2.1 Dataproc image version for debian as suggsted by the reviewers
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.
LGTM
/gcbrun |
1 similar comment
/gcbrun |
/gcbrun |
Tests are passing on 2.0 & 2.1. 2.2 failures doesn't seem to be related to this change, merging this. |
0985c2e
into
GoogleCloudDataproc:master
No description provided.