-
-
Notifications
You must be signed in to change notification settings - Fork 388
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
Resolve linter issues (ansible-lint, yamllint, flake8) #315
Commits on May 3, 2023
-
🐳 chore(Dockerfile): add Docker installation and create gitpod user
The Dockerfile installs Docker and other packages required for development. It also creates a gitpod user with UID 33333. 🔧 chore(gitpod.yml): update Dockerfile path to match new location The .gitpod.yml file is updated to reflect the new location of the Dockerfile.
Configuration menu - View commit details
-
Copy full SHA for bf055e8 - Browse repository at this point
Copy the full SHA bf055e8View commit details -
🔧 chore(playbooks): improve readability of playbooks by removing unne…
…cessary hyphens and quotes The changes in these playbooks are purely cosmetic, removing unnecessary hyphens and quotes to improve readability and consistency. This makes the playbooks easier to read and understand, which can help with maintenance and future development.
Configuration menu - View commit details
-
Copy full SHA for 715fa35 - Browse repository at this point
Copy the full SHA 715fa35View commit details -
🔧 chore(playbook): add
noqa run-once
to run_once tasks to suppress ……Ansible linting warnings The `run_once` tasks in the playbook were updated to include a `noqa run-once` comment to suppress Ansible linting warnings. This improves the readability of the playbook and reduces the noise generated by the linter.
Configuration menu - View commit details
-
Copy full SHA for c13447a - Browse repository at this point
Copy the full SHA c13447aView commit details -
🚀 chore(add_balancer.yml): add descriptive name to playbook and host
The playbook now has a descriptive name "Add load balancer" to improve readability and understanding of the playbook's purpose. The host section now has a name "new_balancer" to improve the clarity of the playbook's target.
Configuration menu - View commit details
-
Copy full SHA for 1806dba - Browse repository at this point
Copy the full SHA 1806dbaView commit details -
📝 chore(add_balancer.yml): update task names to improve readability a…
…nd traceability The first task name has been updated to include the name of the playbook file to improve traceability. The second task name has been updated to include more descriptive language to improve readability.
Configuration menu - View commit details
-
Copy full SHA for 7644c6b - Browse repository at this point
Copy the full SHA 7644c6bView commit details -
🐛 fix(add_pgnode.yml): change include to include_tasks to avoid depre…
…cation warning The include keyword is deprecated and will be removed in future versions of Ansible. The include_tasks keyword should be used instead to include tasks from another file.
Configuration menu - View commit details
-
Copy full SHA for 890bb04 - Browse repository at this point
Copy the full SHA 890bb04View commit details -
🚀 chore(add_pgnode.yml): improve readability and add descriptive name…
…s to tasks The changes in this commit improve the readability of the playbook by adding descriptive names to the tasks. The name of the task now reflects the purpose of the task, making it easier to understand what is happening in the playbook. Additionally, the changes make the playbook more modular by importing the Consul playbook instead of duplicating the code.
Configuration menu - View commit details
-
Copy full SHA for 05ee14b - Browse repository at this point
Copy the full SHA 05ee14bView commit details -
📝 chore(add_pgnode.yml): add task names to improve readability and tr…
…aceability The commit adds task names to the playbook to improve readability and traceability of the tasks. The task names are now prefixed with the name of the playbook, which makes it easier to identify which playbook a task belongs to.
Configuration menu - View commit details
-
Copy full SHA for c6e4cc6 - Browse repository at this point
Copy the full SHA c6e4cc6View commit details
Commits on May 5, 2023
-
🔥 chore(balancers.yml): remove unnecessary line
The line "---" was removed from the beginning of the file as it was unnecessary.
Configuration menu - View commit details
-
Copy full SHA for d6998a1 - Browse repository at this point
Copy the full SHA d6998a1View commit details -
🎨 style(balancers.yml): add name to playbook and update hosts key to …
…improve readability The name key was added to the playbook to provide a clear and concise description of what the playbook does. The hosts key was updated to use a more descriptive name to improve readability and make it easier to understand what the playbook is targeting.
Configuration menu - View commit details
-
Copy full SHA for eb3b9a2 - Browse repository at this point
Copy the full SHA eb3b9a2View commit details -
📝 docs(balancers.yml): update playbook name in balancers.yml to impro…
…ve readability The playbook name in balancers.yml has been updated to include the filename for better readability.
Configuration menu - View commit details
-
Copy full SHA for ebb6824 - Browse repository at this point
Copy the full SHA ebb6824View commit details -
🔥 chore(config_pgcluster.yml): remove unnecessary lines
The changes in this commit remove unnecessary whitespace and a line that was not needed in the configuration file. This improves the readability and maintainability of the code.
Configuration menu - View commit details
-
Copy full SHA for b1aaad2 - Browse repository at this point
Copy the full SHA b1aaad2View commit details -
📝 docs(config_pgcluster.yml): add file name to playbook name and impr…
…ove task names The playbook name now includes the file name to improve clarity and the task names have been improved to better describe what they do.
Configuration menu - View commit details
-
Copy full SHA for df96a2d - Browse repository at this point
Copy the full SHA df96a2dView commit details -
📝 docs(balancers.yml): fix typo in playbook name
The playbook name was misspelled as "balancer.yml" instead of "balancers.yml". This commit fixes the typo to improve the readability and clarity of the playbook name.
Configuration menu - View commit details
-
Copy full SHA for 9583b63 - Browse repository at this point
Copy the full SHA 9583b63View commit details -
🔧 chore(config_pgcluster.yml): improve playbook readability by adding…
… descriptive names to tasks The playbook has been updated to include descriptive names for each task. This improves the readability of the playbook and makes it easier to understand what each task is doing. The names have been added to the tasks that configure the Postgres cluster, deploy the Postgres cluster, and configure HAProxy load balancing.
Configuration menu - View commit details
-
Copy full SHA for 541b58a - Browse repository at this point
Copy the full SHA 541b58aView commit details -
🐛 fix(config_pgcluster.yml): change include to include_tasks in handlers
The include keyword is deprecated and should be replaced with include_tasks. This commit updates the handlers in the config_pgcluster.yml file to use include_tasks instead of include.
Configuration menu - View commit details
-
Copy full SHA for d74843e - Browse repository at this point
Copy the full SHA d74843eView commit details
Commits on May 6, 2023
-
Configuration menu - View commit details
-
Copy full SHA for d0d7a36 - Browse repository at this point
Copy the full SHA d0d7a36View commit details -
🔍 chore(consul.yml): remove unnecessary whitespace and comments
The changes in this commit are purely cosmetic. The commit removes unnecessary whitespace and comments in the consul.yml file to improve readability and maintainability.
Configuration menu - View commit details
-
Copy full SHA for a790732 - Browse repository at this point
Copy the full SHA a790732View commit details -
🎨 style(consul.yml): add name to playbook and update host names for r…
…eadability The name "Consul Playbook" was added to the playbook to improve readability and make it easier to identify the purpose of the playbook. The host names were also updated to be more descriptive and improve readability.
Configuration menu - View commit details
-
Copy full SHA for fdd2be5 - Browse repository at this point
Copy the full SHA fdd2be5View commit details -
📝 chore(consul.yml): update playbook name and task names for better r…
…eadability The playbook name and task names have been updated to include the filename and to be more descriptive. This improves the readability of the playbook and makes it easier to understand what each task is doing.
Configuration menu - View commit details
-
Copy full SHA for dc4ce0c - Browse repository at this point
Copy the full SHA dc4ce0cView commit details -
🔥 chore(deploy_pgcluster.yml): remove unnecessary lines
The changes in this commit remove unnecessary lines in the deploy_pgcluster.yml file. Specifically, the removed lines are the name of the playbook and the ellipsis at the end of the file.
Configuration menu - View commit details
-
Copy full SHA for 33fa52e - Browse repository at this point
Copy the full SHA 33fa52eView commit details -
🚀 chore(deploy_pgcluster.yml): add names to import_playbook tasks for…
… better readability The import_playbook tasks in the deploy_pgcluster.yml file have been given names to improve the readability of the playbook. The names now reflect the purpose of the tasks, making it easier to understand what each task does.
Configuration menu - View commit details
-
Copy full SHA for 171b17a - Browse repository at this point
Copy the full SHA 171b17aView commit details -
📝 chore(deploy_pgcluster.yml): add task names to playbook for better …
…readability The playbook now has task names for each section to improve readability and make it easier to understand what each section does. The task names are "Postgres Cluster Configuration", "Install and configure pgBackRest", and "PostgreSQL Cluster Deployment".
Configuration menu - View commit details
-
Copy full SHA for a8cb1ed - Browse repository at this point
Copy the full SHA a8cb1edView commit details -
🚀 chore(deploy_pgcluster.yml): add playbook name to all tasks for bet…
…ter readability The playbook name was added to all tasks in the deploy_pgcluster.yml file to improve readability and make it easier to identify which playbook a task belongs to.
Configuration menu - View commit details
-
Copy full SHA for 74aea7b - Browse repository at this point
Copy the full SHA 74aea7bView commit details -
🐛 fix(ansible): set default value for firewall_ports_dynamic_var to a…
…n empty list The firewall_ports_dynamic_var variable is now set to a default value of an empty list to avoid errors when the variable is not defined. This change was made in multiple ansible files.
Configuration menu - View commit details
-
Copy full SHA for 195f10a - Browse repository at this point
Copy the full SHA 195f10aView commit details -
🐛 fix(firewall): set default value for firewall_additional_rules to a…
…n empty list ✨ feat(firewall): ensure firewall_additional_rules is unique The firewall_additional_rules variable is now set to a default value of an empty list to avoid errors when the variable is not defined. The unique filter is also applied to the variable to ensure that there are no duplicate rules.
Configuration menu - View commit details
-
Copy full SHA for a56b54c - Browse repository at this point
Copy the full SHA a56b54cView commit details
Commits on May 7, 2023
-
📝 chore(etcd_cluster.yml): update playbook name and add description
The playbook name has been updated to better reflect its purpose. The name change also improves readability and makes it easier to understand what the playbook does. A description has been added to provide more context and clarity.
Configuration menu - View commit details
-
Copy full SHA for 9377db7 - Browse repository at this point
Copy the full SHA 9377db7View commit details -
🐛 fix(main.yml): wrap min_ansible_version and platform versions in qu…
…otes The min_ansible_version and platform versions are now wrapped in quotes to ensure that they are interpreted as strings. This is important because some versions may contain characters that could be interpreted as numbers or other data types. Wrapping them in quotes ensures that they are interpreted as strings and avoids any potential issues.
Configuration menu - View commit details
-
Copy full SHA for 887d239 - Browse repository at this point
Copy the full SHA 887d239View commit details -
🔥 chore(main.yml): remove high-availability tag from galaxy_tags
The high-availability tag is being removed from the galaxy_tags list as it is not relevant to this role.
Configuration menu - View commit details
-
Copy full SHA for 347a4cb - Browse repository at this point
Copy the full SHA 347a4cbView commit details
Commits on May 8, 2023
-
🔨 refactor(converge.yml): remove unnecessary comments and whitespace
The commit removes unnecessary comments and whitespace in the converge.yml file. This improves the readability of the file and makes it easier to maintain.
Configuration menu - View commit details
-
Copy full SHA for fdea635 - Browse repository at this point
Copy the full SHA fdea635View commit details -
🚀 chore(converge.yml): add name to import_playbook task and remove un…
…necessary hyphen The name attribute was added to the import_playbook task to improve readability and make it easier to identify the task in the output. The hyphen before the import_playbook task was removed as it was unnecessary.
Configuration menu - View commit details
-
Copy full SHA for cbdc11e - Browse repository at this point
Copy the full SHA cbdc11eView commit details -
🔥 chore(remove_cluster.yml): remove unnecessary lines and trailing wh…
…itespace The changes in this commit are purely cosmetic. Removed unnecessary comments and trailing whitespace to improve readability and consistency.
Configuration menu - View commit details
-
Copy full SHA for 648df92 - Browse repository at this point
Copy the full SHA 648df92View commit details -
🎨 style(remove_cluster.yml): add descriptive names to playbook and plays
The playbook and plays in the remove_cluster.yml file have been updated to include descriptive names. The PostgreSQL and Etcd plays have been named to improve readability and make it easier to understand what each play does.
Configuration menu - View commit details
-
Copy full SHA for b725925 - Browse repository at this point
Copy the full SHA b725925View commit details -
🔥 chore(remove_cluster.yml): update playbook names and add comments
The playbook names have been updated to include the filename for better identification. Comments have been added to explain the purpose of the variables.
Configuration menu - View commit details
-
Copy full SHA for e78f252 - Browse repository at this point
Copy the full SHA e78f252View commit details -
🚀 chore(converge.yml): add descriptive name to import_playbook task
The task to import the deploy_pgcluster.yml playbook has been given a more descriptive name to improve readability and clarity of the task's purpose.
Configuration menu - View commit details
-
Copy full SHA for 4e4501b - Browse repository at this point
Copy the full SHA 4e4501bView commit details -
🔍 chore(yedit.py): fix typos and formatting in options description
The commit fixes a typo in the options description of the yedit.py file. The word "follow" was missing a space after it. The commit also removes an unnecessary newline character.
Configuration menu - View commit details
-
Copy full SHA for 30a971e - Browse repository at this point
Copy the full SHA 30a971eView commit details -
🔧 chore(.flake8): ignore E501 error code to allow lines to exceed 79 …
…characters The E501 error code is ignored to allow lines to exceed the default 79 character limit. This is done to improve readability and reduce the need for unnecessary line breaks.
Configuration menu - View commit details
-
Copy full SHA for 6fddafa - Browse repository at this point
Copy the full SHA 6fddafaView commit details -
🔧 chore(.flake8): add W503 to ignored errors
🐛 fix(yedit.py): fix typo in error message and split it into two lines for readability 🔥 refactor(yedit.py): remove unused json_roundtrip_clean function The ignored errors in flake8 have been updated to include W503. In yedit.py, a typo in the error message has been fixed and the message has been split into two lines for better readability. The unused json_roundtrip_clean function has been removed.
Configuration menu - View commit details
-
Copy full SHA for ae2e372 - Browse repository at this point
Copy the full SHA ae2e372View commit details -
🔨 refactor(yedit.py): improve readability by removing unnecessary str…
…ing concatenation The commit removes unnecessary string concatenation in two raise statements in the Yedit class. The code is now more readable and easier to understand.
Configuration menu - View commit details
-
Copy full SHA for 1c66975 - Browse repository at this point
Copy the full SHA 1c66975View commit details -
🐛 fix(yedit.py): fix formatting of exception message
The exception message was split into two lines, which made it harder to read. This commit fixes the formatting of the message to make it more readable.
Configuration menu - View commit details
-
Copy full SHA for b478278 - Browse repository at this point
Copy the full SHA b478278View commit details -
🐛 fix(yedit.py): fix formatting of exception message
The exception message was formatted in a way that made it difficult to read. This commit fixes the formatting of the exception message to make it more readable.
Configuration menu - View commit details
-
Copy full SHA for c15c7f8 - Browse repository at this point
Copy the full SHA c15c7f8View commit details -
🐛 fix(yedit.py): remove unnecessary line break in YeditException message
The line break in the YeditException message was unnecessary and made the message harder to read. This commit removes the line break to improve readability.
Configuration menu - View commit details
-
Copy full SHA for 30b3236 - Browse repository at this point
Copy the full SHA 30b3236View commit details -
🐛 fix(yedit.py): remove unnecessary line break in if statement
The line break in the if statement was unnecessary and made the code harder to read. This commit removes the line break to improve readability.
Configuration menu - View commit details
-
Copy full SHA for 41730f7 - Browse repository at this point
Copy the full SHA 41730f7View commit details -
🐛 fix(yedit.py): remove unnecessary parentheses and improve code read…
…ability The parentheses around the condition in the elif statement are unnecessary and have been removed to improve code readability.
Configuration menu - View commit details
-
Copy full SHA for 8977fb6 - Browse repository at this point
Copy the full SHA 8977fb6View commit details -
🐛 fix(yedit.py): fix whitespace around subtraction operator in if sta…
…tement The if statement was checking if an index was greater than the length of a list minus one. The subtraction operator was not surrounded by whitespace, which is not consistent with PEP8 style guide. This commit fixes the whitespace around the subtraction operator to improve code readability and consistency with PEP8.
Configuration menu - View commit details
-
Copy full SHA for f4a05db - Browse repository at this point
Copy the full SHA f4a05dbView commit details -
🐛 fix(yedit.py): remove unnecessary line breaks in if statements
The changes remove unnecessary line breaks in if statements to improve code readability.
Configuration menu - View commit details
-
Copy full SHA for 7ee6a2f - Browse repository at this point
Copy the full SHA 7ee6a2fView commit details -
🐛 fix(.flake8): add E402 to ignored errors to avoid false positives
The E402 error is now ignored in the flake8 configuration file. This change was made to avoid false positives in the codebase.
Configuration menu - View commit details
-
Copy full SHA for 387e9c8 - Browse repository at this point
Copy the full SHA 387e9c8View commit details
Commits on May 23, 2023
-
🐛 fix(config_pgcluster.yml): add check for PostgreSQL version when en…
…abling TimescaleDB ✨ feat(config_pgcluster.yml): add support for TimescaleDB by adding 'timescaledb' to 'shared_preload_libraries' PostgreSQL parameter The check for the PostgreSQL version is added to ensure that the current version is supported by TimescaleDB. If the version is not supported, the playbook will fail. The support for TimescaleDB is added by adding 'timescaledb' to the 'shared_preload_libraries' PostgreSQL parameter. This is done by modifying the 'postgresql_parameters' variable. If 'timescaledb' is not already present in the value of 'shared_preload_libraries', it is appended to the end of the value. If it is already present, the value is left unchanged.
Configuration menu - View commit details
-
Copy full SHA for f75ba88 - Browse repository at this point
Copy the full SHA f75ba88View commit details -
🔍 chore(.gitignore): add .vscode/settings.json to ignore list
The .vscode/settings.json file is now being ignored by git. This file is used to store user settings for the Visual Studio Code editor.
Configuration menu - View commit details
-
Copy full SHA for 27963e7 - Browse repository at this point
Copy the full SHA 27963e7View commit details -
🐛 fix(deploy_pgcluster.yml): add check for PostgreSQL version when en…
…abling TimescaleDB ✨ feat(deploy_pgcluster.yml): add support for TimescaleDB installation via repository The playbook now checks the PostgreSQL version before enabling TimescaleDB to ensure that it is compatible. If the version is not compatible, the playbook will fail. Additionally, support for TimescaleDB installation via repository has been added. The playbook now ensures that 'timescaledb' is in 'shared_preload_libraries' when TimescaleDB is enabled.
Configuration menu - View commit details
-
Copy full SHA for 8d99dcd - Browse repository at this point
Copy the full SHA 8d99dcdView commit details -
🚀 feat(converge.yml): add enable_timescale variable for TimescaleDB c…
…luster deployment test The enable_timescale variable is set to true to enable testing of TimescaleDB cluster deployment. The variable is only set if the ansible_distribution is not Ubuntu 20.04, as there is a known issue with TimescaleDB on that version.
Configuration menu - View commit details
-
Copy full SHA for 4af8781 - Browse repository at this point
Copy the full SHA 4af8781View commit details -
Configuration menu - View commit details
-
Copy full SHA for 8170576 - Browse repository at this point
Copy the full SHA 8170576View commit details
Commits on May 24, 2023
-
Co-authored-by: Vitaliy Kukharik <37010174+vitabaks@users.noreply.github.com>
Configuration menu - View commit details
-
Copy full SHA for 5d04d2d - Browse repository at this point
Copy the full SHA 5d04d2dView commit details -
Co-authored-by: Vitaliy Kukharik <37010174+vitabaks@users.noreply.github.com>
Configuration menu - View commit details
-
Copy full SHA for 9e254cb - Browse repository at this point
Copy the full SHA 9e254cbView commit details