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

support tsm servername stanzas via a defined type #7

Open
tosmi opened this issue Feb 10, 2015 · 7 comments
Open

support tsm servername stanzas via a defined type #7

tosmi opened this issue Feb 10, 2015 · 7 comments
Assignees
Milestone

Comments

@tosmi
Copy link
Member

tosmi commented Feb 10, 2015

tsm::stanza { 'new_stanza':
servername => $name,
COMMethod => 'tcp',
tcpport => 35000,
tcpserveraddress => 'tsmserver.com',
config_hash => {},
}

this should be possible via a separate concat fragment.

@tosmi tosmi self-assigned this Feb 10, 2015
@tosmi tosmi added this to the 1.2.0 milestone Apr 15, 2016
@tosmi tosmi modified the milestones: 1.3.0, 1.2.0, 2.0.0 Aug 12, 2016
@bschonec
Copy link

Any progress on this? I'm in need of this very enhancement.

@ZeroPointEnergy
Copy link
Contributor

Hello, we require this feature and I'm currently implementing it. The branch is heavily work in progress and not finished and tested yet but I thought it never hurts to inform early.

The Branch is here: stanza_implementation_wip

I try to implement it in a way that is backwards compatible, so people can update the module and migrate the data separately.

Let me know what you think if you have some time to look at it.

@tosmi
Copy link
Member Author

tosmi commented Nov 2, 2017

thank you for you work. could you open a pull request for this feature, even if it's not finished yet? i could add comments to the code in the pull request if required.

toni

@ZeroPointEnergy
Copy link
Contributor

Ok, opened the PR #50

@ZeroPointEnergy
Copy link
Contributor

Created a new PR with a clean commit PR #51

@vangual
Copy link

vangual commented Dec 6, 2017

Any chance this could be integrated? We're in need of this feature aswell and we'd rather not run a custom fork.

@tosmi
Copy link
Member Author

tosmi commented Dec 7, 2017

sorry for the delay. i'm currently busy with work, i'm going to look into this over the holidays, hopefully this is ok.

thanks for your patience
toni

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants