Comments (12)
I was actually wondering about this as well.
from consul.
Looks like this is the pull request where the feature was added to consul:
hashicorp/consul#591
I guess this is the section of the cookbook which would need updating?
https://github.com/johnbellone/consul-cookbook/blob/9029990b25c0eefb96674b98ffbc8f189d1a7dda/resources/check_def.rb
Also, in the readme, the syntax of a check is given as follows:
consul_service_def 'server1' do
port 80
tags ['http']
check(
interval: '10s',
http: 'http://localhost:80'
)
notifies :reload, 'service[consul]'
end
Where as, I had to write it as :
consul_service_def 'server1' do
port 80
tags ['http']
check = {
interval: '10s',
http: 'http://localhost:80'
}
notifies :reload, 'service[consul]'
end
Perhaps it needs defining as an array of hashes to define multiple checks.
from consul.
@johnbellone Would you like some help implementing this on general road to 1.0.0 stuff? I have a need for this feature and don't mind doing some of the lifting. We have been getting a ton of mileage from all your leg work already and I don't mind helping out where I can.
Not sure what sort of timeline you are looking for but I can probably manage the time over the next couple weeks to knock it out.
from consul.
@zarry Yes that'd be great. Take a look at the develop branch. I need to test the HWRP thoroughly and document them.
from consul.
@johnbellone Are you changing some of the terminology from the current version/state to what is in the develop branch? Looks like consul_service is now used to actually define a consul instance and not represent a service in consul? Would consul_definition be where we define services? Looks not fully implemented(or just lacking documentation)?
What is the current state of the develop branch?
from consul.
Never mind, thats what I get for quickly skimming it. I see what is going on. I should have some time in the next day or two to start playing around with the development branch. If you could make sure anything you have locally is pushed that would be great.
Thanks!
from consul.
Right now I need to thoroughly test that branch, but otherwise I am waiting to get some time this week or next to document the definition HWRP. It would be a huge help to do that!
There are no commits locally. Once the README is written I am going to cut a patch release with remaining changes for 0.10 and merge in 1.0.
from consul.
Actually, as I dive in I still think consul_definition may have been missing some attributes. for_keeps is empty and we have no way of supplying a service or check definition json to the file system.
Regardless, I will get some documentation around it and make any changes I think should be there to make it work as I would expect. We can hash out any of the details or your idea of how it should look when I submit the PR. (To be fair I don't have a ton of experience with HWRP so I may be missing a common pattern)
from consul.
@zarry I'll tackle this during my train ride home tonight.
from consul.
@johnbellone Thanks, I spent a little time with it and I think I got a little lost with the HWRP. I don't think I fully groked your vision and I wasn't able to pick it up and finish it from there. If you can get a little more documentation I can do some testing for you.
from consul.
@zarry No worries. I really need to write up a lengthy blog post and more documentation here.
from consul.
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
from consul.
Related Issues (20)
- Dangerfile should use failure instead of fail
- Remove .rubocop.yml with Dangerfile HOT 1
- Update Changelog HOT 2
- Run latest cookstyle HOT 2
- Update builds to be parallel HOT 1
- undefined method `join_path' for PoiseArchive::Resources::PoiseArchive::Resource HOT 6
- Segment part of the config is wrong type HOT 1
- Poise dependency is abandoned HOT 3
- Rewrite Consul cookbook to use custom resources HOT 5
- `services` definition throws "no implicit conversion of Symbol into Integer" HOT 3
- Support arm64 on linux
- Diplomat gem fails on Chef 16.9.29+ HOT 4
- Error in configure_diplomat with Diplomat 2.5.0 HOT 12
- Chef 17 - Poise no longer works HOT 2
- Consul 1.9.x - ui/ui_dir deprecated HOT 1
- Why are unit tests disabled? Any plans to return them back? HOT 1
- consul_service ignores program property HOT 1
- Dependency Dashboard
- Consul 1.12.x deprecated fields
- Consul client install latest packages missing checksums
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from consul.