| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| |
| |
| |
| | |
src/script: extract mypy config to mypy.ini
Reviewed-by: Ernesto Puerta <epuertat@redhat.com>
Reviewed-by: Sebastian Wagner <sebastian.wagner@suse.com>
|
| |
| |
| |
| |
| |
| | |
Now the default config can be read by other scripts/tools.
Signed-off-by: Alfonso MartÃnez <almartin@redhat.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Add a new flag, --prepare, that when used will fetch all necessary info and
cherrypick all original commits automatically.
It will also prevent the main code to be executed.
After running with --prepare, and if there were no conflicts, you can simply
remove the flag from the command and run the script again.
This time it will run the main code and create the PR and update the backport
issue.
p.e.:
ceph-backport.sh 19206 jewel --prepare
ceph-backport.sh 19206 jewel
Signed-off-by: Tiago Melo <tmelo@suse.com>
|
|\ \
| |/
|/|
| |
| |
| | |
script/run_mypy: Sort groups
Reviewed-by: Alfonso MartÃnez <almartin@redhat.com>
Reviewed-by: Patrick Nawracay <pnawracay@suse.com>
|
| |
| |
| |
| |
| |
| | |
Workaround for https://github.com/python/mypy/issues/2144
Signed-off-by: Sebastian Wagner <sebastian.wagner@suse.com>
|
|/
|
|
|
|
| |
Also use the latest master as default FROM
Signed-off-by: Sebastian Wagner <sebastian.wagner@suse.com>
|
|
|
|
| |
Signed-off-by: Nathan Cutler <ncutler@suse.com>
|
|\
| |
| |
| |
| |
| |
| | |
script/backport-create-issue: handle long Redmine issue names
Reviewed-by: Casey Bodley <cbodley@redhat.com>
Reviewed-by: Kefu Chai <kchai@redhat.com>
Reviewed-by: Abhishek Lekshmanan <abhishek@suse.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Redmine issue names have a maximum length of 255 characters. When we prepend,
e.g., "nautilus: " to a very long issue name, we can end up with a string that
is over 255 characters long. Such a string will be refused by Redmine:
ValidationError: Subject is too long (maximum is 255 characters)
This patch avoids such an eventuality by truncating the post-prepend string to
255 characters.
Signed-off-by: Nathan Cutler <ncutler@suse.com>
|
|\ \
| |/
|/|
| |
| | |
scripts: ceph_dump_log.py
Reviewed-by: Josh Durgin <jdurgin@redhat.com>
|
| |
| |
| |
| | |
Signed-off-by: Brad Hubbard <bhubbard@redhat.com>
|
| |
| |
| |
| |
| |
| |
| | |
Replace the hard-coded list of milestones with an API request
that obtains this data from github directly.
Signed-off-by: Lenz Grimmer <lgrimmer@suse.com>
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
* refs/pull/27281/head:
script/ceph-release-notes: alternate merge commit format
Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
Reviewed-by: Abhishek Lekshmanan <abhishek.lekshmanan@gmail.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
When PRs are merged via GitHub web GUI, the merge commit looks like
this:
Merge pull request #27161 from cbodley/wip-qa-rgw-nautilus
however, nowadays PRs are also getting merged via the "ptl-tool.py" CLI
tool, which generates merge commits that look like this:
Merge PR #27139 into nautilus
This commit adapts the regex so it matches both.
Signed-off-by: Nathan Cutler <ncutler@suse.com>
|
|/ /
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Unfortunately, kubejacker script is not widely used.
In the meantime, simplify it to my needs, as I don't need to push new binaries right now.
Also: updated to new `ceph/ceph` base image
Fixes: http://tracker.ceph.com/issues/39065
Signed-off-by: Sebastian Wagner <sebastian.wagner@suse.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
before this change, the script always push to "origin", in my case,
"origin" points to "ceph/ceph", while we need to create the branch in
private repo. and the name of private repo might vary. so would be great
if we can customize this setting.
Signed-off-by: Kefu Chai <kchai@redhat.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* no need for checking negative numbers
* use regex for matching the number instead of using bash's pattern
matching, otherwise i have
./ceph-backport.sh: line 79: syntax error in conditional expression:
unexpected token `('
the pattern matching works fine if i use it in command line, but
it fails to work in shell script.
Signed-off-by: Kefu Chai <kchai@redhat.com>
|
| |
| |
| |
| |
| | |
Co-Authored-by: Patrick Nawracay <pnawracay@suse.com>
Signed-off-by: Sebastian Wagner <sebastian.wagner@suse.com>
|
| |
| |
| |
| | |
Signed-off-by: Nathan Cutler <ncutler@suse.com>
|
| |
| |
| |
| | |
Signed-off-by: Nathan Cutler <ncutler@suse.com>
|
| |
| |
| |
| | |
Signed-off-by: Nathan Cutler <ncutler@suse.com>
|
| |
| |
| |
| | |
Signed-off-by: Sebastian Wagner <sebastian.wagner@suse.com>
|
| |
| |
| |
| |
| |
| | |
Make ptl-tool.py work from a subdirectory inside the repo (e.g., build/).
Signed-off-by: Sage Weil <sage@redhat.com>
|
| |
| |
| |
| | |
Signed-off-by: Alfredo Deza <adeza@redhat.com>
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
* refs/pull/25020/head:
script/ceph-backport.sh: adjust to change in github API
Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This curl command was working fine before, but recently it started
throwing the following error:
+ curl --silent --data-binary
'{"milestone":"$milestone_number","assignee":"smithfarm","labels":["bug
fix","core"]}'
'https://api.github.com/repos/ceph/ceph/issues/25018?access_token=8ecba7355a11fab00365e44fece350bf01388553'
{
"message": "Invalid request.\n\nFor 'properties/milestone',
\"$milestone_number\" is not an integer or null.",
"documentation_url": "https://developer.github.com/v3"
}
Signed-off-by: Nathan Cutler <ncutler@suse.com>
|
|/ /
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
package-lock.json: This file is modified automatically by npm any time we update
a frontend package, and can amount to large number of line changes.
For example, updating around 10 packages can result in 3k lines changed
in package-lock.json.
.xlf: They are used for the I18M of the dashboard and are automatically created
by Angular/transifex. Curretly each has around 5k lines and those lines should
not be credited to the commiter.
Signed-off-by: Tiago Melo <tmelo@suse.com>
|
|\ \
| | |
| | |
| | |
| | | |
script/kubejacker: include cls libs
Reviewed-by: Sage Weil <sage@redhat.com>
|
| | |
| | |
| | |
| | |
| | |
| | | |
...so that rbd, rgw etc will work.
Signed-off-by: John Spray <john.spray@redhat.com>
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* refs/pull/24596/head:
ptl-tool.py: move githubmap update into merge commit
Reviewed-by: Sage Weil <sage@redhat.com>
Reviewed-by: Neha Ojha <nojha@redhat.com>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Otherwise Github will reject the push because one commit is not part of the PR
(and "untested").
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
|
|/ / /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
* `Exception.message` is deprecated.
* Use relative imports.
* `filter` returns an iterator in Py3
* `urlparse` moved.
* `s/log.warn/log.warning`
* Dockerfile: install Py3 versions.
Signed-off-by: Sebastian Wagner <sebastian.wagner@suse.com>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Nathan Cutler <ncutler@suse.com>
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* refs/pull/24207/head:
script/ptl-tool.py: fix BASE_PATH
Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
|
| | | |
| | | |
| | | |
| | | | |
Signed-off-by: Neha Ojha <nojha@redhat.com>
|
|/ / /
| | |
| | |
| | | |
Signed-off-by: Kefu Chai <kchai@redhat.com>
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
script/kubejacker: Add openSUSE based images
Reviewed-by: John Spray <john.spray@redhat.com>
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
* Added executable bit of `kubejacker.sh`.
* Reduced the number of `RUN` commands
* Added zypper commands for SUSE.
Signed-off-by: Sebastian Wagner <sebastian.wagner@suse.com>
|
|/ /
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Make the script optionally take a comma-separated list of issue numbers.
(Could be just one issue.)
Before this patch, backport-create-issue script insisted on looping over all
issues in Pending Backport status. This was cumbersome in cases when only
one issue (or a couple issues) needed to be processed.
Signed-off-by: Nathan Cutler <ncutler@suse.com>
|
| |
| |
| |
| |
| |
| |
| |
| | |
For those with running k8s/Rook environments, that want
to inject newly built binaries from their development
tree into their kubernetes environment.
Signed-off-by: John Spray <john.spray@redhat.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Once the local backport git branch has been prepared (i.e. after cherry-picking
stage), this script automates the process of opening the backport PR and
updating the backport tracker issue.
Parts of this script were written by Loic Dachary.
Signed-off-by: Nathan Cutler <ncutler@suse.com>
|
| |
| |
| |
| |
| |
| | |
Fixes: http://tracker.ceph.com/issues/24870
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
|
|\ \
| | |
| | |
| | |
| | | |
scripts: backport-create-issue: complain about duplicates and support mimic
Reviewed-by: Kefu Chai <kchai@redhat.com>
|
| | |
| | |
| | |
| | |
| | | |
Fixes: http://tracker.ceph.com/issues/24071
Signed-off-by: Nathan Cutler <ncutler@suse.com>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Nathan Cutler <ncutler@suse.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The update_relations() function call does not care about the return value, so
eliminate the code that was trying to make it return True or False. Instead,
make update_relations() explicitly return None.
Signed-off-by: Nathan Cutler <ncutler@suse.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
It can happen that duplicate backport issues get created. As long
as they are properly linked (via "copied" relation) to the mother
bug, this patch will report them when the script is run.
Signed-off-by: Nathan Cutler <ncutler@suse.com>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
|
| | |
| | |
| | |
| | |
| | |
| | | |
Based on ceph-disk, most probably unused
Signed-off-by: Alfredo Deza <adeza@redhat.com>
|
|/ /
| |
| |
| | |
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
|