In D612#9490, @alken-orin wrote:Build has FAILED
Link to build: https://ci.nasqueron.org/job/test-notifications-phab/159/
See console output for more information: https://ci.nasqueron.org/job/test-notifications-phab/159/console
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Feed Advanced Search
Advanced Search
Advanced Search
Aug 20 2016
Aug 20 2016
dereckson added a project to T694: See how to organize storage/ folder regeneration: User-Sandlayth.
dereckson added a project to T769: Note the base image of every container launched in the Agora table : User-Sandlayth.
Next step is to provide a configuration with:
- the mapping job/project
- emulate what we would need at Jenkins plugin level: only fire notifications on job failed
dereckson committed rNOTIF8d700507fd7c: Add support for Jenkins notification plug-in (authored by dereckson).
Add support for Jenkins notification plug-in
Rebased. More doc.
@Rashk0 any advice?
Aug 19 2016
Aug 19 2016
Summary of previous discussions
dereckson moved T843: Handle GitHub pull_request event from Backlog to Next on the User-Dereckson board.
dereckson moved T844: Handle GitHub issue_comment event from Backlog to Next on the User-Dereckson board.
dereckson added a project to D618: Fix docker run command's MySQL environment in readme: github-pull-request.
dereckson committed rDPHAB6813590b6237: Fix docker run command's MySQL environment in readme (authored by Gaurav Arora <gauravsworld@gmail.com>).
Fix docker run command's MySQL environment in readme
dereckson closed D618: Fix docker run command's MySQL environment in readme by committing rDPHAB6813590b6237: Fix docker run command's MySQL environment in readme (authored by Gaurav Arora <gauravsworld@gmail.com>).
Submitted by Gaurav Arora at https://github.com/nasqueron/docker-phabricator/pull/4
dereckson retitled D618: Fix docker run command's MySQL environment in readme from to Fix docker run command's MySQL environment in readme.
dereckson moved T891: .+prenom/.+nom: check if item already exists from Next to Backlog on the User-Dereckson board.
dereckson moved T984: Pick a format for Notifications::$rawContent property from Backlog to Next on the User-Dereckson board.
dereckson added a project to T984: Pick a format for Notifications::$rawContent property: User-Dereckson.
Add notifications:payload command
dereckson closed T983: CLI command to get a notification payload from a service payload as Resolved by committing rNOTIFb42260e53626: Add notifications:payload command.
Better code documentation
In D617#9435, @dereckson wrote:There is an issue in logic code too.
[ErrorException] Undefined property: stdClass::$repository
Payload was already a notification, not a DockerHub payload.
More tests.
There is an issue in logic code too.
dereckson retitled D617: Add notifications:payload command from to Add notifications:payload command.
Clean commented out code
dereckson closed D526: Clean commented out code by committing rDWD92ff7cf74e2f: Clean commented out code.
Avoid shadow variable name
dereckson closed D569: Avoid shadow variable name by committing rDWDa991d5d09084: Avoid shadow variable name.
Aug 18 2016
Aug 18 2016
dereckson added inline comments to rDPHAB82a577551716: Merge 48b2ffbf1481dbab46100e6d5bf9999ec34826df into….
dereckson committed rDPHABf1d02969c688: Improve code readability: 8 Mb, restart operation (authored by xcombelle).
Improve code readability: 8 Mb, restart operation
dereckson closed D602: Improve code readability: 8 Mb, restart operation by committing rDPHABf1d02969c688: Improve code readability: 8 Mb, restart operation (authored by xcombelle).
Rebased against master.
That would be set -e
There is no demonstrated advantage to use that, the json parsing issue in D347 was a false issue.
dereckson awarded D615: Explain why mysql.host is written in hard a Piece of Eight token.
dereckson committed rGROVEb3ae033adfc2: Define explicitly method visibility (authored by dereckson).
Define explicitly method visibility
dereckson retitled D616: Define explicitly method visibility from to Define explicitly method visibility.
According Bradipo from Freenode tcl channel, parsing is okay (and produces a correct dict) with json 1.3.3.
JSON makes a lot of sense, there is no need to parse twice a JSON payload, once for our notification format, once for the source payload.
Aug 17 2016
Aug 17 2016
dereckson committed rKBROKER545d2db54f4c: Explicitely set visibility to public methods (authored by dereckson).
Explicitely set visibility to public methods
dereckson retitled D614: Explicitely set visibility to public methods from to Explicitely set visibility to public methods.
According Bradipo from TCL, parsing is okay (and produces a correct dict) with json 1.3.3.
According Bradipo from TCL, parsing is okay (and produces a correct dict) with json 1.3.3.
dereckson added a project to T984: Pick a format for Notifications::$rawContent property: architecture.
Currently, D347 is, as far as I know, the first piece of code consuming raw content.
Aug 16 2016
Aug 16 2016
The error occurs at the JSON parser level, invoked by json::json2dict $payload in get_image_from_docker_payload proc.
dereckson moved T983: CLI command to get a notification payload from a service payload from Backlog to Working on on the Notifications center board.
dereckson retitled D604: only touch .initialized on success of service restart from only touch .initialized on succes of service restart to only touch .initialized on success of service restart.
To fail on error is probably a good idea to avoid to say .initialized when an error occurs
Rebased against master.
More tests
dereckson moved T981: Define a policy for acceptable vhosts on Eglide from Backlog to Needs Review / Blocked / Waiting on the User-Dereckson board.
Aug 15 2016
Aug 15 2016
Fix issue when status is absent, like for job started notifications.
Publish JUnit XML report
Nasqueron DevCentral · If it had been much bigger the moon would have had a core of ice. · Powered by Phabricator