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

chore: add ocha_monitoring module #412

Closed
wants to merge 1 commit into from

Conversation

lazysoundsystem
Copy link
Contributor

Refs: OPS-10130, OPS-10119

If there are any other steps needed for configuration, would be good to add them to the README at https://github.com/UN-OCHA/ocha_monitoring

Copy link
Contributor

Coverage Report

Totals Coverage

Copy link
Contributor

Build output

Composer Validate `success`

PHP Lint `success`

Docker Build `success`

Environment Setup `success`

Site Install `success`

PHP Code Sniffer `success`

Software Versions

```PHP 8.2.15 (cli) (built: Jan 20 2024 14:17:05) (NTS)
Copyright (c) The PHP Group
Zend Engine v4.2.15, Copyright (c) Zend Technologies
with Zend OPcache v8.2.15, Copyright (c), by Zend Technologies
with Xdebug v3.3.1, Copyright (c) 2002-2023, by Derick Rethans
Composer version 2.7.1 2024-02-09 15:26:28
```

Drupal Logs

``````

*Pusher: @lazysoundsystem, Action: \`pull_request\`, Workflow: \`Run tests\`*`;

Copy link
Contributor

@attiks attiks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Too many sensors in config, did you run drush deploy
The install should delete all sensors except for ours

Out you can do drush eval"ocha_monitoring_enforce_monitors();" && drush -y cex

@attiks
Copy link
Contributor

attiks commented Feb 29, 2024

Refs: OPS-10130, OPS-10119

If there are any other steps needed for configuration, would be good to add them to the README at https://github.com/UN-OCHA/ocha_monitoring

Steps are in https://humanitarian.atlassian.net/browse/OPS-9765

@lazysoundsystem
Copy link
Contributor Author

Closing this as the slimmed down version #414 is now merged

@lazysoundsystem lazysoundsystem deleted the OPS-10119-monitoring branch March 4, 2024 17:40
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.

2 participants