Comments (10)
@Mik3yZ ,
see the fix #570
Thanks for reporting the issue!
from ansible-collection-checkmk.general.
Hi @Mik3yZ,
I would say works as intended.
Folder title and path (folder name) are two different parameters.
path
(folder name) is and to best of my knowledge should be low case.
title
(alias name
in folder module) on the other hand, can also contain capital letters.
from ansible-collection-checkmk.general.
It should be
self.desired["title"] = self.params.get("name", self.desired["name"])
I'll make corrections
from ansible-collection-checkmk.general.
Hi @msekania
Maybe my description was a bit off, changed it to reflect what i mean. Hope this makes thing clearer.
from ansible-collection-checkmk.general.
So, to come back to your last sentence @msekania, title
(alias name
in the module) is parsed through function _normalize_path
, and thus alway transformed to lowercase.
from ansible-collection-checkmk.general.
So, to come back to your last sentence @msekania,
title
(aliasname
in the module) is parsed through function_normalize_path
, and thus alway transformed to lowercase.
lines 220-221: _normalize_path(...) is called with a parameter path
, hence path
is parsed with _normalize_path(...) not the name (alias title).
from ansible-collection-checkmk.general.
true, however it does set (self.desired["parent"], self.desired["name"])
with that processed value. please run my example with 4.3.1 and see for yourself.
from ansible-collection-checkmk.general.
ansible-collection-checkmk.general/plugins/modules/folder.py
Lines 223 to 224 in 300764e
states it explicitly what goes to title
name
parameter of the module aliases title
ansible-collection-checkmk.general/plugins/modules/folder.py
Lines 481 to 485 in 300764e
from ansible-collection-checkmk.general.
yes, i saw that too. however i do get only lowercase names while creating the folders.
i would then say line 220 needs to be changed from:
(self.desired["parent"], self.desired["name"]) = self._normalize_path(
to
(self.desired["parent"], self.desired["path"]) = self._normalize_path(
from ansible-collection-checkmk.general.
yes, i saw that too. however i do get only lowercase names while creating the folders.
i would then say line 220 needs to be changed from:
(self.desired["parent"], self.desired["name"]) = self._normalize_path(
to(self.desired["parent"], self.desired["path"]) = self._normalize_path(
self.desired["title"] = self.params.get("title", self.desired["name"])
No, actually that's fine, because if title is not defined self.desired["name"]
folder is a default name
problem is somewhere else, because self.params.get("title")
is Null
instead of Name.
from ansible-collection-checkmk.general.
Related Issues (20)
- [BUG] activation module with redirect set to true fails if API returns a 302 (possibly only HTTPS) HOT 6
- [BUG] update_attributes on folder with network scan enabled (and a network scan result present) fails HOT 5
- [FEED]Agent download fails in distributed setup HOT 1
- [BUG] Inconsistent behaviour of Ansible modules (update vs remove + compared to API) HOT 10
- [BUG]: reset_password in user module recreates the user HOT 1
- host_tag_group Endpoint changed the parameters id/ident HOT 1
- [BUG] notification rules is not recognized by checkmk.general.rule HOT 3
- [FEED] HOT 2
- [BUG] idempotency of rule fails on datasource_programs ruleset HOT 8
- [BUG] Register Linux agent for TLS fails with CRE edition HOT 3
- [BUG] Agent role has unecessary changes, when the agent is already registered and added to CheckMK HOT 2
- [Feature Request] Create Host and Service Groups Rules via Ansible HOT 4
- [Feed] folder lookup plugin does not return the title of the folder HOT 3
- fixed downtime? HOT 4
- [BUG] "Register Agent for TLS" does not allow for "!" in checkmk_agent_auth on Linux HOT 1
- [FEED] Only register agent if host was created on the server or agent was installed HOT 4
- [FEED] Implement module for LDAP Connections
- [BUG] module "folder" not idempotent HOT 10
- [BUG] `discovery` module out of sync with the Checkmk REST API HOT 2
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 ansible-collection-checkmk.general.