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

disable utf8 on IDO resource in icingaweb2 resource #141

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kouk
Copy link

@kouk kouk commented Mar 16, 2018

Fixes: #121
Replaces: #139

See below for an explanation:
Icinga/icingaweb2#3170
skm42#1

@bebehei
Copy link
Contributor

bebehei commented Mar 16, 2018

It looks promising, that this is the actual issue. Thank you for your PR. I will review/test it later.

@skm42
Copy link

skm42 commented Mar 17, 2018

works perfect and matches director requirement for utf8. Wasn't aware about it.

@skm42
Copy link

skm42 commented Mar 18, 2018

bebehei highlighted the question of migration #139 (comment). Copied data directory created with 67bb188 and launched by docker-compose. content/etc/icingaweb2/resources.ini is not in place due to volume mount ./data/icinga/etc/icingaweb2:/etc/icingaweb2. After update of ./data/icinga/etc/icingaweb2/resources.ini graphite is looking ok and IDO looks like this:


MariaDB [(none)]> use icinga2idomysql
Reading table information for completion of table and column names
You can turn off this feature to get a quicker startup with -A

Database changed
MariaDB [icinga2idomysql]> show variables like '%char%';
+--------------------------+----------------------------+
| Variable_name | Value |
+--------------------------+----------------------------+
| character_set_client | utf8mb4 |
| character_set_connection | utf8mb4 |
| character_set_database | latin1 |
| character_set_filesystem | binary |
| character_set_results | utf8mb4 |
| character_set_server | latin1 |
| character_set_system | utf8 |
| character_sets_dir | /usr/share/mysql/charsets/ |
+--------------------------+----------------------------+
8 rows in set (0.00 sec)


what about using ini_set in content/opt/setup/60-icingaweb2 to drop in configuration update?

@arcao arcao mentioned this pull request May 2, 2018
@fufar
Copy link

fufar commented Oct 12, 2021

any news? tired of changing charset on any container reinstall :)

@jjethwa
Copy link
Owner

jjethwa commented Oct 12, 2021

Hi @fufar

Sorry, I haven't looked at this one in a while, I think the workarounds in the referenced Issues/PRs are ok, right? If not, what are you having to modify?

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

Successfully merging this pull request may close these issues.

module graphite 0.9.0 on jjethwa/icinga2 have problem
5 participants