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

Added missing rules on router config, fixed ordering of multiple rule… #1616

Merged
merged 1 commit into from
Jul 25, 2016

Conversation

pdube
Copy link
Contributor

@pdube pdube commented Jul 22, 2016

…s, removed duplicate rules, added fix for network stats, added a check for b64 decoding (to pad incorrect b64). Also added a catch exception to be logged on the configure main.

https://issues.apache.org/jira/browse/CLOUDSTACK-9430
https://issues.apache.org/jira/browse/CLOUDSTACK-9431
https://issues.apache.org/jira/browse/CLOUDSTACK-9435
https://issues.apache.org/jira/browse/CLOUDSTACK-9440

…s, removed duplicate rules, added fix for network stats, added a check for b64 decoding (to pad incorrect b64). Also added a catch exception to be logged on the configure main.
@swill
Copy link
Contributor

swill commented Jul 25, 2016

CI RESULTS

Tests Run: 85
  Skipped: 0
   Failed: 0
   Errors: 0
 Duration: 4h 07m 15s

Associated Uploads

/tmp/MarvinLogs/DeployDataCenter__Jul_22_2016_22_13_50_JE2HDL:

/tmp/MarvinLogs/test_network_ZB64IG:

/tmp/MarvinLogs/test_vpc_routers_JE0NWR:

Uploads will be available until 2016-09-24 02:00:00 +0200 CEST

Comment created by upr comment.

@asfgit asfgit merged commit 9ab6762 into apache:master Jul 25, 2016
asfgit pushed a commit that referenced this pull request Jul 25, 2016
Added missing rules on router config, fixed ordering of multiple rules, removed duplicate rules, added fix for network stats, added a check for b64 decoding (to pad incorrect b64). Also added a catch exception to be logged on the configure main.

https://issues.apache.org/jira/browse/CLOUDSTACK-9430
https://issues.apache.org/jira/browse/CLOUDSTACK-9431
https://issues.apache.org/jira/browse/CLOUDSTACK-9435
https://issues.apache.org/jira/browse/CLOUDSTACK-9440

* pr/1616:
  Added missing rules on router config, fixed ordering of multiple rules, removed duplicate rules, added fix for network stats, added a check for b64 decoding (to pad incorrect b64). Also added a catch exception to be logged on the configure main.

Signed-off-by: Will Stevens <williamstevens@gmail.com>
@karuturi
Copy link
Member

@swill missing required LGTMs..

@swill
Copy link
Contributor

swill commented Jul 26, 2016

@karuturi yes, but it is a blocking issue for RC2 and I need to get RC2 out. Do you have problems with anything?

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.

4 participants