You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by ab...@apache.org on 2019/04/09 15:22:47 UTC

[mesos] branch 1.6.x updated (34ffae5 -> 80c0673)

This is an automated email from the ASF dual-hosted git repository.

abudnik pushed a change to branch 1.6.x
in repository https://gitbox.apache.org/repos/asf/mesos.git.


    from 34ffae5  Added MESOS-9529 to 1.6.3 CHANGELOG.
     new ee630c7  Fix `link::lo()` to deal with `None` value.
     new 80c0673  Added MESOS-9707 to the 1.6.3 CHANGELOG.

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 CHANGELOG                       | 1 +
 src/linux/routing/link/link.cpp | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)


[mesos] 01/02: Fix `link::lo()` to deal with `None` value.

Posted by ab...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

abudnik pushed a commit to branch 1.6.x
in repository https://gitbox.apache.org/repos/asf/mesos.git

commit ee630c7f8db2e59ca92f286f246bd1b5b4f39d21
Author: Pavel Kirillov <49...@users.noreply.github.com>
AuthorDate: Tue Apr 9 15:44:24 2019 +0200

    Fix `link::lo()` to deal with `None` value.
    
    Some network links may cause `link::internal::test(link, IFF_LOOPBACK)`
    to be `None`.
    `Link::lo` should deal with `None` values of test var.
    
    This closes #330
---
 src/linux/routing/link/link.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/linux/routing/link/link.cpp b/src/linux/routing/link/link.cpp
index 9dc30c5..bff172d 100644
--- a/src/linux/routing/link/link.cpp
+++ b/src/linux/routing/link/link.cpp
@@ -99,7 +99,7 @@ Result<string> lo()
     Result<bool> test = link::internal::test(link, IFF_LOOPBACK);
     if (test.isError()) {
       return Error("Failed to check the flag on link: " + link);
-    } else if (test.get()) {
+    } else if (test.isSome() && test.get()) {
       return link;
     }
   }


[mesos] 02/02: Added MESOS-9707 to the 1.6.3 CHANGELOG.

Posted by ab...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

abudnik pushed a commit to branch 1.6.x
in repository https://gitbox.apache.org/repos/asf/mesos.git

commit 80c0673f1844771b6f4baf0433ba9ac27479e722
Author: Andrei Budnik <ab...@mesosphere.com>
AuthorDate: Tue Apr 9 17:22:35 2019 +0200

    Added MESOS-9707 to the 1.6.3 CHANGELOG.
---
 CHANGELOG | 1 +
 1 file changed, 1 insertion(+)

diff --git a/CHANGELOG b/CHANGELOG
index 8d22141..392fd25 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -7,6 +7,7 @@ Release Notes - Mesos - Version 1.6.3 (WIP)
   * [MESOS-9529] - `/proc` should be remounted even if a nested container set `share_pid_namespace` to true.
   * [MESOS-9564] - Logrotate container logger lets tasks execute arbitrary commands in the Mesos agent's namespace.
   * [MESOS-9692] - Quota may be under allocated for disk resources.
+  * [MESOS-9707] - Calling link::lo() may cause runtime error
 
 ** Improvement
   * [MESOS-8880] - Add minimum capabilities in the master.