Francois Armand fanf

Organizations

@Normation @LDAPGTF
fanf pushed to master at Normation/ncf
@fanf
fanf merged pull request Normation/ncf#202
@fanf
Fixes #6618: change the name of techniques category to \"User Techniques\" instead of \"Meta Techniques\"
1 commit with 2 additions and 2 deletions
fanf commented on pull request Normation/ncf#202
@fanf

It should target branch 3.1

fanf opened pull request Normation/rudder#865
@fanf
Fixes #6663: Compliance in dashboard and rule page are not consistant
1 commit with 27 additions and 5 deletions
@fanf
  • @fanf a56b447
    Merge branch 'branches/rudder/3.0' into branches/rudder/3.1
  • @fanf 9ad244c
    Merge branch 'branches/rudder/2.11' into branches/rudder/3.0
  • 2 more commits »
@fanf
  • @fanf a56b447
    Merge branch 'branches/rudder/3.0' into branches/rudder/3.1
  • @fanf 9ad244c
    Merge branch 'branches/rudder/2.11' into branches/rudder/3.0
  • 2 more commits »
@fanf
  • @fanf 9ad244c
    Merge branch 'branches/rudder/2.11' into branches/rudder/3.0
  • @ncharles 0146626
    Merge pull request #864 from fanf/bug_6519_bad_expected_report_for_ad…
  • 1 more commit »
fanf commented on pull request Normation/rudder#863
@fanf

Appart for the formatting details, it's OK with me, you can self-merge after the udpate

fanf commented on pull request Normation/rudder#863
@fanf

Could you use our standard formatting here ? if(needDeployment) { ... } ?

@fanf
  • @VinceMacBuche c68240d
    Fixes #6619: When only adding directives to Rules, reporting is not d…
@fanf
  • @VinceMacBuche c6100ba
    Fixes #6619: When only adding directives to Rules, reporting is not d…
fanf commented on pull request Normation/rudder#864
@fanf

It's the diff on two set to get what directives changed - ha, and actually I could use --, it could be clearer than &~

fanf opened pull request Normation/rudder#864
@fanf
Fixes #6619: When only adding directives to Rules, reporting is not displayed for added directives
1 commit with 85 additions and 50 deletions
@fanf
  • @VinceMacBuche c00c20a
    Fixes #6619: When only adding directives to Rules, reporting is not d…
fanf commented on pull request Normation/rudder#862
@fanf

This PR does not solve the previous problem of updating serial on rule that are not actually changed. I'm closing it to propose an alternative sol…

@fanf
fanf merged pull request Normation/rudder#860
@fanf
Fixes #6616: Do not check dns when checking promises
1 commit with 11 additions and 9 deletions
@fanf
fanf merged pull request Normation/rudder#859
@fanf
Fixes #6601: Remove invalid default public key for root server
1 commit with 0 additions and 1 deletion
@fanf
fanf merged pull request Normation/ldap-inventory#65
@fanf
Fixes #6600: Cannot modify root server inventory after new install
1 commit with 46 additions and 18 deletions
fanf commented on pull request Normation/ldap-inventory#65
@fanf

at the end ==> where the method was used

fanf commented on pull request Normation/ldap-inventory#65
@fanf

You don't use keystatus at all in that method, so you are introducing a non needed coupling here. Could you remove it, and name the method "extrac…

fanf commented on pull request Normation/ldap-inventory#65
@fanf

is undefined

@fanf
fanf merged pull request Normation/rudder-doc#102
@fanf
Fixes #6559: fix typo in documentation - use proper date format for b…
1 commit with 5 additions and 5 deletions
fanf commented on pull request Normation/rudder#859
@fanf

Depending of the way #6000 if fiexed, that one may not be necessary at all