Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed apt repo error #84

Closed
wants to merge 1 commit into from
Closed

Conversation

hkumarmk
Copy link
Contributor

Corrected the repo key for ubuntu.

In ubuntu trusty, it was giving below error, changing the repo key to correct value
fixed the error.

Apr 23 22:50:28 node1 puppet-user[14480]: validate_re():
"FBABD5FB20255ECAB22EE194D13D58E479EA5ED4" does not match
["\\A(0x)?[0-9a-fA-F]{8}\\Z", "\\A(0x)?[0-9a-fA-F]{16}\\Z"] at
/etc/puppet/environments/production/modules/apt/manifests/key.pp:60 on node
node1.domain.name

Corrected the repo key for ubuntu.

In ubuntu, it was giving below error, changing the repo key to correct value
fixed the error.
Apr 23 22:50:28 node1 puppet-user[14480]: validate_re():
"FBABD5FB20255ECAB22EE194D13D58E479EA5ED4" does not match
["\\A(0x)?[0-9a-fA-F]{8}\\Z", "\\A(0x)?[0-9a-fA-F]{16}\\Z"] at
/etc/puppet/environments/production/modules/apt/manifests/key.pp:60 on node
node1.domain.name
@karolisc
Copy link
Contributor

Hi,

take a look at this commit

Karolis

@exptom
Copy link
Contributor

exptom commented Apr 26, 2015

@hkumarmk what version of puppet are you using? Is it puppet enterprise? I committed a similar fix to another puppet module which they merged but later reverted because they needed to support an old version of PE that cannot cope with the full length keys.

@hkumarmk
Copy link
Contributor Author

@exptom I use free version of puppet - 3.7.5.

@exptom
Copy link
Contributor

exptom commented Apr 29, 2015

That really shouldn't be erroring for you then - what version of the apt module are you using?

@dj-wasabi
Copy link

Hi,

I think an older version of the apt module. Few weeks ago I found out that the apt wasn't in the metadata.json file was, so I added it. (https://github.com/dj-wasabi/puppet-zabbix/blob/master/metadata.json#L14).

Kind regards,
Werner

@exptom
Copy link
Contributor

exptom commented Apr 30, 2015

@dj-wasabi You have a missing comma at the end of this line: https://github.com/dj-wasabi/puppet-zabbix/blob/master/metadata.json#L13

@dj-wasabi
Copy link

Hi @exptom

Thanks you for noticing. I just updated the file.

@hkumarmk
I close this pull request, because the fix for the metadata will fix the original issue to. Thank you for making time to create the request though. If anything found in the future, don't hesitate to create a new pull request.

Kind regards,
Werner

@dj-wasabi dj-wasabi closed this May 11, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants