Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package okteto for openSUSE:Factory checked 
in at 2024-01-07 21:42:19
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/okteto (Old)
 and      /work/SRC/openSUSE:Factory/.okteto.new.28375 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "okteto"

Sun Jan  7 21:42:19 2024 rev:67 rq:1137402 version:2.23.2

Changes:
--------
--- /work/SRC/openSUSE:Factory/okteto/okteto.changes    2023-12-13 
18:34:27.721294756 +0100
+++ /work/SRC/openSUSE:Factory/.okteto.new.28375/okteto.changes 2024-01-07 
21:42:21.540507963 +0100
@@ -1,0 +2,7 @@
+Sun Jan 07 18:08:07 UTC 2024 - opensuse_buildserv...@ojkastl.de
+
+- Update to version 2.23.2:
+  * fix: do not fail if error retrieving envs from image (#4099)
+    (#4100)
+
+-------------------------------------------------------------------

Old:
----
  okteto-2.23.1.obscpio

New:
----
  okteto-2.23.2.obscpio

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ okteto.spec ++++++
--- /var/tmp/diff_new_pack.2Z1rRz/_old  2024-01-07 21:42:22.608546825 +0100
+++ /var/tmp/diff_new_pack.2Z1rRz/_new  2024-01-07 21:42:22.608546825 +0100
@@ -1,7 +1,7 @@
 #
 # spec file for package okteto
 #
-# Copyright (c) 2023 SUSE LLC
+# Copyright (c) 2024 SUSE LLC
 #
 # All modifications and additions to the file contributed by third parties
 # remain the property of their copyright owners, unless otherwise agreed
@@ -19,7 +19,7 @@
 %define __arch_install_post export NO_BRP_STRIP_DEBUG=true
 
 Name:           okteto
-Version:        2.23.1
+Version:        2.23.2
 Release:        0
 Summary:        Develop your applications directly in your Kubernetes Cluster
 License:        Apache-2.0

++++++ _service ++++++
--- /var/tmp/diff_new_pack.2Z1rRz/_old  2024-01-07 21:42:22.640547989 +0100
+++ /var/tmp/diff_new_pack.2Z1rRz/_new  2024-01-07 21:42:22.640547989 +0100
@@ -3,10 +3,10 @@
     <param name="url">https://github.com/okteto/okteto</param>
     <param name="scm">git</param>
     <param name="exclude">.git</param>
-    <param name="revision">2.23.1</param>
+    <param name="revision">2.23.2</param>
     <param name="versionformat">@PARENT_TAG@</param>
     <param name="changesgenerate">enable</param>
-    <param name="match-tag">2.23.1</param>
+    <param name="match-tag">2.23.2</param>
   </service>
   <service name="set_version" mode="manual">
     <param name="basename">okteto</param>

++++++ _servicedata ++++++
--- /var/tmp/diff_new_pack.2Z1rRz/_old  2024-01-07 21:42:22.660548717 +0100
+++ /var/tmp/diff_new_pack.2Z1rRz/_new  2024-01-07 21:42:22.664548862 +0100
@@ -1,6 +1,6 @@
 <servicedata>
 <service name="tar_scm">
                 <param name="url">https://github.com/okteto/okteto</param>
-              <param 
name="changesrevision">5736dda29a09fd253fa2ed5a0574ebdb4ff097c6</param></service></servicedata>
+              <param 
name="changesrevision">cc3e71019898e5d6e875646736486ae067808060</param></service></servicedata>
 (No newline at EOF)
 

++++++ okteto-2.23.1.obscpio -> okteto-2.23.2.obscpio ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/okteto-2.23.1/cmd/up/exec.go 
new/okteto-2.23.2/cmd/up/exec.go
--- old/okteto-2.23.1/cmd/up/exec.go    2023-12-12 16:27:56.000000000 +0100
+++ new/okteto-2.23.2/cmd/up/exec.go    2023-12-22 17:38:36.000000000 +0100
@@ -220,9 +220,15 @@
                return nil, err
        }
 
-       imageEnvs, err := 
eg.imageEnvsGetter.getEnvsFromImage(apps.GetDevContainer(app.PodSpec(), 
"").Image)
+       svcImage := apps.GetDevContainer(app.PodSpec(), "").Image
+       imageEnvs, err := eg.imageEnvsGetter.getEnvsFromImage(svcImage)
        if err != nil {
-               return nil, err
+               // we must not fail since the image may not be found because 
the platform
+               // of the image executed in the pod might not match the local 
platform used
+               // for the search.
+               reason := fmt.Sprintf("Could not to retrieve environment 
variables from the image '%s'", svcImage)
+               oktetoLog.Debugf("%s: %s", reason, err.Error())
+               oktetoLog.Warning(reason)
        }
        envs = append(envs, imageEnvs...)
 
@@ -297,7 +303,7 @@
        return envs, nil
 }
 
-func (cmg *configMapGetter) getEnvsFromConfigMap(ctx context.Context, name 
string, namespace string, client kubernetes.Interface) ([]string, error) {
+func (cmg *configMapGetter) getEnvsFromConfigMap(ctx context.Context, name, 
namespace string, client kubernetes.Interface) ([]string, error) {
        var envs []string
 
        cmName := pipeline.TranslatePipelineName(name)
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/okteto-2.23.1/cmd/up/exec_test.go 
new/okteto-2.23.2/cmd/up/exec_test.go
--- old/okteto-2.23.1/cmd/up/exec_test.go       2023-12-12 16:27:56.000000000 
+0100
+++ new/okteto-2.23.2/cmd/up/exec_test.go       2023-12-22 17:38:36.000000000 
+0100
@@ -43,11 +43,11 @@
        envs []string
 }
 
-func (f *fakeGetter) getEnvsFromDevContainer(ctx context.Context, spec 
*apiv1.PodSpec, name string, namespace string, client kubernetes.Interface) 
([]string, error) {
+func (f *fakeGetter) getEnvsFromDevContainer(ctx context.Context, spec 
*apiv1.PodSpec, name, namespace string, client kubernetes.Interface) ([]string, 
error) {
        return f.envs, f.err
 }
 
-func (f *fakeGetter) getEnvsFromConfigMap(ctx context.Context, name string, 
namespace string, client kubernetes.Interface) ([]string, error) {
+func (f *fakeGetter) getEnvsFromConfigMap(ctx context.Context, name, namespace 
string, client kubernetes.Interface) ([]string, error) {
        return f.envs, f.err
 }
 
@@ -224,6 +224,38 @@
                                },
                        },
                },
+               {
+                       name: "error retrieving envs from image",
+                       client: fake.NewSimpleClientset(&appsv1.StatefulSet{
+                               ObjectMeta: metav1.ObjectMeta{
+                                       Name:      "test",
+                                       Namespace: "test",
+                               },
+                               Spec: appsv1.StatefulSetSpec{
+                                       Template: v1.PodTemplateSpec{
+                                               Spec: v1.PodSpec{
+                                                       Containers: 
[]v1.Container{
+                                                               {
+                                                                       
VolumeMounts: []v1.VolumeMount{
+                                                                               
{
+                                                                               
        MountPath: "/data",
+                                                                               
},
+                                                                       },
+                                                               },
+                                                       },
+                                               },
+                                       },
+                               },
+                       }),
+                       fakeConfigMapEnvsGetter: fakeGetter{},
+                       fakeSecretEnvsGetter:    fakeGetter{},
+                       fakeImageEnvsGetter:     fakeGetter{err: 
assert.AnError},
+                       expectedEnvs:            []string{},
+                       dev: &model.Dev{
+                               Name:      "test",
+                               Namespace: "test",
+                       },
+               },
        }
 
        for _, tt := range tests {
@@ -300,33 +332,6 @@
                                },
                        }),
                },
