Skip to content

Commit 954e6c0

Browse files
author
Dan Bode
committedMar 2, 2012
Remove reference to ssh keys
Swift does not need to have keys. This commit removes the ssh keys.
1 parent 2dbcf74 commit 954e6c0

File tree

2 files changed

+1
-32
lines changed

2 files changed

+1
-32
lines changed
 

‎QUESTIONS

+1
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
do we need ssh keys for anything?

‎manifests/init.pp

-32
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,6 @@
77
# [*package_ensure*] The ensure state for the swift package.
88
# Optional. Defaults to present.
99
#
10-
# [*swift_ssh_key*] NOT YET IMPLEMENTED. I am not entirely sure what
11-
# this key is intended to be used for.
12-
#
1310
# == Dependencies
1411
#
1512
# Class['ssh::server::install']
@@ -24,11 +21,9 @@
2421
#
2522
class swift(
2623
$swift_hash_suffix,
27-
# $swift_ssh_key,
2824
$package_ensure = 'present'
2925
) {
3026

31-
# maybe I should just install ssh?
3227
Class['ssh::server::install'] -> Class['swift']
3328

3429
package { 'swift':
@@ -56,31 +51,4 @@
5651
mode => 0660,
5752
content => template('swift/swift.conf.erb'),
5853
}
59-
60-
# if ($swift_ssh_key) {
61-
# if $swift_ssh_key !~ /^(ssh-...) +([^ ]*) *([^ \n]*)/ {
62-
# err("Can't parse swift_ssh_key")
63-
# notify { "Can't parse public key file $name on the keymaster: skipping ensure => $e
64-
#nsure": }
65-
# } else {
66-
# $keytype = $1
67-
# $modulus = $2
68-
# $comment = $3
69-
# ssh_authorized_key { $comment:
70-
# ensure => "present",
71-
# user => "swift",
72-
# type => $keytype,
73-
# key => $modulus,
74-
# options => $options ? { "" => undef, default => $options },
75-
# require => File["/home/swift"]
76-
# }
77-
# }
78-
# }
79-
# does swift need an ssh key?
80-
# they are adding one in the openstack modules
81-
82-
#
83-
# I do not understand how to configure the rings
84-
# or why rings would be configured on the proxy?
85-
8654
}

0 commit comments

Comments
 (0)
Please sign in to comment.