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

[dev.icinga.com #7321] Test deduplicated Puppet code #1048

Closed
icinga-migration opened this issue Sep 30, 2014 · 24 comments
Closed

[dev.icinga.com #7321] Test deduplicated Puppet code #1048

icinga-migration opened this issue Sep 30, 2014 · 24 comments
Labels
enhancement New feature or improvement

Comments

@icinga-migration
Copy link

This issue has been migrated from Redmine: https://dev.icinga.com/issues/7321

Created by aklimov on 2014-09-30 09:38:22 +00:00

Assignee: (none)
Status: Closed (closed on 2014-12-19 08:30:05 +00:00)
Target Version: (none)
Last Update: 2014-12-19 08:30:05 +00:00 (in Redmine)


The deduplicated Puppet code (#6842) has to be tested whether it does its work well.

Acceptance criteria:

  1. It has to work. :)
  2. The user MUST NOT have to provision the VM more than once.

Changesets

2014-09-30 09:41:34 +00:00 by aklimov 1396372

Add successor for '.vagrant-puppet/manifests/default.pp'

refs #7321

2014-11-25 10:45:05 +00:00 by aklimov c982ef4

Add successor for '.vagrant-puppet/manifests/default.pp'

refs #7321

Parent Task: #6842

@icinga-migration
Copy link
Author

Updated by aklimov on 2014-09-30 09:46:59 +00:00

  • Description updated

@icinga-migration
Copy link
Author

Updated by icinga-kanban on 2014-09-30 10:09:02 +00:00

Build !#89 triggered by commit 845de64 failed.

Branch: origin/feature/Test-deduplicated-Puppet-code-7321
Author: Alexander Klimov

@icinga-migration
Copy link
Author

Updated by jmeyer on 2014-10-20 11:10:54 +00:00

I've set up the box with the above commands and was able to login. Visiting /icingaweb/preference gave me an error: "You need to configure how to store preferences first."

When looking into the config.ini I noticed that the `preference' section was missing. This should be part of the default config populated by puppet.

Please have also a look at the default logging configuration, as we've changed in #6933 how logging is configured.

@icinga-migration
Copy link
Author

Updated by jmeyer on 2014-10-20 11:19:32 +00:00

Having no LDAP authentication anymore by default, is something i can live with. But why is the PHP module for LDAP not installed by default? This way we need to install it manually every time we'd like to test something LDAP specific.

date.timezone in PHP.ini should also being set to UTC by default, as it's a requirement for the new web installer.

@icinga-migration
Copy link
Author

Updated by icinga-kanban on 2014-10-20 12:31:48 +00:00

Build !#128 triggered by the commits 2a2c784, 570b31f, 5ca97c1, 6c59c22, 6dfefb0, 501aca4, d2d6532, 04a8df5, 97677ee, 985154d, a2772a1, d475ccd, f9e7e6d, 92b4f4f, 6494019, 937dbe2, 6124f98, c67e7c3, cc61dc2, db804f9, 6622042, 8f26f0d, 0825b2f, c48c7f4, 87a8765, f77a0e9, 4c23d98, afc97b1, fcc56c9, a9ae75b, 1b23fd7, 2126b8c, f954e8c, 8a8ce49, 421263a, 865ef76, d27d993, 5143c78, a012276, 97d2a92, e96065e, 36681bb, 2956d9e, fa96a1c, a3409b1, 9be52a9, 9db76bf, 8106fe4, 1a2d15b, 051128b, e192b93, 081d8ee, ee63dfd, ec9e9e8, fa2c9e4, 017d4b8, f315e10, ed91e11, 531448e, 74bd9b3, 64d41ac, f188bf6, bbee06b, fa9bd59, 0846915, 266936b, ebde422, 56a1af4, 46078f5, 1c7eb0d, 8008387, 8eb7db9, f190385, 9a57ddc, e6c674e, ac2aeca, 23c1097, ab8436d, dacbd32, 3ee9cd2, f7b7e02, 31609ec, 285d10e, b2f12e1, c73b291, cf997e9, c8a24f7, d7f8a78, 98fa84b, 45fc768, 032437c, 0ac7574, c4d13a7, cde3ec6, 3229e5e, 1710a50, 0b723bb, 3034ac5, c8f00be, 4258b13, e847770, 051c969, df18eab, 6625e8d, 70500be, 9601942, b107370, 085102e, 07a4b25, e784aa3, 2ff932f, 6c8f324, 4ce71ef, dc6562b, 44e5fe7, 7385be0, 6f1cb6f, 65203fd, dc05b2e, ef2f332, 2dd81d1, 264d818, 119e23e, 27650be, 274f2e7, 90dbcdb, 4b7096b, bea110d, 72a9a53, 5b5f553, c13823d, c51b052, 3c9859a, 5ba96c3, 7eb51f6, f1d3b72, 854e284, 7949102, af58732, d10afa1, f640874, 8afdc3d, 5cdcf36, 71ffd0e, 4d8b6dd, a2a6458, 0084fed, 0577665, cc2d06e, d091e21, 88b3b5d, c04768e, 44c4ec7, 0a81aee, 92d855e, 0e6e371, 18aad56, b8d2429, 0be8b97, b38ef9c, c21d7d1, 5c64771, f8edb4f, e2b5e05, 8bf6642, 9e0d665, f48094f, a18287d, 6bfab25, e50221f, 33e0fbb, 9b5c1f0, 17e04e9, 5d07b04, 5f13db7, 3a9774e, 4d784c6, 442f956, e5e806a, 7f0a2d5, 16bc0e5, 1932a91, 6c3c466, e42b7b3, 1d54c7f, 3f216f2, beecf16, c0e3447, aca5a2e, 8ceef96, 93777ac, ad53f7a, 3a5415f, 0e926ba, 583fd46, b6ac31d, 1df8076, b4faa01, 746e75e, 3845301, 9257159, 22771e6, c40ac6f, 1247fdc, 3615b96, 65564cd, 5757a6f, 171b366, 28a66c8, d9fbbca, a841b09, 83772c6, e7c0218, ceeb3a9, 6bde740, 0693e7c, 9d8f810, 40947ac, 7dbc83e, e4fccdd, 31978e1, cee3c32, a5c027b, d6377ca, c947ba3, 6525d69, f7f6bcc, c8ce108, 34bbe65, c8bdb70, 1137c01, f53519c, 0e7ca59, 12b4865, 930e6e7, 7215e27, ae9e5a4, fc72ddf, 7d21265, d021747, 8846f17, a83ddb9, a94e84f, 9cfd74d, 1358a5d, fe47441, 1c8f880, f57bb0f, 6f8fc3e, ed645b1, cb23ef3, 3d0a74b, bc1d00f, 8d10913, a45b17f, 439d189, 9d66cc9, c3c0043, a4da3b6, 49562e7, 5d1f2ad, 2025fb3, 906de4e, de11131, 774db9a, 1a4e908, 54a8342, 0b1d2bf, dd3901e, 5ce9bef, c7a4098, 5485ca8, 95c839a, 5464321, 1ba3df3, 2d86e6b, 5b14d8f, 539ab91, 338f549, aedc8cc, 39bb01b, 9bcd861, d4c4ab7, 8478ef3, 7143837, ae18016, c27c569, 4b210f9, c005ef4, d47951f, 8686074, 176a1a7, ff10cda, 5a100ea, 939f8b8, e8b70ff, deebb64, e2cdff6, 920c0b9, 0edd145, efd16ba, c5e35c9, cd29adc, cdbd24e, 5474c66, 010c5b4, fda4878, d2d69b9, 0e63e36, dae2967, 8cacd2f, cd8015f, 30965c2, 0822aff, 57dd9a6, f3e74f0, a01ee00, fb5685b, 8c82f1a, 4d353ac, 6324192, ae35650, 6fdc436, 2490993, ec46b36, 2ac4a85, e7eae87, 0e2e1bc, 364c7c0, e020dd3, 79b0ed6, bf23688, 486104d, 4ca1eaa, e132905, 6783d51, 0ff79b0, d3c2fc8, 65dd3f9, 98b2bce, 6b88f1c, 45cbd3f, 5c7999f, 52534a2, c93ab79, 7b221e2, 403f745, 5028ec7, a0b2c0b, 75a0c17, 2b879b3, bc05d2e, d1b1bc3, b81e965, 1106349, bb7972a, 7311ab9, e7da9c0, fe63ce6, e6bcda2, 4b1169c, f23bf91, eea43e9, be14844, a627571, f0a9927, 5203f82, a37e65b, d260c3f, c36e30a, e07f2a2, 4e1e845, 5e6ef57, 5a73811, d7ed6bd, a78d113, 02c3223, b743cf8, ed10e49, 488ea46, 644b3a1, c06db74, 50de21e, af5a326, 26d42da, a6bd802, 5f76521, af898cc, 26a78dc, c3731fa, 1902b4f, 0964316, 9c434fe, edf49d0, d8b468c, 235c4a8, 4fe46a2, fd912da, 936f65e, c8d61f7, 9cdd891, 8b2cc3a, 0bf0213, 25c9ee5, 75e09f2, a73e2ee, a9f0b95, f5ac592, 6ed3d5f, 7157b11, 5da14d3, f4ff2c9, 6ac471c, 8f0b989, 78a6175, 37fd3de, 58188c5, 802c095, 21cd503, 47ae4d7, 1de2d0c, 6690410, 630b36e, 0dce204, 1a1263d passed successfully.

Branch: origin/feature/Test-deduplicated-Puppet-code-7321
Author: Alexander Klimov

@icinga-migration
Copy link
Author

Updated by icinga-kanban on 2014-10-21 10:32:34 +00:00

Build !#130 triggered by the commits 414b669, e27724c, a4c4f50, 8222606, c2a3770, 81d49db, 3c10d47, 1bb3a58, 424cee6, 2b67683, aa56f30, d170cf0, d1228de, 79ade94, cee261b, 53608c8, a2e1663 passed successfully.

Branch: origin/feature/Test-deduplicated-Puppet-code-7321
Author: Alexander Klimov

@icinga-migration
Copy link
Author

Updated by aklimov on 2014-10-21 10:36:47 +00:00

@johannes:
I've fixed the 3 things you've reported.
Please re-create your VM after every such fix (for testing).

@icinga-migration
Copy link
Author

Updated by elippmann on 2014-10-21 12:22:04 +00:00

Hi Alex,

Would please drop the test branch and use the original feature branch instead for testing because cross merging the master and your feature branch into this branch without making any code change does not make sense and it's annoying :)

Cheers,
Eric

@icinga-migration
Copy link
Author

Updated by aklimov on 2014-10-22 08:18:11 +00:00

Hi Eric,

The testing branch is only for separating status quo and the successor and will be dropped (without merging) when the stuff has been tested enough.
At the same time I'll replace the default.pp w/ successor.pp in #6842.
I understand U at all, but it's quite annoying to resolve the default.pp's merge conflicts every time. ."

Best,
AK

@icinga-migration
Copy link
Author

Updated by jmeyer on 2014-10-29 12:46:34 +00:00

I know that it has been stated to boot up the vagrant machine with the --no-provision switch, but if I forget this the VM is broken which is not acceptable. The provisioning should be able to handle this somehow either by patching the current state or by just refusing to run at all.

@icinga-migration
Copy link
Author

Updated by aklimov on 2014-10-29 12:50:10 +00:00

@johannes:
This is just for testing – we'll have to "refactor" the `Vagrantfile' a little bit anyway before putting the stuff into master.

@icinga-migration
Copy link
Author

Updated by icinga-kanban on 2014-10-29 16:45:45 +00:00

Build !#158 triggered by the commits 86ae4b2, f3b4ec0, 39729c9, 2698516, eb9fe09, d9194c2, 5fc1f85, 8ee52f8, 319c5e7, e7ef182, 1cc1521, 6973492, 71e25f2, d04ecea, 99b00b7, 12cadb5, 38220af, f44b052, f68c591, ba12d7b passed successfully.

Branch: origin/feature/Test-deduplicated-Puppet-code-7321
Author: Alexander Klimov

@icinga-migration
Copy link
Author

Updated by icinga-kanban on 2014-10-30 12:03:30 +00:00

Build !#164 triggered by the commits 8ec002d, 1541bce, b7691c6 passed successfully.

Branch: origin/feature/Test-deduplicated-Puppet-code-7321
Author: Alexander Klimov

@icinga-migration
Copy link
Author

Updated by elippmann on 2014-10-30 12:49:19 +00:00

Hi Alex,

the test module is not enabled and phpunit is missing. Please verify that our test suite runs out of the box and please add the Icinga CGIs.

Cheers,
Eric

@icinga-migration
Copy link
Author

Updated by icinga-kanban on 2014-10-30 13:08:51 +00:00

Build !#166 triggered by the commits 6f86233, a7ab667, ba368f0 passed successfully.

Branch: origin/feature/Test-deduplicated-Puppet-code-7321
Author: Alexander Klimov

@icinga-migration
Copy link
Author

Updated by icinga-kanban on 2014-10-31 08:53:15 +00:00

Build !#170 triggered by the commits 4a0f6ac, 64445ae, 38c9758, 9471ced passed successfully.

Branch: origin/feature/Test-deduplicated-Puppet-code-7321
Author: Alexander Klimov

@icinga-migration
Copy link
Author

Updated by jmeyer on 2014-11-04 07:32:29 +00:00

PostgreSQL server is not ran at startup and seemingly not registered as service.

Edit: And is it intentional that there is no IDO for PostgreSQL?

@icinga-migration
Copy link
Author

Updated by elippmann on 2014-11-17 07:59:28 +00:00

icinga2-gui or something like that could not be installed.

@icinga-migration
Copy link
Author

Updated by mfriedrich on 2014-11-17 08:43:11 +00:00

We should move that into a separate repository, like requested in #7645

I'd like to take the opportunity to make 'puppet-icinga2' ready for anyone: users, testers, vagrant boxes, etc. - I figured that when working on the graylog2 demo boxes.

We should have a chat on that after the OSMC is over.

@icinga-migration
Copy link
Author

Updated by icinga-kanban on 2014-11-24 13:58:11 +00:00

Build !#309 triggered by the commits 438d9c0, 607b0fc, 30ee6c9 passed successfully.

Branch: origin/feature/Test-deduplicated-Puppet-code-7321
Author: Alexander Klimov

@icinga-migration
Copy link
Author

Updated by aklimov on 2014-11-25 12:44:17 +00:00

  • Description updated

@icinga-migration
Copy link
Author

Updated by elippmann on 2014-11-25 16:14:34 +00:00

  • MySQL and PostgreSQL Icinga Web 2 database is missing an administrative user (icingaadmin/icinga):
    INSERT INTO icingaweb_user (name, active, password_hash) VALUES ('icingaadmin', 1, '$1$JMdnEc9M$FW7yapAjv0atS43NkapGo/');
  • The logging path for Icinga Web 2 should be /var/log/icingaweb
  • Please re-add LDAP authentication

@icinga-migration
Copy link
Author

Updated by aklimov on 2014-11-26 12:08:03 +00:00

  • Status changed from New to Feedback

@icinga-migration
Copy link
Author

Updated by elippmann on 2014-12-19 08:30:05 +00:00

  • Status changed from Feedback to Closed

@icinga-migration icinga-migration added the enhancement New feature or improvement label Jan 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or improvement
Projects
None yet
Development

No branches or pull requests

1 participant