Github user abhinandanprateek commented on the issue:
https://github.com/apache/cloudstack/pull/1667
@rhtyd created another PR for 4.9
https://github.com/apache/cloudstack/pull/1670
---
If your project is set up for it, you can reply to this email and have your
reply appear on
GitHub user abhinandanprateek opened a pull request:
https://github.com/apache/cloudstack/pull/1670
CLOUDSTACK-9481: Convert MyISAM table to InnoDB for consistency
You can merge this pull request into a Git repository by running:
$ git pull
Thanks, Koushik.
Have created a PR to make the engine type consistent for all tables.
InnoDB allows for replication, that is all the more reason that we change the
engine type of these few tables to InnoDB.
Regards,
-abhi
PR: https://github.com/apache/cloudstack/pull/1667
From: Koushik Das
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1669
During build step, the l10n JS files are built:
> [INFO] --- exec-maven-plugin:1.2.1:exec (generate-sources) @
cloud-client-ui ---
> Exporting compiled dictionary:
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1667
LGTM. @abhinandanprateek can you edit the base branch of this PR to 4.9.
Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1663
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
Github user rhtyd closed the pull request at:
https://github.com/apache/cloudstack/pull/1669
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
GitHub user rhtyd reopened a pull request:
https://github.com/apache/cloudstack/pull/1669
Make CloudStack JSP-free
CLOUDSTACK-9842: Make UI JSP Free
We use some JSP file just for translation of strings in the UI. This is
achievable purely in JavaScript. This
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1663
Thanks all, based on the test results and reviews from this PR with the
marvin test and the original PR #1653 from where the main changes were taken,
I'll go ahead and merge this now.
---
If
Github user borisstoyanov closed the pull request at:
https://github.com/apache/cloudstack/pull/1668
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1668
Yes @serg38 you're tight. Closing it now.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user nvazquez commented on the issue:
https://github.com/apache/cloudstack/pull/1660
Thanks @rhtyd!
I rebased branch '4.9' and pushed some changes based on your last comment.
I had to change visibility of `com.cloud.utils.stringUtils.UTF8` from private
to public.
I
Marco, I heard rumors that you were trying to reschedule for November. If
that is the case, I would suggest you push the reschedule farther in the
future for a couple reasons:
- ApacheCon is in November already in Europe so that makes travel to both
events very difficult.
- People traveling from
It's traditional. We wait for the last minute to get our talk proposals
in for conferences.
Well, the last minute has arrived. The CFP for ApacheCon Seville closes
on September 9th, which is less than 2 weeks away. It's time to get your
talks in, so that we can make this the best ApacheCon yet.
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1663
Thanks @borisstoyanov
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user serg38 commented on the issue:
https://github.com/apache/cloudstack/pull/1668
@borisstoyanov This PR can be probably closed since the issue with correct
ping responses is being addressed globally in PR1651
---
If your project is set up for it, you can reply to this email
GitHub user rhtyd opened a pull request:
https://github.com/apache/cloudstack/pull/1669
Make CloudStack JSP-free
CLOUDSTACK-9842: Make UI JSP Free
We use some JSP file just for translation of strings in the UI. This is
achievable purely in JavaScript. This
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1663
LGTM, I was able to build and test the PR. Found a little issue with the
tests and fixed it.
Here are the tests results:
```
$ nosetests --with-xunit
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1621
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1621
Thanks @milamberspace for the PR, I'll merge it now.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
I cannot think of any. Most probably in all cases the engine was not explicitly
specified and the default got used. As per below default was myisam before
mysql 5.5.5.
https://dev.mysql.com/doc/refman/5.5/en/storage-engine-setting.html
Thanks,
Koushik
From: Abhinandan Prateek
Github user abhinandanprateek commented on the issue:
https://github.com/apache/cloudstack/pull/1640
@marcaurele @koushik-das When the MS thinks that the VM is down, it issues
a stop command. This is done to clear up the resources on management server db
tied up for that VM. Now it
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1667
@abhinandanprateek can you reopen this for 4.9 branch, since it's a bugfix.
LTS users should all be on innodb to avoid any replication issues, please also
move the changes suitably to
Hi,
Was there any specific reason that some tables in cloudstack schema use
MyISAM engine.
I know of “ quota_account” that I created and put in the diff table type
unknowingly. I assume such is the case with other such tables –
load_balancer_cert_map, monitoring_services, nic_ip_alias,
GitHub user abhinandanprateek opened a pull request:
https://github.com/apache/cloudstack/pull/1667
CLOUDSTACK-9481: Convert MyISAM table to InnoDB for consistency
You can merge this pull request into a Git repository by running:
$ git pull
Github user milamberspace closed the pull request at:
https://github.com/apache/cloudstack/pull/1621
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
GitHub user milamberspace reopened a pull request:
https://github.com/apache/cloudstack/pull/1621
[CLOUDSTACK-9444] Fix a little issue from PR1610 if the db.properties file
hasn't EOL câ¦
â¦haracter at the end of file
And some improvements about the dir/file using
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1665
With new db schema/versioning and upgrade changes in #1654 we can merge
this with some additional changes once #1654 is merged and fwd-merged.
---
If your project is set up for it, you can reply
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1666
LGTM. @jburwell we should have this in 4.9.1.0-RC1.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1666
Test test_07_1_egress_fr7 failure was VM deploy failure. Re-run this
particular test again, to enusre it succeeds.
---
If your project is set up for it, you can reply to this email and
Github user wido commented on the issue:
https://github.com/apache/cloudstack/pull/1645
@kiwiflyer That seems good indeed. If we are not setting a source format
this looks good indeed.
LGTM
---
If your project is set up for it, you can reply to this email and have your
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1666
@JayapalUradi could you please review
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
GitHub user murali-reddy opened a pull request:
https://github.com/apache/cloudstack/pull/1666
CLOUDSTACK-9480: Egress Firewall: Incorrect use of Allow/Deny for ICMP
When 'default egress policy' is set to 'allow' in the network offering, any
egress rule that is added will 'deny'
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1654
@jburwell I've fixed the issue failing db upgrade, it was a minor
comparison issue in the beginning of the upgrade process. On my local env the
upgrade worked as expected. LGTM.
---
If your
Github user milamberspace commented on the issue:
https://github.com/apache/cloudstack/pull/1636
Thanks for reply @rhtyd I will wait the merge of #1654 before force to
re-check this PR by travis/jenkins
---
If your project is set up for it, you can reply to this email and have your
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1654
@rhtyd I have updated this patch to properly support four position version
numbers, as well as, remove the requirement that all versions have a
``DbUpgrade`` class defined and/or an upgrade
36 matches
Mail list logo
