Project

General

Profile

Actions

Bug #1536

closed

The promises are not correctly reloaded from the git when the references commit doesn't exist anymore

Added by Nicolas CHARLES over 13 years ago. Updated over 12 years ago.

Status:
Released
Priority:
4
Category:
Web - Config management
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:

Description

To reproduce :
- make a commit
- have this commit loaded in rudder
- destroy this revision, and create a new one

Rudder won't load the new revision

Hint : GitRevisionProvider

Actions #1

Updated by Jonathan CLARKE over 13 years ago

  • Target version changed from 16 to 19
Actions #2

Updated by Jonathan CLARKE about 13 years ago

  • Target version changed from 19 to 21
Actions #3

Updated by Jonathan CLARKE about 13 years ago

  • Target version changed from 21 to 23
Actions #4

Updated by Jonathan CLARKE about 13 years ago

  • Target version changed from 23 to 2.3.0
Actions #5

Updated by Jonathan CLARKE about 13 years ago

  • Target version changed from 2.3.0 to 18
Actions #6

Updated by Jonathan CLARKE about 13 years ago

Stack trace:

15:43:31.922 [main]            ERROR o.s.web.context.ContextLoader - Context initialization failed
org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'ldapNodeConfigurationMapper' defined in class bootstrap.liftweb.AppConfig: Instantiation of bean failed; nested exception is org.springframework.beans.factory.BeanDefinitionStoreException: Factory method [public com.normation.rudder.repository.ldap.LDAPNodeConfigurationMapper bootstrap.liftweb.AppConfig.ldapNodeConfigurationMapper()] threw exception; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'policyPackageService' defined in class bootstrap.liftweb.AppConfig: Instantiation of bean failed; nested exception is org.springframework.beans.factory.BeanDefinitionStoreException: Factory method [public com.normation.cfclerk.services.impl.PolicyPackageServiceImpl bootstrap.liftweb.AppConfig.policyPackageService()] threw exception; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'policyPackagesReader' defined in class bootstrap.liftweb.AppConfig: Instantiation of bean failed; nested exception is org.springframework.beans.factory.BeanDefinitionStoreException: Factory method [public com.normation.cfclerk.services.PolicyPackagesReader bootstrap.liftweb.AppConfig.policyPackagesReader()] threw exception; nested exception is org.eclipse.jgit.errors.MissingObjectException: Missing tree f3cf458b17cf80e277ece6405e15a28db6131452

This happens on boot, when the local PT library doesn't match the ref read from the LDAP directory

Actions #7

Updated by François ARMAND about 13 years ago

  • Target version changed from 18 to 2.4.0~alpha1
Actions #8

Updated by Jonathan CLARKE almost 13 years ago

  • Target version changed from 2.4.0~alpha1 to 2.4.0~alpha2
Actions #9

Updated by Jonathan CLARKE almost 13 years ago

  • Target version changed from 2.4.0~alpha2 to 2.4.0~alpha3
Actions #10

Updated by Jonathan CLARKE almost 13 years ago

  • Target version changed from 2.4.0~alpha3 to 2.4.0~alpha4
Actions #11

Updated by Jonathan CLARKE almost 13 years ago

  • Priority changed from 3 to 4
Actions #12

Updated by François ARMAND almost 13 years ago

  • Target version changed from 2.4.0~alpha4 to 2.4.0~alpha5
Actions #13

Updated by Jonathan CLARKE almost 13 years ago

  • Target version changed from 2.4.0~alpha5 to 2.4.0~alpha6
Actions #14

Updated by Jonathan CLARKE over 12 years ago

  • Target version changed from 2.4.0~alpha6 to 2.4.0~beta1
Actions #15

Updated by François ARMAND over 12 years ago

  • Status changed from New to In progress
  • Assignee set to François ARMAND

This bug is rather critic, even if the probability to hit it for a "normal" user (i.e, one not playing to much with Rudder LDAP/Git) is low, because the correction implies suppressing LDAP values by hand.

Actions #16

Updated by François ARMAND over 12 years ago

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

Applied in changeset commit:1e72b16f082e998052099a1f5727252c70ab6b39.

Actions #17

Updated by Nicolas CHARLES over 12 years ago

  • Status changed from Pending technical review to Released

This looks valid, thank you Francois

Actions

Also available in: Atom PDF