opendev/elastic-recheck
Code Issues Proposed changes

Switch gerrit URL to review.opendev.org

Browse Source 
As of the great renaming of 2019 we need to update the
openstack gerrit URL default to review.opendev.org.

Change-Id: I2e3f7e7fb03be0deba0c95995265376dbce3c5b6
Story: #2005498
Task: #30599
This commit is contained in:
Matt Riedemann 2019-04-22 11:50:04 -04:00
parent 754fdeae04
commit fbdbbc0f40
7 changed files with 24 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
elasticRecheck.conf.sample
View File

@ -14,7 +14,7 @@ ci_username=jenkins
[gerrit] [gerrit]
user=treinish user=treinish
host=review.openstack.org host=review.opendev.org
query_file=/home/mtreinish/elasticRecheck/queries query_file=/home/mtreinish/elasticRecheck/queries
key=/home/mtreinish/.ssh/id_rsa key=/home/mtreinish/.ssh/id_rsa

6
elastic_recheck/cmd/graph.py
View File

@ -76,9 +76,9 @@ def get_launchpad_bug(bug):
def get_open_reviews(bug_number, attempt=0): def get_open_reviews(bug_number, attempt=0):
"""return list of open gerrit reviews for a given bug.""" """return list of open gerrit reviews for a given bug."""
r = requests.get("https://review.openstack.org:443/changes/" r = requests.get("https://review.opendev.org:443/changes/"
"?q=status:open++message:`%s`+NOT+" "?q=status:open++message:`%s`+NOT+"
"project:openstack-infra/elastic-recheck" % bug_number) "project:opendev/elastic-recheck" % bug_number)
# If we got a proxy error let's retry until we're out of attempts. # If we got a proxy error let's retry until we're out of attempts.
if r.status_code == 502 and attempt < 3: if r.status_code == 502 and attempt < 3:
LOG.debug('Retry changes query for bug %s. Attempt %s of 3.', LOG.debug('Retry changes query for bug %s. Attempt %s of 3.',
@ -87,7 +87,7 @@ def get_open_reviews(bug_number, attempt=0):
# strip off first few chars because 'the JSON response body starts with a # strip off first few chars because 'the JSON response body starts with a
# magic prefix line that must be stripped before feeding the rest of the # magic prefix line that must be stripped before feeding the rest of the
# response body to a JSON parser' # response body to a JSON parser'
# https://review.openstack.org/Documentation/rest-api.html # https://review.opendev.org/Documentation/rest-api.html
reviews = [] reviews = []
result = None result = None
try: try:

2
elastic_recheck/config.py
View File

@ -26,7 +26,7 @@ JOBS_RE = 'dsvm'
CI_USERNAME = 'jenkins' CI_USERNAME = 'jenkins'
GERRIT_QUERY_FILE = 'queries' GERRIT_QUERY_FILE = 'queries'
GERRIT_HOST = 'review.openstack.org' GERRIT_HOST = 'review.opendev.org'
PID_FN = '/var/run/elastic-recheck/elastic-recheck.pid' PID_FN = '/var/run/elastic-recheck/elastic-recheck.pid'

22
elastic_recheck/tests/unit/gerrit/events.json
View File

@ -27,7 +27,7 @@
"eventCreatedOn": 1389371115, "eventCreatedOn": 1389371115,
"change": { "change": {
"topic": "bp\/mapping", "topic": "bp\/mapping",
"url": "https:\/\/review.openstack.org\/60244", "url": "https:\/\/review.opendev.org\/60244",
"number": "60244", "number": "60244",
"project": "openstack\/keystone", "project": "openstack\/keystone",
"branch": "master", "branch": "master",
@ -68,7 +68,7 @@
"eventCreatedOn": 1389384794, "eventCreatedOn": 1389384794,
"change": { "change": {
"topic": "65361", "topic": "65361",
"url": "https:\/\/review.openstack.org\/65361", "url": "https:\/\/review.opendev.org\/65361",
"number": "65361", "number": "65361",
"project": "openstack\/requirements", "project": "openstack\/requirements",
"branch": "master", "branch": "master",
@ -109,7 +109,7 @@
"eventCreatedOn": 1389383805, "eventCreatedOn": 1389383805,
"change": { "change": {
"topic": "bp\/image-flavor-selection-ux-enhancement", "topic": "bp\/image-flavor-selection-ux-enhancement",
"url": "https:\/\/review.openstack.org\/63078", "url": "https:\/\/review.opendev.org\/63078",
"number": "63078", "number": "63078",
"project": "openstack\/horizon", "project": "openstack\/horizon",
"branch": "master", "branch": "master",
@ -150,7 +150,7 @@
"eventCreatedOn": 1389306576, "eventCreatedOn": 1389306576,
"change": { "change": {
"topic": "slow-down-there-cowboy", "topic": "slow-down-there-cowboy",
"url": "https:\/\/review.openstack.org\/65805", "url": "https:\/\/review.opendev.org\/65805",
"number": "65805", "number": "65805",
"project": "openstack-infra\/devstack-gate", "project": "openstack-infra\/devstack-gate",
"branch": "master", "branch": "master",
@ -191,7 +191,7 @@
"eventCreatedOn": 1389391347, "eventCreatedOn": 1389391347,
"change": { "change": {
"topic": "1187119-a", "topic": "1187119-a",
"url": "https:\/\/review.openstack.org\/65378", "url": "https:\/\/review.opendev.org\/65378",
"number": "65378", "number": "65378",
"project": "openstack\/api-site", "project": "openstack\/api-site",
"branch": "master", "branch": "master",
@ -232,7 +232,7 @@
"eventCreatedOn": 1389378289, "eventCreatedOn": 1389378289,
"change": { "change": {
"topic": "bug\/1265108", "topic": "bug\/1265108",
"url": "https:\/\/review.openstack.org\/64749", "url": "https:\/\/review.opendev.org\/64749",
"number": "64749", "number": "64749",
"project": "openstack\/keystone", "project": "openstack\/keystone",
"branch": "master", "branch": "master",
@ -273,7 +273,7 @@
"eventCreatedOn": 1389378289, "eventCreatedOn": 1389378289,
"change": { "change": {
"topic": "bug\/1265108", "topic": "bug\/1265108",
"url": "https:\/\/review.openstack.org\/64750", "url": "https:\/\/review.opendev.org\/64750",
"number": "64750", "number": "64750",
"project": "openstack\/keystone", "project": "openstack\/keystone",
"branch": "master", "branch": "master",
@ -314,7 +314,7 @@
"eventCreatedOn": 1389387264, "eventCreatedOn": 1389387264,
"change": { "change": {
"topic": "bugs\/1267682", "topic": "bugs\/1267682",
"url": "https:\/\/review.openstack.org\/65902", "url": "https:\/\/review.opendev.org\/65902",
"number": "65902", "number": "65902",
"project": "openstack\/neutron", "project": "openstack\/neutron",
"branch": "master", "branch": "master",
@ -355,7 +355,7 @@
"eventCreatedOn": 1389388172, "eventCreatedOn": 1389388172,
"change": { "change": {
"topic": "bug\/1253497", "topic": "bug\/1253497",
"url": "https:\/\/review.openstack.org\/66064", "url": "https:\/\/review.opendev.org\/66064",
"number": "66064", "number": "66064",
"project": "openstack\/glance", "project": "openstack\/glance",
"branch": "master", "branch": "master",
@ -396,7 +396,7 @@
"eventCreatedOn": 1389354908, "eventCreatedOn": 1389354908,
"change": { "change": {
"topic": "del-param", "topic": "del-param",
"url": "https:\/\/review.openstack.org\/65925", "url": "https:\/\/review.opendev.org\/65925",
"number": "65925", "number": "65925",
"project": "openstack\/savanna", "project": "openstack\/savanna",
"branch": "master", "branch": "master",
@ -437,7 +437,7 @@
"eventCreatedOn": 1387806669, "eventCreatedOn": 1387806669,
"change": { "change": {
"topic": "bug\/1259078", "topic": "bug\/1259078",
"url": "https:\/\/review.openstack.org\/61074", "url": "https:\/\/review.opendev.org\/61074",
"number": "61074", "number": "61074",
"project": "openstack\/heat", "project": "openstack\/heat",
"branch": "master", "branch": "master",

2
elastic_recheck/tests/unit/test_bot.py
View File

@ -111,7 +111,7 @@ class TestBotWithTestTools(tests.TestCase):
commenting=False) commenting=False)
def fake_print(self, cls, channel, msg): def fake_print(self, cls, channel, msg):
reference = ("openstack/keystone change: https://review.openstack.org/" reference = ("openstack/keystone change: https://review.opendev.org/"
"64750 failed because of: " "64750 failed because of: "
"gate-keystone-python26: " "gate-keystone-python26: "
"https://bugs.launchpad.net/bugs/123456, " "https://bugs.launchpad.net/bugs/123456, "

12
elastic_recheck/tests/unit/test_stream.py
View File

@ -44,7 +44,7 @@ class TestStream(tests.TestCase):
self.assertEqual(event.rev, 6) self.assertEqual(event.rev, 6)
self.assertEqual(event.project, "openstack/keystone") self.assertEqual(event.project, "openstack/keystone")
self.assertEqual(event.name(), "64750,6") self.assertEqual(event.name(), "64750,6")
self.assertEqual(event.url, "https://review.openstack.org/64750") self.assertEqual(event.url, "https://review.opendev.org/64750")
self.assertEqual(sorted(event.build_short_uuids()), self.assertEqual(sorted(event.build_short_uuids()),
["5dd41fe", "d3fd328"]) ["5dd41fe", "d3fd328"])
self.assertTrue(event.is_included_job()) self.assertTrue(event.is_included_job())
@ -62,7 +62,7 @@ class TestStream(tests.TestCase):
self.assertEqual(event.rev, 6) self.assertEqual(event.rev, 6)
self.assertEqual(event.project, "openstack/keystone") self.assertEqual(event.project, "openstack/keystone")
self.assertEqual(event.name(), "64749,6") self.assertEqual(event.name(), "64749,6")
self.assertEqual(event.url, "https://review.openstack.org/64749") self.assertEqual(event.url, "https://review.opendev.org/64749")
self.assertEqual(sorted(event.build_short_uuids()), self.assertEqual(sorted(event.build_short_uuids()),
["5dd41fe", "d3fd328"]) ["5dd41fe", "d3fd328"])
self.assertTrue(event.is_included_job()) self.assertTrue(event.is_included_job())
@ -80,7 +80,7 @@ class TestStream(tests.TestCase):
self.assertEqual(event.rev, 19) self.assertEqual(event.rev, 19)
self.assertEqual(event.project, "openstack/horizon") self.assertEqual(event.project, "openstack/horizon")
self.assertEqual(event.name(), "63078,19") self.assertEqual(event.name(), "63078,19")
self.assertEqual(event.url, "https://review.openstack.org/63078") self.assertEqual(event.url, "https://review.opendev.org/63078")
self.assertEqual(event.build_short_uuids(), ["ab07162"]) self.assertEqual(event.build_short_uuids(), ["ab07162"])
event = stream.get_failed_tempest() event = stream.get_failed_tempest()
@ -88,7 +88,7 @@ class TestStream(tests.TestCase):
self.assertEqual(event.rev, 2) self.assertEqual(event.rev, 2)
self.assertEqual(event.project, "openstack/requirements") self.assertEqual(event.project, "openstack/requirements")
self.assertEqual(event.name(), "65361,2") self.assertEqual(event.name(), "65361,2")
self.assertEqual(event.url, "https://review.openstack.org/65361") self.assertEqual(event.url, "https://review.opendev.org/65361")
self.assertEqual(event.build_short_uuids(), ["8209fb4"]) self.assertEqual(event.build_short_uuids(), ["8209fb4"])
self.assertRaises( self.assertRaises(
@ -144,7 +144,7 @@ class TestStream(tests.TestCase):
self.assertEqual(event.rev, 6) self.assertEqual(event.rev, 6)
self.assertEqual(event.project, "openstack/keystone") self.assertEqual(event.project, "openstack/keystone")
self.assertEqual(event.name(), "64750,6") self.assertEqual(event.name(), "64750,6")
self.assertEqual(event.url, "https://review.openstack.org/64750") self.assertEqual(event.url, "https://review.opendev.org/64750")
self.assertEqual(sorted(event.build_short_uuids()), self.assertEqual(sorted(event.build_short_uuids()),
["5dd41fe", "d3fd328"]) ["5dd41fe", "d3fd328"])
self.assertTrue(event.is_included_job()) self.assertTrue(event.is_included_job())
@ -172,7 +172,7 @@ class TestStream(tests.TestCase):
self.assertEqual(event.rev, 6) self.assertEqual(event.rev, 6)
self.assertEqual(event.project, "openstack/keystone") self.assertEqual(event.project, "openstack/keystone")
self.assertEqual(event.name(), "64749,6") self.assertEqual(event.name(), "64749,6")
self.assertEqual(event.url, "https://review.openstack.org/64749") self.assertEqual(event.url, "https://review.opendev.org/64749")
self.assertEqual(sorted(event.build_short_uuids()), self.assertEqual(sorted(event.build_short_uuids()),
["5dd41fe", "d3fd328"]) ["5dd41fe", "d3fd328"])
self.assertTrue(event.is_included_job()) self.assertTrue(event.is_included_job())

2
tools/unaccounted_rechecks.py
View File

@ -45,7 +45,7 @@ def get_options():
def connect_to_gerrit(user, key): def connect_to_gerrit(user, key):
return gerrit.Gerrit('review.openstack.org', user, 29418, key) return gerrit.Gerrit('review.opendev.org', user, 29418, key)
def collect_rechecks(gerrit, days="14"): def collect_rechecks(gerrit, days="14"):