Closed allthesignals closed 6 years ago
I think if we can get the production config file in sync with origin master, the Ansible installer will mesh a little better with these code changes. It appears to be a fairly small issue, though... would just be cleaner to keep this up-to-date with origin master. Is there anything specifically blocking that?
Yes it is out of sync. The branch is large and there are many conflicts. I have been prioritizing the features for the demo. If a temp fix can fix it, it would be great. I will sync it within a matter of days.
Reposting from Slack: "a temp fix could be updating the production.rb file. I will PR something soon. Thank you."
Thanks!
I'm using the Ansible-based installer here to deploy Consul from origin master: https://github.com/consul/installer
This seems to work fine, but when I attempt to deploy feature/2-ideas, I am seeing failures having to do with syntax errors in the
config/deploy.rb
step.Full Log
``` /home/deploy/.rvm/gems/ruby-2.3.2/gems/activesupport-4.2.10/lib/active_support/dependencies.rb:274:in `require': /home/deploy/consul/config/environments/production.rb:55: syntax error, unexpected ':', expecting keyword_end (SyntaxError) port: "25", ^ /home/deploy/consul/config/environments/production.rb:55: syntax error, unexpected ',', expecting keyword_end from /home/deploy/.rvm/gems/ruby-2.3.2/gems/activesupport-4.2.10/lib/active_support/dependencies.rb:274:in `block in require' from /home/deploy/.rvm/gems/ruby-2.3.2/gems/activesupport-4.2.10/lib/active_support/dependencies.rb:240:in `load_dependency' from /home/deploy/.rvm/gems/ruby-2.3.2/gems/activesupport-4.2.10/lib/active_support/dependencies.rb:274:in `require' from /home/deploy/.rvm/gems/ruby-2.3.2/gems/railties-4.2.10/lib/rails/engine.rb:598:in `block (2 levels) inThere is a relevant deployment step that writes some configuration into that file: https://github.com/consul/installer/blob/c296c71919f489ab8d8dfb55471b334e0caf19b1/roles/email/tasks/main.yml
When I ssh into the server to see the Ansible-written file, I see this:
This is invalid Ruby syntax.
Any thoughts on this? My guess is that something there changed, or is out of sync with master, and is throwing off the regex portion of the Ansible task.