Coder Social home page Coder Social logo

Comments (5)

Dhanush0369 avatar Dhanush0369 commented on June 27, 2024

@leecalcote I have understood the issue. I know to use go package "gopkg.in/yaml.v3" for parsing the manifest and ignore redundant YAML document boundary markers.
If kuberntes client-go library is preferable then I can complete the task using that as well. Please assign me.

from meshery.

leecalcote avatar leecalcote commented on June 27, 2024

Thanks for stepping up, @Dhanush0369 πŸ‘ @theBeginner86, please be aware of @Dhanush0369's impending activity here. // @MUzairS15

from meshery.

theBeginner86 avatar theBeginner86 commented on June 27, 2024

@leecalcote
Unless I'm mistaken, there might be some misunderstanding here.
Any user can very well import a manifest with multiple boundary markers --- inside of it.

The issue you are facing is due to the use of manifests that has CRDs unrecognized by your Meshery Server, that is, missing registered components for Kustomize, ExternalSecret etc.
Secondly, there is are couple syntax error around line 892 that has misplacement of boundary marker.

Screenshot 2024-01-25 at 8 00 53β€―PM

// @Dhanush0369

from meshery.

theBeginner86 avatar theBeginner86 commented on June 27, 2024

For testing you can use the following manifest that has multiple boundary marker (including redundant ones). It works as expected.

apiVersion: v1
kind: Service
metadata:
  name: my-service
spec:
  selector:
    app.kubernetes.io/name: MyApp
  ports:
    - protocol: TCP
      port: 80
      targetPort: 9376
---
apiVersion: v1
kind: Service
metadata:
  name: my-service
spec:
  selector:
    app.kubernetes.io/name: MyApp
  ports:
    - protocol: TCP
      port: 80
      targetPort: 9376
---
---
apiVersion: v1
kind: Service
metadata:
  name: my-service
spec:
  selector:
    app.kubernetes.io/name: MyApp
  ports:
    - protocol: TCP
      port: 80
      targetPort: 9376
---
apiVersion: v1
kind: Service
metadata:
  name: my-service
spec:
  selector:
    app.kubernetes.io/name: MyApp
  ports:
    - protocol: TCP
      port: 80
      targetPort: 9376
---
---
apiVersion: v1
kind: Service
metadata:
  name: my-service
spec:
  selector:
    app.kubernetes.io/name: MyApp
  ports:
    - protocol: TCP
      port: 80
      targetPort: 9376

from meshery.

leecalcote avatar leecalcote commented on June 27, 2024

Thank you for proving this. πŸ‘

I wonder if @Dhanush0369 can help with other enhancements to support Kustomize.

from meshery.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    πŸ–– Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. πŸ“ŠπŸ“ˆπŸŽ‰

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❀️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.