Skip to content
This repository has been archived by the owner on Jan 15, 2019. It is now read-only.

[dev.icinga.com #2682] upgradedb script typo #1003

Closed
icinga-migration opened this issue Jun 13, 2012 · 5 comments
Closed

[dev.icinga.com #2682] upgradedb script typo #1003

icinga-migration opened this issue Jun 13, 2012 · 5 comments

Comments

@icinga-migration
Copy link

This issue has been migrated from Redmine: https://dev.icinga.com/issues/2682

Created by pdeneu on 2012-06-13 08:47:12 +00:00

Assignee: mfriedrich
Status: Resolved (closed on 2012-06-13 13:08:48 +00:00)
Target Version: 1.7.1
Last Update: 2014-12-08 14:46:42 +00:00 (in Redmine)

Icinga Version: 1.10.0
OS Version: any

Hello,

in upgradb script in line 77 are two little bugs.
The line has to look like this:

my $file="../mysql/upgrade/mysql-upgrade-$thisversion.sql";

Kind regrads,
Philipp

Changesets

2012-06-13 12:00:26 +00:00 by mfriedrich 271eb06

idoutils: fix upgradedb script typos & past changes #2682

fixes #2682

2012-06-13 12:03:51 +00:00 by mfriedrich 458d41e

mark the upgrade script "this version" #2682

refs #2682
@icinga-migration
Copy link
Author

Updated by mfriedrich on 2012-06-13 09:46:37 +00:00

  • Category set to 24
  • Status changed from New to Assigned
  • Assigned to set to mfriedrich
  • Priority changed from Normal to Low
  • Target Version set to 1.7.1

thanks. the script furthermore lacks off the changed dbversion table creation plus correct initial inserts with newly provided timestamps.

@icinga-migration
Copy link
Author

Updated by mfriedrich on 2012-06-13 11:53:13 +00:00

btw what do you mean with 2 little bugs? i can only see one.

@icinga-migration
Copy link
Author

Updated by pdeneu on 2012-06-13 12:00:34 +00:00

two in one line :)
first was the path:

- ../mysql/mysql-upgrade...
+ ../mysql/upgrade/mysql-upgrade...

second the variable of new version:

- $version
+ $thisversion

@icinga-migration
Copy link
Author

Updated by mfriedrich on 2012-06-13 13:08:48 +00:00

  • Status changed from Assigned to Resolved
  • Done % changed from 0 to 100

Applied in changeset 271eb06.

@icinga-migration
Copy link
Author

Updated by mfriedrich on 2014-12-08 14:46:42 +00:00

  • Project changed from 18 to Core, Classic UI, IDOUtils
  • Category changed from 24 to IDOUtils
  • Icinga Version set to 1
  • OS Version set to any

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant