Skip to content

Commit

Permalink
Merge pull request #60 from lsst-it/remove-avro-templates
Browse files Browse the repository at this point in the history
Remove avro templates setting from the config file
  • Loading branch information
glennmorris authored Jan 29, 2025
2 parents 88f1338 + 577510f commit d450d58
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 20 deletions.
9 changes: 0 additions & 9 deletions REFERENCE.md
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,6 @@ The following parameters are available in the `ccs_sal` class:
* [`kafka_registry_url`](#-ccs_sal--kafka_registry_url)
* [`kafka_sasl_username`](#-ccs_sal--kafka_sasl_username)
* [`kafka_sasl_password`](#-ccs_sal--kafka_sasl_password)
* [`kafka_templates_directory`](#-ccs_sal--kafka_templates_directory)
* [`lfa`](#-ccs_sal--lfa)
* [`prefix_service`](#-ccs_sal--prefix_service)
* [`rpm_repo`](#-ccs_sal--rpm_repo)
Expand Down Expand Up @@ -148,14 +147,6 @@ String giving SASL password

Default value: `'password'`

##### <a name="-ccs_sal--kafka_templates_directory"></a>`kafka_templates_directory`

Data type: `String`

String giving location of templates directory

Default value: `'/home/tonyj/avro-templates'`

##### <a name="-ccs_sal--lfa"></a>`lfa`

Data type: `Hash`
Expand Down
9 changes: 4 additions & 5 deletions manifests/etc.pp
Original file line number Diff line number Diff line change
Expand Up @@ -37,11 +37,10 @@
file { "${dir}/${kafka_salfile}":
ensure => file,
content => epp("${ptitle}/${kafka_salfile}.epp", {
'templates_directory' => $ccs_sal::kafka_templates_directory,
'broker_address' => $ccs_sal::kafka_broker_address,
'sasl_username' => $ccs_sal::kafka_sasl_username,
'sasl_password' => $ccs_sal::kafka_sasl_password,
'registry_url' => $ccs_sal::kafka_registry_url,
'broker_address' => $ccs_sal::kafka_broker_address,
'sasl_username' => $ccs_sal::kafka_sasl_username,
'sasl_password' => $ccs_sal::kafka_sasl_password,
'registry_url' => $ccs_sal::kafka_registry_url,
},
),
* => $attributes,
Expand Down
3 changes: 0 additions & 3 deletions manifests/init.pp
Original file line number Diff line number Diff line change
Expand Up @@ -29,8 +29,6 @@
# String giving SASL username
# @param kafka_sasl_password
# String giving SASL password
# @param kafka_templates_directory
# String giving location of templates directory
# @param lfa
# Hash of properties (id, secret, endpoint, bucket) for large file
# annex (lfa) configuration.
Expand Down Expand Up @@ -65,7 +63,6 @@
String $kafka_registry_url = 'https://tucson-teststand.lsst.codes/schema-registry',
String $kafka_sasl_password = 'password',
String $kafka_sasl_username = 'username',
String $kafka_templates_directory = '/home/tonyj/avro-templates', # FIXME
Hash $lfa = {},
Boolean $prefix_service = true,
## Old: http://www.slac.stanford.edu/~gmorris/lsst/pkgarchive
Expand Down
4 changes: 1 addition & 3 deletions templates/setup-sal-kafka.epp
Original file line number Diff line number Diff line change
@@ -1,11 +1,9 @@
<%- | String $templates_directory,
String $broker_address,
<%- | String $broker_address,
String $sasl_username,
String $sasl_password,
String $registry_url,
| -%>
# This file is managed by Puppet; changes may be overwritten.
export LSST_AVRO_TEMPLATES=<%= $templates_directory %>
export LSST_KAFKA_BROKER_ADDR=<%= $broker_address %>
export LSST_SECURITY_PROTOCOL=SASL_SSL
export LSST_SASL_USERNAME=<%= $sasl_username %>
Expand Down

0 comments on commit d450d58

Please sign in to comment.