All,
Per our release schedule [1], the 4.8, 4.9, and master branches are frozen for
testing. There are some straggling PRs that Rajani and I are working to merge.
Is it acceptable to everyone that for the next two (2) weeks, all PRs require
not only 2 LGTMs, but approval by Rajani or I to be
All,
I had intended to complete updates the release schedule [1] early last week,
but a variety of community and $dayjob activities distracted me. I apologize
for the delay.
I have updated the dates to reflect the recent schedule slips. Also, per our
recent discussions, I have changed the LT
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1683
Just ran in to this issue. Fix LGTM.
---
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 featu
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1678
@abhinandanprateek how difficult would it be to change the base branch of
this PR to 4.8?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1678
@abhinandanprateek could you please create a ticket to refactor `Script` to
use `Duration` internally? Also, are there any component tests that need to be
run in addition to smoke tests to ver
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1678#discussion_r80408774
--- Diff:
core/src/com/cloud/agent/resource/virtualnetwork/VirtualRoutingResource.java ---
@@ -374,9 +375,9 @@ private Answer execute(AggregationControl
Github user abhinandanprateek commented on the issue:
https://github.com/apache/cloudstack/pull/1678
@jburwell updated Script to use Duration as timeout.
---
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 doe
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1680
Packaging result: âcentos6 âcentos7 âdebian. JID-26
---
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 do
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1681
@murali-reddy is there a way to create or update a Marvin test case to
verify this fix?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub a
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1681
@jburwell a Trillian-Jenkins test job (centos7 mgmt + vmware55u3) has been
kicked to run smoke tests
---
If your project is set up for it, you can reply to this email and have your
reply
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1681
@blueorangutan test centos7 vmware55u3
---
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
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1681
Packaging result: âcentos6 âcentos7 âdebian. JID-25
---
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 do
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1677
@serg38 I will kick a blueorganutan run to regression test VMware 5.5. Are
the tests necessary to verify this PR part of the smoke test run or do we need
to run additional component tests?
-
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1638
@rhtyd All configurations run by blueorganutan reported errors. Could you
please investigate/address these issues?
---
If your project is set up for it, you can reply to this email and have y
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@murali-reddy 8 errors occurred in the VMware test run. Could you please
investigate these failures?
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1680
@murali-reddy could you please add and/or update Marvin test cases to
verify this fix?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1600
@mike-tutkowski @syed needs to resolve the conflicts before we can proceed
with package and functional testing.
---
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/1680
@blueorangutan package
---
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 wis
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1680
@jburwell a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply a
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1680
@blueorganutan package
---
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 wis
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1690
@rhtyd any objections to this change?
---
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
Github user mike-tutkowski commented on the issue:
https://github.com/apache/cloudstack/pull/1600
@syed @karuturi @jburwell Just wondering how we're doing on this one. Is
there anything else that Syed needs to do here? Thanks!
---
If your project is set up for it, you can reply to th
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1690
LGTM for code
---
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, o
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1690
@blueorganutan package
---
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 wis
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1684
@abhinandanprateek do we need to add a note to the upgrade documentation
and/or refine the upgrade script to protect against a change in
`user.password.encoders.exclude` and `user.password.enco
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1681
@jburwell a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply a
Github user abhinandanprateek commented on the issue:
https://github.com/apache/cloudstack/pull/1684
@pdion891 You need to look at user.authenticators.order and
user.authenticators.exclude and see if they will provide same authenticator in
chain as the one configured pre-upgrade.
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1681
@blueorangutan package
---
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 wis
Github user abhinandanprateek commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1678#discussion_r80404569
--- Diff:
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtPvlanSetupCommandWrapper.java
---
@@ -87,7 +88,7 @@ pub
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1681
@murali-reddy have you had a chance to address @sateesh-chodapuneedi
comments?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. I
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1678#discussion_r80404113
--- Diff:
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtPvlanSetupCommandWrapper.java
---
@@ -87,7 +88,7 @@ public Answe
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1684
@DagSonsteboSB @rhtyd this issue looks similar one we had with another
upgrade. Do you agree? If so, do you recall what we did to fix it?
---
If your project is set up for it, you can reply
Github user pdion891 commented on the issue:
https://github.com/apache/cloudstack/pull/1684
upgrade from 4.7.0 to 4.9.0.1 not working with same error:
```
WARN [o.a.c.s.a.PBKDF2UserAuthenticator] (qtp204585401-22:ctx-9e4c5615)
(logid:1ca9f551) The stored password for admin isn
Github user pdion891 commented on the issue:
https://github.com/apache/cloudstack/pull/1684
I've applied this fix to 4.7.0 and I got the upgrade from 4.4.4 working.
but when I upgrade 4.4.4 to this PR I end with inability to logon into the UI.
---
If your project is set up for it,
Github user pdion891 commented on the issue:
https://github.com/apache/cloudstack/pull/1684
Unfortunately, Authentication is not working after an upgrade. the
configration table as been updated but login is not working anymore.
Upgrade test from 4.4.4 to the PR (4.9), I can
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1684
@pdion891 yes, it will be merged to 4.8 and forward merged to 4.9 and
master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user pdion891 commented on the issue:
https://github.com/apache/cloudstack/pull/1684
I will test this PR, allow me few days..
This will be ported into master as well right?
Thanks
---
If your project is set up for it, you can reply to this email and have your
repl
GitHub user milamberspace opened a pull request:
https://github.com/apache/cloudstack/pull/1690
Update L10N resource files with 4.10 strings from Transifex (20160925)
Please merge the #1619 before.
And please merge this current PR before 4.10.0.0 release.
cc
Github user milamberspace commented on the issue:
https://github.com/apache/cloudstack/pull/1669
Not look good to me, I've found these issues:
The quote ' in French UI (and other language using quote) is precede by a
backslash:
