ubuntu-bots team mailing list archive

[Martin Packman <martin.packman@xxxxxxxxxxxxx>]

Martin Packman has proposed merging lp:~gz/ubuntu-bots/oops_link_1051902 into lp:ubuntu-bots.

Requested reviews:
  Ubuntu IRC Bots (ubuntu-bots)
Related bugs:
  Bug #1051902 in Ubuntu IRC Bots: "OOPS responses do not link to current oops.canonical.com site"
  https://bugs.launchpad.net/ubuntu-bots/+bug/1051902

For more details, see:
https://code.launchpad.net/~gz/ubuntu-bots/oops_link_1051902/+merge/124648

Since OOPS has been picked up by services other than launchpad, a new site for looking them up was created. The old launchpad specific site still exists for now but new oopses only go to the new site, so the bot should link there instead. This branch switches the address to the new form.

How is deployment handled? Do I need to some extra steps after this change is merged to get the new code live on the ubot5 in #launchpad etc?
-- 
https://code.launchpad.net/~gz/ubuntu-bots/oops_link_1051902/+merge/124648
Your team Ubuntu IRC Bots is requested to review the proposed merge of lp:~gz/ubuntu-bots/oops_link_1051902 into lp:ubuntu-bots.

=== modified file 'Bugtracker/plugin.py'
--- Bugtracker/plugin.py	2012-07-04 13:21:37 +0000
+++ Bugtracker/plugin.py	2012-09-17 10:57:26 +0000
@@ -450,7 +450,7 @@
             return
         if not self.is_ok(channel, 'lpoops', oopsid):
             return
-        irc.reply('https://lp-oops.canonical.com/oops.py/?oopsid=' + oopsid, prefixNick=False)
+        irc.reply('https://oops.canonical.com/?oopsid=' + oopsid, prefixNick=False)
 
     def cveSnarfer(self, irc, msg, match):
         r"(cve[- ]\d{4}[- ]\d{4})"