• Joined on 2021-11-20
ungleich-gitea commented on issue ungleich-public/cdist#147 2021-11-20 15:43:27 +00:00
__ssh_authorized_keys overwrites existing keys

Created by: darko-poljak

@pcoder Can you give an example?

  1. authorized_keys file content before running cdist
  2. init manifest content
  3. authroized_keys file content after running cdist 4.…
ungleich-gitea commented on issue ungleich-public/cdist#147 2021-11-20 15:43:26 +00:00
__ssh_authorized_keys overwrites existing keys

Created by: pcoder

@darko-poljak

  1. authorized_keys file content could be containing any number of keys. For instance lets take this, a randomly chosen public key
ssh-rsa…
ungleich-gitea commented on issue ungleich-public/cdist#147 2021-11-20 15:43:25 +00:00
__ssh_authorized_keys overwrites existing keys

Created by: pcoder

Whats worse is that if we have many keys in authorized_keys file, all of them are overwritten with only testtesttestsetestststestsetest

ungleich-gitea commented on issue ungleich-public/cdist#147 2021-11-20 15:43:23 +00:00
__ssh_authorized_keys overwrites existing keys

Created by: darko-poljak

@pcoder I cannot reproduce a case where key is valid but has trailing whitespace. Do you have an example? I have reproduced a case where key is invalid. Here the problem…

ungleich-gitea commented on issue ungleich-public/cdist#147 2021-11-20 15:43:22 +00:00
__ssh_authorized_keys overwrites existing keys

Created by: pcoder

@darko-poljak check this out

__ssh_authorized_keys root \
  --key "ssh-rsa…
ungleich-gitea commented on issue ungleich-public/cdist#147 2021-11-20 15:43:21 +00:00
__ssh_authorized_keys overwrites existing keys

Created by: darko-poljak

@pcoder Nope, it works with this key. Are you sure you have only this in your manifest and that it clears existing file?

ungleich-gitea commented on issue ungleich-public/cdist#147 2021-11-20 15:43:19 +00:00
__ssh_authorized_keys overwrites existing keys

Created by: pcoder

I think you are right there @darko-poljak I may have overlooked this.

ungleich-gitea commented on issue ungleich-public/cdist#147 2021-11-20 15:43:18 +00:00
__ssh_authorized_keys overwrites existing keys

Created by: darko-poljak

@pcoder Ok, I will prepare a fix for an invalid key case.

ungleich-gitea commented on issue ungleich-public/cdist#146 2021-11-20 15:43:17 +00:00
Temporary directory is not cleaned up in cdist-web

Created by: darko-poljak

@pcoder Are you using ungleich/master?

ungleich-gitea commented on issue ungleich-public/cdist#146 2021-11-20 15:43:15 +00:00
Temporary directory is not cleaned up in cdist-web

Created by: pcoder

Yes @darko-poljak and this commit 08bc38dd0fc85e88b65d3068827346686a1e20f0

ungleich-gitea commented on issue ungleich-public/cdist#146 2021-11-20 15:43:14 +00:00
Temporary directory is not cleaned up in cdist-web

Created by: darko-poljak

@pcoder Can you git pull and try again? I made a fix.

ungleich-gitea commented on issue ungleich-public/cdist#146 2021-11-20 15:43:12 +00:00
Temporary directory is not cleaned up in cdist-web

Created by: pcoder

Thanks @darko-poljak for the quick fix. That works like a charm 👍

ungleich-gitea commented on issue ungleich-public/cdist#145 2021-11-20 15:43:11 +00:00
Add support for dnf

Created by: darko-poljak

Can you perhaps provide PR with new __package_dnf type?

ungleich-gitea commented on issue ungleich-public/cdist#145 2021-11-20 15:43:10 +00:00
Add support for dnf

Created by: schvabodka-man

That would be hard to provide because i obliviously don't have code for pull request

ungleich-gitea commented on issue ungleich-public/cdist#145 2021-11-20 15:43:08 +00:00
Add support for dnf

Created by: sideeffect42

@schvabodka-man Isn't yum symlinked to dnf on RedHat systems with dnf?

Regarding the PR: Are the dnf commands different from their yum equivalents? If not you…

ungleich-gitea commented on issue ungleich-public/cdist#145 2021-11-20 15:43:07 +00:00
Add support for dnf

closed

ungleich-gitea commented on issue ungleich-public/cdist#145 2021-11-20 15:43:06 +00:00
Add support for dnf

I just bumped into this trying to make matrix types. Can you reopen the issue / assign it to me?

ungleich-gitea commented on issue ungleich-public/cdist#145 2021-11-20 15:43:04 +00:00
Add support for dnf

reopened

ungleich-gitea commented on issue ungleich-public/cdist#145 2021-11-20 15:43:03 +00:00
Add support for dnf

assigned to @fnux

ungleich-gitea commented on issue ungleich-public/cdist#145 2021-11-20 15:43:02 +00:00
Add support for dnf

@fnux Here you are.