-               {
-                       name: "error retrieving envs from image",
-                       client: fake.NewSimpleClientset(&appsv1.StatefulSet{
-                               ObjectMeta: metav1.ObjectMeta{
-                                       Name:      "test",
-                                       Namespace: "test",
-                               },
-                               Spec: appsv1.StatefulSetSpec{
-                                       Template: v1.PodTemplateSpec{
-                                               Spec: v1.PodSpec{
-                                                       Containers: 
[]v1.Container{
-                                                               {
-                                                                       
VolumeMounts: []v1.VolumeMount{
-                                                                               
{
-                                                                               
        MountPath: "/data",
-                                                                               
},
-                                                                       },
-                                                               },
-                                                       },
-                                               },
-                                       },
-                               },
-                       }),
-                       fakeConfigMapEnvsGetter: fakeGetter{},
-                       fakeSecretEnvsGetter:    fakeGetter{},
-                       fakeImageEnvsGetter:     fakeGetter{err: 
assert.AnError},
-               },
        }
 
        for _, tt := range tests {

++++++ okteto.obsinfo ++++++
--- /var/tmp/diff_new_pack.2Z1rRz/_old  2024-01-07 21:42:23.016561671 +0100
+++ /var/tmp/diff_new_pack.2Z1rRz/_new  2024-01-07 21:42:23.020561817 +0100
@@ -1,5 +1,5 @@
 name: okteto
-version: 2.23.1
-mtime: 1702394876
-commit: 5736dda29a09fd253fa2ed5a0574ebdb4ff097c6
+version: 2.23.2
+mtime: 1703263116
+commit: cc3e71019898e5d6e875646736486ae067808060
 

++++++ vendor.tar.gz ++++++
/work/SRC/openSUSE:Factory/okteto/vendor.tar.gz 
/work/SRC/openSUSE:Factory/.okteto.new.28375/vendor.tar.gz differ: char 5, line 
1

Reply via email to