- 
        Couldn't load subscription status. 
- Fork 250
Docs : Add Known Issues and Limitations in FAQ section #635
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Docs : Add Known Issues and Limitations in FAQ section #635
Conversation
b2f01bf    to
    570de89      
    Compare
  
    >> - Current list of known issues >> - Limitations mentioned >> - Additional information also included
570de89    to
    253f0bc      
    Compare
  
    | Hi @jakobmoellerdev , could you take a look at my PR and let me know if any changes are required? | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for opening a PR for this!
        
          
                website/docs/docs/faq.md
              
                Outdated
          
        
      | - **Workaround**: Manual recreation of deleted resources required | ||
| - **GitHub Issue**: [#520](https://github.com/kro-run/kro/issues/520) | ||
|  | ||
| **2. RGD instances stuck in "IN_PROGRESS" status** | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe any issue under investigation should not be part of a known issues section, we have tons of those and updating this section should be reserved only for the most frequently observed and problemnatic ones
        
          
                website/docs/docs/faq.md
              
                Outdated
          
        
      | - **Impact**: ResourceGraphDefinition CRD and APIs are not solidified and subject to change | ||
| - **Recommendation**: Use only for development and testing environments | ||
|  | ||
| **2. API stability** | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the same as lacking production readiness
        
          
                website/docs/docs/faq.md
              
                Outdated
          
        
      | - **Impact**: Upgrades may require manual migration of existing ResourceGraphDefinitions | ||
| - **Recommendation**: Plan for potential breaking changes during upgrades | ||
|  | ||
| **3. Resource dependency complexity** | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is not a limitation, but a UX challenge, should not be part of this section IMO
        
          
                website/docs/docs/faq.md
              
                Outdated
          
        
      | - **Workaround**: Manual validation of schema changes required | ||
| - **GitHub Issue**: [#186](https://github.com/kro-run/kro/issues/186) | ||
|  | ||
| **4. SNS topic creation failures** | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very specific bug report, not sure if this should be adopted here
        
          
                website/docs/docs/faq.md
              
                Outdated
          
        
      | **3. Missing detection of CRD breaking changes** | ||
|  | ||
| - **Impact**: Kro does not detect breaking changes in ResourceGroup schemas when modified | ||
| - **Status**: Feature enhancement needed | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why is this under feature enhancement needed when there when we even have an upcoming change here? #352
        
          
                website/docs/docs/faq.md
              
                Outdated
          
        
      | **1. Kro does not detect deleted resources** | ||
|  | ||
| - **Impact**: Deleted resources from kro instances are not detected and recreated automatically | ||
| - **Status**: Open bug | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this issue has not been confirmed
        
          
                website/docs/docs/faq.md
              
                Outdated
          
        
      |  | ||
| This section is community-maintained: | ||
|  | ||
| - Found a recurring issue? Check if it's already tracked in our [GitHub Issues](https://github.com/kro-run/kro/issues) | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this section should be replaced with a link to https://github.com/kro-run/kro/blob/main/CONTRIBUTING.md#reporting-bugsfeature-requests
        
          
                website/docs/docs/faq.md
              
                Outdated
          
        
      |  | ||
|  | ||
|  | ||
| **Stay Updated** | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is duplicate maintenance of https://github.com/kro-run/kro?tab=readme-ov-file#community-participation
| @jakobmoellerdev Thank you for the guidance! I realized I had misinterpreted a few points earlier, which led to some mistakes. I’ve revised the section by removing duplicates and unconfirmed items, merging overlaps, and keeping only confirmed issues and key limitations. Requesting your review — happy to make further changes if needed. | 
| solidified and highly subject to change. | ||
|  | ||
|  | ||
| 6. **What are the current known issues and limitations?** | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe we can introduce a separate page for this next to FAQ and link it here, considering this will likely maintained separatey
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jakobmoellerdev Ok.
|  | ||
| This section documents the most frequently observed and problematic issues reported by the kro community. | ||
|  | ||
| **Current Known Issues** | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Im missing a reference to the Roadmap and Dynamic Watches
| - **Impact**: Wrong API endpoints used with Fleet and other GitOps tools causing integration failures | ||
| - **Status**: Confirmed integration problem affecting GitOps workflows | ||
| - **Workaround**: Manual intervention or alternative deployment strategies | ||
| - **GitHub Issue**: [#524](https://github.com/kro-run/kro/issues/524) | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Im not sure if this is really the right issue / relevance. we really only deal with an inflection issue here no?
| - **Impact**: Some resources may not be created when ResourceGraphDefinitions have complex dependencies, particularly with topological sorting issues | ||
| - **Status**: Confirmed issue affecting multiple users | ||
| - **Workaround**: Simplify resource dependencies or manually verify all resources are created | ||
| - **GitHub Issue**: [#225](https://github.com/kro-run/kro/issues/225) | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you verify that #225 is still valid? I think this could just be a leftover
| **2. CEL expression observability** | ||
| - **Limitation**: No visibility into CEL expression execution costs, evaluation performance, or debugging information | ||
| - **Impact**: Difficult to troubleshoot and optimize complex ResourceGraphDefinitions when CEL expressions are expensive or fail | ||
| - **Status**: Architectural gap in observability framework | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I dont think this is a framework issue, rather something we just didnt implement in the project yet. There was even a PR but that was rejected for formality reasons
| - **Impact**: When Platform teams modify ResourceGraphDefinition schemas, there's no migration path for existing instances created by Developer teams | ||
| - **Status**: Major architectural limitation blocking production adoption | ||
| - **Workaround**: Manual recreation of all instances when RGD schemas change | ||
| - **Community Impact**: Most frequently reported limitation preventing enterprise use | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why have community impact and impact? this is redundant for me
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was going through this blog
I see the following big ideas the team is going to work on.
Resource group versioning
Resource change management
External references and adoption
Building collections (with for loop-like support)
Garbage collection
Enhanced status reporting
Enhanced security integrations
Addressing RGD versioning is a definite requirement.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was not the point of my comment. My point was that you have a statement "Impact" and "Community Impact" which looks / reads redundantly
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apart from that the blog you linked is from 12 NOV 2024 so I dont think you should take this as a source of truth.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for the first point I see recently there is proposal for it.
| @jakobmoellerdev Could you please give me some idea that how to structure this section and i tried to incorporate the relevant things but it's not matching the requirements for this section. Also tell me that if i have to create a separate page for this section as it will require management after this also. | 
| I think I gave a rather detailed review here so im wondering what you need on top of this? | 
| @jakobmoellerdev Thank you for the detailed review. I was occupied with some events over the past week, but I’ll be resuming work on it tomorrow. | 
| Unknown CLA label state. Rechecking for CLA labels. Send feedback to sig-contributor-experience at kubernetes/community. /check-cla | 
| 
 | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for working on this! I noticed some of the items listed might be better suited for our issue tracker rather than documentation. Several appear to be bugs or features in development rather than permanent limitations.
I'm thining ...perhaps we could mention design docs or upcoming features where we're actively seeking community feedback? That would be more forward looking and valuable for users who want to contribute to the project's direction, rather than listing current bugs that might give the wrong impression about stability.
| @Kavish-12345 can you please sign the CLA? | 
| /hold | 
| closing to continue the discussion in the github issue #621 (comment) | 
Summary
Adds a new "Known Issues and Limitations" section to the FAQ page as requested in the community call decision.
Changes
Addresses
Fixes #621