Skip to content

Commit

Permalink
Merge pull request #468 from pohly/prow-update-master
Browse files Browse the repository at this point in the history
master: update release-tools
  • Loading branch information
k8s-ci-robot authored Feb 4, 2021
2 parents 165f6e0 + 096ef49 commit cbe212e
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 8 deletions.
11 changes: 9 additions & 2 deletions release-tools/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -44,10 +44,17 @@ is the recommended way of maintaining a copy of the rules inside the
changes also locally, test them and then push them back to the shared
repository at a later time.

We no longer care about importing the full commit history, so `--squash` should be used
when submitting a `release-tools` update. Also make sure that the PR for that
contains the automatically generated commit message in the PR description.
It contains the list of individual commits that were squashed. The script from
https://github.com/kubernetes-csi/csi-release-tools/issues/7 can create such
PRs automatically.

Cheat sheet:

- `git subtree add --prefix=release-tools https://github.com/kubernetes-csi/csi-release-tools.git master` - add release tools to a repo which does not have them yet (only once)
- `git subtree pull --prefix=release-tools https://github.com/kubernetes-csi/csi-release-tools.git master` - update local copy to latest upstream (whenever upstream changes)
- `git subtree add --squash --prefix=release-tools https://github.com/kubernetes-csi/csi-release-tools.git master` - add release tools to a repo which does not have them yet (only once)
- `git subtree pull --squash --prefix=release-tools https://github.com/kubernetes-csi/csi-release-tools.git master` - update local copy to latest upstream (whenever upstream changes)
- edit, `git commit`, `git subtree push --prefix=release-tools [email protected]:<user>/csi-release-tools.git <my-new-or-existing-branch>` - push to a new branch before submitting a PR

verify-shellcheck.sh
Expand Down
8 changes: 3 additions & 5 deletions release-tools/boilerplate/boilerplate.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,6 @@
import argparse
import difflib
import glob
import json
import mmap
import os
import re
import sys
Expand All @@ -33,12 +31,12 @@
nargs='*')

# Rootdir defaults to the directory **above** the repo-infra dir.
rootdir = os.path.dirname(__file__) + "./../../../"
rootdir = os.path.dirname(__file__) + "./../../"
rootdir = os.path.abspath(rootdir)
parser.add_argument(
"--rootdir", default=rootdir, help="root directory to examine")

default_boilerplate_dir = os.path.join(rootdir, "csi-driver-nfs/hack/boilerplate")
default_boilerplate_dir = os.path.abspath(os.path.dirname(__file__))

parser.add_argument(
"--boilerplate-dir", default=default_boilerplate_dir)
Expand Down Expand Up @@ -199,4 +197,4 @@ def main():
return 0

if __name__ == "__main__":
sys.exit(main())
sys.exit(main())
2 changes: 1 addition & 1 deletion release-tools/prow.sh
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ version_to_git () {
version="$1"
shift
case "$version" in
latest) echo "master";;
latest|master) echo "master";;
release-*) echo "$version";;
*) echo "v$version";;
esac
Expand Down

0 comments on commit cbe212e

Please sign in to comment.