{"action":"created","issue":{"url":"https://api.github.com/repos/kubic-project/caasp-container-manifests/issues/189","repository_url":"https://api.github.com/repos/kubic-project/caasp-container-manifests","labels_url":"https://api.github.com/repos/kubic-project/caasp-container-manifests/issues/189/labels{/name}","comments_url":"https://api.github.com/repos/kubic-project/caasp-container-manifests/issues/189/comments","events_url":"https://api.github.com/repos/kubic-project/caasp-container-manifests/issues/189/events","html_url":"https://github.com/kubic-project/caasp-container-manifests/pull/189","id":328090603,"node_id":"MDExOlB1bGxSZXF1ZXN0MTkxNzE2MzYx","number":189,"title":"Configure `innodb_log_file_size` to a 128M limit","user":{"login":"ereslibre","id":8706,"node_id":"MDQ6VXNlcjg3MDY=","avatar_url":"https://avatars2.githubusercontent.com/u/8706?v=4","gravatar_id":"","url":"https://api.github.com/users/ereslibre","html_url":"https://github.com/ereslibre","followers_url":"https://api.github.com/users/ereslibre/followers","following_url":"https://api.github.com/users/ereslibre/following{/other_user}","gists_url":"https://api.github.com/users/ereslibre/gists{/gist_id}","starred_url":"https://api.github.com/users/ereslibre/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ereslibre/subscriptions","organizations_url":"https://api.github.com/users/ereslibre/orgs","repos_url":"https://api.github.com/users/ereslibre/repos","events_url":"https://api.github.com/users/ereslibre/events{/privacy}","received_events_url":"https://api.github.com/users/ereslibre/received_events","type":"User","site_admin":false},"labels":[{"id":935488721,"node_id":"MDU6TGFiZWw5MzU0ODg3MjE=","url":"https://api.github.com/repos/kubic-project/caasp-container-manifests/labels/release-3.0-candidate","name":"release-3.0-candidate","color":"fbca04","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2018-05-31T10:52:05Z","updated_at":"2018-06-05T08:12:39Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/kubic-project/caasp-container-manifests/pulls/189","html_url":"https://github.com/kubic-project/caasp-container-manifests/pull/189","diff_url":"https://github.com/kubic-project/caasp-container-manifests/pull/189.diff","patch_url":"https://github.com/kubic-project/caasp-container-manifests/pull/189.patch"},"body":"When a salt output is big enough, mysql will refuse to insert the offending\r\nrow for being too big, with an error:\r\n\r\n```\r\n[ERROR   ] Could not store events - returner 'mysql.event_return' raised exception:\r\n(1118, 'The size of BLOB/TEXT data inserted in one transaction is greater than 10% of redo log size.\r\nIncrease the redo log size using innodb_log_file_size.')\r\n```\r\n\r\nWhatever we set as `innodb_log_file_size` will be an arbitrary number that will\r\neventually be flooded if the cluster is big enough, or if salt is noisy enough.\r\nGiven a cluster size, this can suddenly fail if we add more states (thus, increasing\r\nsalt's output). Obviously, given the same salt states, we can also reach this limit\r\nby increasing the cluster size.\r\n\r\nThere is not a definitive fix for this issue, all we can do for now (without a proper\r\nrefactor of the way we integrate salt and velum) is to ensure that with todays salt\r\nstates we can reach a certain number of nodes. As said, this can no longer be true if\r\nwe add more salt states and we reach again the limit for the same cluster size.\r\n\r\nFixes: bsc#1095335\r\n\r\nDepends on: https://github.com/kubic-project/container-images/pull/31"},"comment":{"url":"https://api.github.com/repos/kubic-project/caasp-container-manifests/issues/comments/394621447","html_url":"https://github.com/kubic-project/caasp-container-manifests/pull/189#issuecomment-394621447","issue_url":"https://api.github.com/repos/kubic-project/caasp-container-manifests/issues/189","id":394621447,"node_id":"MDEyOklzc3VlQ29tbWVudDM5NDYyMTQ0Nw==","user":{"login":"pgonin","id":6192686,"node_id":"MDQ6VXNlcjYxOTI2ODY=","avatar_url":"https://avatars3.githubusercontent.com/u/6192686?v=4","gravatar_id":"","url":"https://api.github.com/users/pgonin","html_url":"https://github.com/pgonin","followers_url":"https://api.github.com/users/pgonin/followers","following_url":"https://api.github.com/users/pgonin/following{/other_user}","gists_url":"https://api.github.com/users/pgonin/gists{/gist_id}","starred_url":"https://api.github.com/users/pgonin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pgonin/subscriptions","organizations_url":"https://api.github.com/users/pgonin/orgs","repos_url":"https://api.github.com/users/pgonin/repos","events_url":"https://api.github.com/users/pgonin/events{/privacy}","received_events_url":"https://api.github.com/users/pgonin/received_events","type":"User","site_admin":false},"created_at":"2018-06-05T08:12:39Z","updated_at":"2018-06-05T08:12:39Z","author_association":"NONE","body":"The change unblocked the scaling tests and more nodes could be added to cluster (beyond 140 nodes)"}}
{"actor":{"display_login":"pgonin"}}