Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

Monitoring and various documentation fixes #111

Merged
merged 11 commits into from
Mar 31, 2020
Merged

Monitoring and various documentation fixes #111

merged 11 commits into from
Mar 31, 2020

Conversation

banzo
Copy link
Member

@banzo banzo commented Mar 13, 2020

No description provided.

banzo and others added 8 commits February 13, 2020 11:22
@banzo
Update INSTALL.md
2b97772
@banzo
CI/CD with minikube
9607ba2
@AyadiAmen
ldap documentation
29c34a6 
elastic-stack ldap documentation
@banzo
Details on JHub LDAP documentation
bfc7df7
@banzo
typo in jhub ldap doc
573fb0e
@banzo
Helm 3 - Remove deprecated tiller ref, updated traefik install version
ffcc85c
@banzo
Deprecate tiller (doc update)
bceabbb
@AyadiAmen @banzo
@alexnuttinck @fzalila
Feature/zabbix (#110)
90383fd 
* Documentation links fix (#95)

* fix the password to connect to Adminer (#99)

* Update logging doc  (#103)

* Update INSTALL.md

* CI/CD with minikube

* ldap documentation

elastic-stack ldap documentation

* Details on JHub LDAP documentation

* typo in jhub ldap doc

Co-authored-by: Sebastien Dupont <dupont.sebastien@gmail.com>

* Zabbix doc: cetic/helm-fadi#27

* Update LOGGING.md

Co-authored-by: Sebastien Dupont <dupont.sebastien@gmail.com>
Co-authored-by: Alexandre Nuttinck <alexandre.nuttinck@cetic.be>
Co-authored-by: Faiez Zalila <fzalila@users.noreply.github.com>
@banzo banzo changed the title Develop Monitoring and various documentation fixes Mar 13, 2020
@alexnuttinck alexnuttinck self-requested a review March 24, 2020 14:15
@alexnuttinck alexnuttinck added documentation enhancement New feature or request labels Mar 24, 2020
alexnuttinck
alexnuttinck reviewed Mar 25, 2020
View reviewed changes
Copy link
Contributor

@alexnuttinck alexnuttinck left a comment

Choose a reason for hiding this comment

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

@AyadiAmen please, see my comments before we merge.

doc/MONITORING.md Outdated
Make sure to enable zabbix in the `values.yaml` file, then to access the front-end use the following command:

```
minikube service zabbix-web
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it not fadi-zabbix-web ? @AyadiAmen

Copy link
Contributor

Choose a reason for hiding this comment

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

@AyadiAmen you miss my comment I think. Is it not fadi-zabbix-web ?

Copy link
Contributor

Choose a reason for hiding this comment

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

@alexnuttinck sorry i didn't see this comment, no it's not fadi-zabbix-web, it's zabbix-web. the release name is not included in the service name.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ok thanks, I think it should be change in the next releases of zabbix in order to have the same format of service names.

Copy link
Contributor

Choose a reason for hiding this comment

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

@alexnuttinck it's done, i added the release name to the service name and now it's fadi-zabbix-web .

Copy link
Contributor

Choose a reason for hiding this comment

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

perfect!

Copy link
Contributor

Choose a reason for hiding this comment

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

yes, the version should be 0.1.2.

doc/MONITORING.md
minikube service zabbix-web
```

The default username/password are `Admin`/`zabbix`, once connected make sure the zabbix-server is working on the global view, to see the received metrics ( or graphs ) head to the tab `Latest data` under `monitoring` and then click on select on **host groups** and **hosts** and choose your `zabbix servers`.
Copy link
Contributor

Choose a reason for hiding this comment

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

@AyadiAmen not the ldap creds by default?

Copy link
Contributor

Choose a reason for hiding this comment

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

@alexnuttinck no, for now it is the default creds, after linking it manually with ldap it changes to the default ldap creds.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ok, thanks.

alexnuttinck
alexnuttinck approved these changes Mar 27, 2020
View reviewed changes
Copy link
Contributor

@alexnuttinck alexnuttinck left a comment •
edited

Choose a reason for hiding this comment

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

Ok for me. @banzo

@banzo banzo merged commit ae0b32b into master Mar 31, 2020
banzo added a commit that referenced this pull request Nov 15, 2021
@banzo @AyadiAmen
@alexnuttinck @fzalila
Userguide update (#135)
0a552b6 
* Monitoring and various documentation fixes (#111)
* Update INSTALL.md
* CI/CD with minikube
* ldap documentation
* elastic-stack ldap documentation
* Details on JHub LDAP documentation
* typo in jhub ldap doc
* Helm 3 - Remove deprecated tiller ref, updated traefik install version
* Feature/zabbix (#110)
* Documentation links fix (#95)
* fix the password to connect to Adminer (#99)
* Update logging doc  (#103)

Co-authored-by: Sebastien Dupont <dupont.sebastien@gmail.com>

* Zabbix doc: cetic/helm-fadi#27

* Update LOGGING.md

Co-authored-by: Sebastien Dupont <dupont.sebastien@gmail.com>
Co-authored-by: Alexandre Nuttinck <alexandre.nuttinck@cetic.be>
Co-authored-by: Faiez Zalila <fzalila@users.noreply.github.com>

* update zabbix service name

* fix #105 (#115)

Co-authored-by: Amen Ayadi <ayadi.amen@gmail.com>
Co-authored-by: Amen AYADI <37482206+AyadiAmen@users.noreply.github.com>
Co-authored-by: Alexandre Nuttinck <alexandre.nuttinck@cetic.be>
Co-authored-by: Faiez Zalila <fzalila@users.noreply.github.com>

* Simplified description

* Remove outdated Helm architecture schema

* Delete helm-architecture.png

* removed erroneous dashboard link, added troubleshooting guidance (logs howto and FAQ link)

* Add kubectl logs/events guidance to bug report template

* FAQ: add logs/debug instructions, change FADI acronym explanation, add links to how to extend FADI

* Delete helm/tiller directory

* Add ingress note in FAQ, fixes #121

* typo

Co-authored-by: Amen Ayadi <ayadi.amen@gmail.com>
Co-authored-by: Amen AYADI <37482206+AyadiAmen@users.noreply.github.com>
Co-authored-by: Alexandre Nuttinck <alexandre.nuttinck@cetic.be>
Co-authored-by: Faiez Zalila <fzalila@users.noreply.github.com>
banzo added a commit that referenced this pull request Nov 15, 2021
@banzo @AyadiAmen
@alexnuttinck @fzalila @Sellto
Deprecate develop branch (#140)
0bdb378 
* Feature/zabbix (#110)
* Documentation links fix (#95)
* fix the password to connect to Adminer (#99)
* Update logging doc  (#103)
* fix #105 (#115)
* Feature/rancher proxmox (#117)
* Documentation/binderhub (#112)
* group management (#113)
* Feature/tests (#123) - Setup the testing framework for fadi. Add automated testing of the services using Jest and Puppeteer, test cases and scenarios specifications and implementation.
* Usermanagement documentation (Nifi) + Tensorflow use case (#130)
* NiFi - LDAP Documentation
* Feature/seldon - ML models management (#122)
* Add new flag to helm repo add to overwrite the cetic chart repo if already present (#133)
* Add zakaria2905 to contributors
* Userguide update (#135)
* Monitoring and various documentation fixes (#111)
* Update INSTALL.md
* CI/CD with minikube
* ldap documentation
* elastic-stack ldap documentation
* Details on JHub LDAP documentation
* Helm 3 - Remove deprecated tiller ref, updated traefik install version
* Feature/zabbix (#110)
* Documentation links fix (#95)
* fix the password to connect to Adminer (#99)
* Update logging doc  (#103)
* Zabbix doc: cetic/helm-fadi#27
* fix #105 (#115)
* fix #121

Co-authored-by: Sebastien Dupont <sebastien.dupont@cetic.be>
Co-authored-by: Amen Ayadi <ayadi.amen@gmail.com>
Co-authored-by: Alexandre Nuttinck <alexandre.nuttinck@cetic.be>
Co-authored-by: Faiez Zalila <fzalila@users.noreply.github.com>
Co-authored-by: Sellto <selleslagh.tom@gmail.com>
Co-authored-by: Faiez Zalila <faiez.zalila@cetic.be>
Co-authored-by: Rami Sellami <rami.sellami@cetic.be>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@alexnuttinck alexnuttinck alexnuttinck approved these changes

@AyadiAmen AyadiAmen AyadiAmen left review comments

Assignees

@banzo banzo

Labels
documentation enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@banzo @alexnuttinck @AyadiAmen @fzalila