Skip to content

Initial import of script for nightly build notification#24

Merged
smlng merged 3 commits intoRIOT-OS:masterfrom
miri64:notify-nightlies
Aug 27, 2019
Merged

Initial import of script for nightly build notification#24
smlng merged 3 commits intoRIOT-OS:masterfrom
miri64:notify-nightlies

Conversation

@miri64
Copy link
Copy Markdown
Member

@miri64 miri64 commented Jun 6, 2019

This provides a script to notify about the build results of the nightly builds (only via E-Mail for now, but it is extendible). Only the results in output.html are considered for now, as I don't know how the Robot Framework stuff is integrated (would be great if it was PR'd here and in murdock-html :-)). The configuration for the notification engine hooks into the murdock.toml configuration file and should be straight forward to understand. Here is an example configuration (just comment out what you need to connect to the SMTP server).

[notifications]                     # section must exist to enable notifications but can remain empty
    branches = ["master"]           # optional
    only_changes = true             # report only when result status changes, option (default: false)

[notifiers]
    [notifiers.mail]
    server = "localhost"            # required
    # username = "<smtp username>"
    # password = "<smpt password>"
    # port = 587
    # security = "none"             # may be none, ssl, or starttls
    receiver = "[email protected]"
    sender = "Murdock <[email protected]"

If the murdock.toml is not at /etc/murdock.toml the path can also be provided via the MURDOCK_CONFIG environment variable.

It requires toml (which should already be preinstalled with Murdock) and bs4 (aka BeautifulSoup, to parse the output.html and maybe later the robot.xml)

@miri64 miri64 requested review from kaspar030 and smlng June 6, 2019 15:13
@tcschmidt
Copy link
Copy Markdown
Member

@smlng we should add the HIL summary results here.

@miri64
Copy link
Copy Markdown
Member Author

miri64 commented Jun 6, 2019

@smlng we should add the HIL summary results here.

This can also be done as a follow-up.

@miri64 miri64 force-pushed the notify-nightlies branch from d54c9a2 to cf130b8 Compare June 6, 2019 18:05
@miri64
Copy link
Copy Markdown
Member Author

miri64 commented Jun 6, 2019

Rebased and adapted for current master.

@miri64
Copy link
Copy Markdown
Member Author

miri64 commented Jun 6, 2019

I reworked the configuration a bit (see OP for an updated example).

@smlng
Copy link
Copy Markdown
Member

smlng commented Aug 16, 2019

@miri64 I'm okay with merging this and handle the deployment on ci.riot-os.org.

To summarise what needs to be done:

  1. Merge this PR
  2. Adapt configuration, i.e., /etc/murdock.toml as you showed above
  3. Deploy updated master on webserver
  4. Cross fingers

Correct?

@kaspar030
Copy link
Copy Markdown
Contributor

Correct?

that should do it!

@smlng smlng self-assigned this Aug 16, 2019
@miri64
Copy link
Copy Markdown
Member Author

miri64 commented Aug 27, 2019

Correct?

Yepp.

@smlng
Copy link
Copy Markdown
Member

smlng commented Aug 27, 2019

okay merge then, I'll look into the deployment later today

@smlng smlng merged commit 3df6a3d into RIOT-OS:master Aug 27, 2019
@smlng smlng mentioned this pull request Nov 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants