Project

General

Profile

Actions

Bug #2802

closed

Rudder can't start after a migration if the Rudder Technique Reference Library branch is not the same than the /var/rudder/configuration-repository branch

Added by Nicolas PERRON over 11 years ago. Updated about 9 years ago.

Status:
Released
Priority:
1
Assignee:
Nicolas PERRON
Category:
Packaging
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:

Description

If in rudder-web.properties, the Rudder Technique Reference Library branch is not the same than the /var/rudder/configuration-repository branch, after a migration from 2.3 to 2.4.0~beta3, Rudder is broken.

in rudder-web.properties:

rudder.ptlib.git.refs.path=refs/heads/master

in /var/rudder/configuration-repository :

root@server:/var/rudder/configuration-repository# git branch
* brokenbranch
  master

Rudder can't start with this error message:

[...]
[2012-08-13 14:46:33] INFO  application - Use configuration file defined by JVM property -Drudder.configFile : /opt/rudder/etc/rudder-web.properties
[2012-08-13 14:46:34] INFO  application - Use configuration file defined by JVM property -Drudder.authFile : /opt/rudder/etc/rudder-users.xml
[2012-08-13 14:46:35] ERROR org.springframework.web.context.ContextLoader - Context initialization failed
[...]
Caused by:
java.lang.RuntimeException: Missing techniques root category in Git, expecting category descriptor for Git path: '/var/rudder/configuration-repository/techniques/category.xml'
        at scala.sys.package$.error(package.scala:27)
[...]

Even if I try to reapply rudder-upgrade script to the good branch, Rudder still can't be started with the same error message.


Related issues 2 (0 open2 closed)

Related to Rudder - Bug #2804: In rudder upgrade, the Rudder Technique Reference Library attribute in rudder-wbe.properties should be checked with a more flexbile wayRejectedNicolas PERRON2012-08-13Actions
Related to Rudder - Bug #2805: After a migration from 2.3 to 2.4.0~beta3, if a migration has been made with a wrong git branch on /var/rudder/configuration-repository it is impossible to return in a normal stateRejectedNicolas PERRON2012-08-14Actions
Actions #1

Updated by Nicolas PERRON over 11 years ago

In the migration script, there is clearly a test missing: Do not act on git if we are not the good git branch (Rudder Technique Reference Library branch).

Actions #2

Updated by Nicolas PERRON over 11 years ago

Here are two problems:

  • We are matching only rudder.ptlib.git.refs.path but in rudder 2.4, this attribute is renamed into rudder.techniqueLibrary.git.refs.path
    • Easy: we will check the both of them
  • These attributes can accept a lot of values: (refs/heads/master, refs/tags/mytag, master, mytag, bf46f65cdbba1fc175bd73a74e97c80275ae4339 (commit ID), etc...)
    • We can't use all of these attributes with git checkout for example: refs/heads/master will be in a detached HEAD state contrary to master
    • A first approach will be to remove all "/refs/heads/", "refs/tags/" and "refs/remote/origin/" to use git checkout.
Actions #3

Updated by Nicolas PERRON over 11 years ago

  • Status changed from New to Pending technical review
  • % Done changed from 0 to 100

Nicolas PERRON wrote:

Here are two problems:

  • We are matching only rudder.ptlib.git.refs.path but in rudder 2.4, this attribute is renamed into rudder.techniqueLibrary.git.refs.path
  • Easy: we will check the both of them
  • These attributes can accept a lot of values: (refs/heads/master, refs/tags/mytag, master, mytag, bf46f65cdbba1fc175bd73a74e97c80275ae4339 (commit ID), etc...)
  • We can't use all of these attributes with git checkout for example: refs/heads/master will be in a detached HEAD state contrary to master
  • A first approach will be to remove all "/refs/heads/", "refs/tags/" and "refs/remote/origin/" to use git checkout.

We can consider the last commit as fixing this issue.

A new ticket is open for an enhancement about other conditions than having rudder.ptlib.git.refs.path set to "/refs/heads/*", "refs/tags/*" or "refs/remote/origin/*": #2804

Actions #4

Updated by Matthieu CERDA over 11 years ago

Looks OK, I corrected some comments.

Actions #5

Updated by Jonathan CLARKE over 11 years ago

  • Status changed from Pending technical review to Released

Looks good to me.

Actions #6

Updated by Nicolas PERRON about 11 years ago

  • Project changed from Rudder to 34
  • Category deleted (11)
Actions #7

Updated by Benoît PECCATTE about 9 years ago

  • Project changed from 34 to Rudder
  • Category set to Packaging
Actions

Also available in: Atom PDF