Skip to content

feat(queueconfig): adding entity and basic interface for queue config#50

Merged
kevinlnew merged 6 commits into
mainfrom
kevin.new/queue-config
Feb 24, 2026
Merged

feat(queueconfig): adding entity and basic interface for queue config#50
kevinlnew merged 6 commits into
mainfrom
kevin.new/queue-config

Conversation

@kevinlnew
Copy link
Copy Markdown
Contributor

@kevinlnew kevinlnew commented Feb 24, 2026

Summary

feat(queueconfig): adding entity and basic interface for queue config

why not create an implementation of the extension?

  • This is open to users to implement or have in-house solutions; generally, uberfx services can plug-and-play configfx with minimal extra configs to provide the struct, so the same idea holds for others.
    • I'm open to creating a basic yaml reader if necessary.

Why?
Services require stored configuration for queue config data, readable by queue name (which is passed by request).

What?

  • Added a basic config entity for queue config
  • Added a basic interface for getting a queueconfig by name.

Test Plan

Basic unit testing for constructor for queue config entity.

Issues

@kevinlnew kevinlnew force-pushed the kevin.new/queue-config branch 2 times, most recently from 9f031dc to 8736411 Compare February 24, 2026 01:56
Comment thread entity/queueconfig/queueconfig.go Outdated
Comment thread entity/queueconfig/queueconfig.go Outdated
Comment thread entity/queueconfig/queueconfig_test.go Outdated
Comment thread entity/queueconfig/queueconfig_test.go Outdated
Comment thread entity/queueconfig/queueconfig.go Outdated
Comment thread entity/queueconfig/queueconfig.go Outdated
Comment thread entity/queueconfig/queueconfig.go Outdated
Comment thread entity/queueconfig/queueconfig.go Outdated
Comment thread extension/queueconfig/queueconfig.go
@kevinlnew kevinlnew force-pushed the kevin.new/queue-config branch from 8736411 to c3411b1 Compare February 24, 2026 19:09
@kevinlnew kevinlnew marked this pull request as ready for review February 24, 2026 19:16
@kevinlnew kevinlnew requested review from a team as code owners February 24, 2026 19:16
Comment thread entity/queue_config.go Outdated
Comment thread extension/queueconfig/queueconfig.go Outdated
@kevinlnew kevinlnew force-pushed the kevin.new/queue-config branch from 697eb30 to 6fbbb1d Compare February 24, 2026 22:44
@kevinlnew kevinlnew force-pushed the kevin.new/queue-config branch from 6fbbb1d to c7cd6b4 Compare February 24, 2026 23:13
@kevinlnew kevinlnew merged commit 23e6f5c into main Feb 24, 2026
8 checks passed
@behinddwalls behinddwalls deleted the kevin.new/queue-config branch June 2, 2026 18:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants