cloudstack-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <>
Subject [jira] [Commented] (CLOUDSTACK-9198) VR gets created in the disabled POD
Date Fri, 03 Mar 2017 21:54:45 GMT


ASF GitHub Bot commented on CLOUDSTACK-9198:

Github user GabrielBrascher commented on the issue:
    @anshul1886 I would like to raise the point previously discussed by me and @rafaelweingartner.
I think that we should pay attention if the change of user and caller will really do the job.
So far I do not see how this PR changes the behavior.
    Basically this code changes two parameters in startVirtualRouter [_callerUser_ and _caller_
when calling startVirtualRouter(router, callerUser, caller, routerDeploymentDefinition.getParams())].
However, those parameters are only used inside startVirtualRouter when calling the method
start(router, user, caller, params, null).
    if (router.getRole() != Role.VIRTUAL_ROUTER || !router.getIsRedundantRouter()) {
                return start(router, user, caller, params, null);
    The problem is that the method **start** does not use either the _user_ and the _caller_
parameters in the overridden implementation (the one that you are using).
        protected DomainRouterVO start(DomainRouterVO router, final User user, final Account
caller, final Map<Param, Object> params, final DeploymentPlan planToDeploy)
                throws StorageUnavailableException, InsufficientCapacityException, ConcurrentOperationException,
ResourceUnavailableException {
            s_logger.debug("Starting router " + router);
            try {
                _itMgr.advanceStart(router.getUuid(), params, planToDeploy, null);
            } catch (final OperationTimedoutException e) {
                throw new ResourceUnavailableException("Starting router " + router + " failed!
" + e.toString(), DataCenter.class, router.getDataCenterId());
            if (router.isStopPending()) {
      "Clear the stop pending flag of router " + router.getHostName()
+ " after start router successfully!");
                router = _routerDao.persist(router);
            // We don't want the failure of VPN Connection affect the status of
            // router, so we try to make connection
            // only after router start successfully
            final Long vpcId = router.getVpcId();
            if (vpcId != null) {
            return _routerDao.findById(router.getId());
    Sorry, but I can't see how your code alters the behavior as intended. Can you please show
that by changing the parameters _user_ and _caller_ you are changing the behavior?
    Thanks in advance.

> VR gets created in the disabled POD
> -----------------------------------
>                 Key: CLOUDSTACK-9198
>                 URL:
>             Project: CloudStack
>          Issue Type: Bug
>      Security Level: Public(Anyone can view this level - this is the default.) 
>            Reporter: Anshul Gangwar
>            Assignee: Anshul Gangwar
> VR gets created in the disabled POD

This message was sent by Atlassian JIRA

View raw message