[HOLD for payment 2023-03-31] Plaid process starts again after adding a bank account using the deep link

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Sign out if already signed in
Open https://staging.new.expensify.com/settings/payments/add-bank-account
Sign in with any account
Complete the plaid process and add a new account
On Add bank account page, select the bank account and click Save & continue
Click Continue (notice the Plaid process starts again)
Click the x icon to close the Plaid window (notice the infinite loader)

Expected Result:
Plaid process should not start again
Actual Result:
Plaid process starts again and an infinite loader is displayed
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] MacOS / Chrome / Safari

Version Number: 1.2.72-0
Reproducible in staging?: y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/219260877-699fa0e8-434e-4611-83f6-fdac405da655.mov
https://user-images.githubusercontent.com/43996225/219260894-ef844a31-5b46-46a0-ae64-666e7669024d.mp4
Expensify/Expensify Issue URL:
Issue reported by: @adeel0202
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1676478964850549
View all open jobs on GitHub

Triggered auto assignment to @anmurali (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

@anmurali Huh... This is 4 days overdue. Who can take care of this?

I think this might have the same root cause as https://github.com/Expensify/App/issues/15202

@anmurali 6 days overdue. This is scarier than being forced to listen to Vogon poetry!

@anmurali 10 days overdue. Is anyone even seeing these? Hello?

I think this is a dupe of https://github.com/Expensify/App/issues/15202
Going to track the fix and then we can test this again to see if it gets fixed as well

@anmurali this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

@anmurali Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@anmurali Eep! 4 days overdue now. Issues have feelings too...

@anmurali 10 days overdue. I'm getting more depressed than Marvin.

The fix for this has been merged and deployed to staging: https://github.com/Expensify/App/pull/15679

@anmurali 12 days overdue. Walking. Toward. The. Light...

The solution for this issue has been :rocket: deployed to production :rocket: in version 1.2.88-2 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:

https://github.com/Expensify/App/pull/15679

If no regressions arise, payment will be issued on 2023-03-31. :confetti_ball:
After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.

[ ] External issue reporter
[ ] Contributor that fixed the issue
[ ] Contributor+ that helped on the issue and/or PR

As a reminder, here are the bonuses/penalties that should be applied for any External issue:

Merged PR within 3 business days of assignment - 50% bonus
Merged PR more than 9 business days after assignment - 50% penalty

This issue has the same root cause as https://github.com/Expensify/App/issues/15202 and the BZ checklist there applies to this one as well: https://github.com/Expensify/App/issues/15202#issuecomment-1483204801

Sorry, @adeel0202. I missed that you need to be compensated for reporting the bug. Re-opening so @anmurali can initiate the payment.

@anmurali We're all set to payout the reporting bonus to @adeel0202, thanks!


report: Update cilium version

What type of PR is this?

/kind documentation
/area conformance
What this PR does / why we need it:
As Cilium v1.16.0 is released, it's better to use the official version instead of RC.
Which issue(s) this PR fixes:

NA
Does this PR introduce a user-facing change?:

NONE

/approve


How to clear playback history of videos.

How to clear playback history of videos played? Currently, if you play a video and stop in the middle of it, the app remembers where it was and resumes playback from that point; instead from the beginning of the video. This is also a privacy issue since it shows which videos where played with a specific client. Where is this information stored and how to delete it?
(I am using the Appimage version in Linux)

This is stored in your local browser history at this time. You'd need to clear browser data to clear it out.

The desktop app is an electron app which means it's based on a web browser. It has an internal cache.
There is a button on the bottom of the settings page that clears cache.
It may be a good idea to provide more granular clearing buttons.

I've just tried it (clear the app cache), and it still remembers the timestamp of the video.
As for the privacy thing, it's for the same reasons you clear the history in any browser. It may not concern many people, but for those that it does, I don't see any problem if there was a tool for it.

I'm also interested in how play history is recorded. I've added a checkbox to give the user an option to hide content they have already watched. But, play history is not sync'd across devices. My desktop and my laptop don't reflect the same play history.
Where is this data stored?


fix: Hide buttons in message header bar while editing topic #24888

While editing a topic, other buttons in the header unnecessary for this action temporary disappear, until the user cancels or accepts the operation.
Fixes: Hide buttons in message header bar while editing topic #24888
https://user-images.githubusercontent.com/92218971/228526776-3ffddabb-7840-46c5-8242-9b93681fd614.mov

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Thanks!
Please clean up your commit history and post again to request a review. See here for guidelines.


Add a separate copyright Policy Page

Is your feature request related to a problem? Please describe.
To comply with legal standards and protect the intellectual property rights associated with the content on chaoss, I propose the addition of a dedicated Copyright Policy Page.
Describe the solution you'd like
Add a separate page to the platform detailing the copyright policy. This page will outline the rights and responsibilities related to the content, including how to report infringements and the consequences of violating intellectual property rights.
Describe alternatives you've considered
A possible alternative could be integrating copyright information within the existing terms of service. However, this may make the document too long and difficult to navigate, reducing clarity for users seeking copyright-related information.
Additional context
giving the sample of other  platforms' copyright policy pages (e.g., Google, YouTube, or GitHub) as a reference for how the page could be structured.
Example:
YouTube Copyright Policy
kindly assign me this issue under Hacktoberfest 2024

Is your feature request related to a problem? Please describe. To comply with legal standards and protect the intellectual property rights associated with the content on chaoss, I propose the addition of a dedicated Copyright Policy Page.
Describe the solution you'd like Add a separate page to the platform detailing the copyright policy. This page will outline the rights and responsibilities related to the content, including how to report infringements and the consequences of violating intellectual property rights.
Describe alternatives you've considered A possible alternative could be integrating copyright information within the existing terms of service. However, this may make the document too long and difficult to navigate, reducing clarity for users seeking copyright-related information.
Additional context giving the sample of other platforms' copyright policy pages (e.g., Google, YouTube, or GitHub) as a reference for how the page could be structured.
Example:
YouTube Copyright Policy
kindly assign me this issue under Hacktoberfest 2024

Thanks for your interest in contributing to CHAOSS Education project. I will tag @germonprez @sgoggins and @ElizabethN  to review your proposal.
I will assign this issue to you as soon as the topic is approved here or in another CHAOSS project.
Thank you.

Thank you for considering my contribution to the CHAOSS Education project! I appreciate the opportunity to collaborate and look forward to the review and feedback from @germonprez, @sgoggins, @ElizabethN, and others. I'll wait for the topic approval and am ready to begin once the issue is assigned. Let me know if there's anything further you'd like me to provide in the meantime.
Thank you again!

Thank you for considering my contribution to the CHAOSS Education project! I appreciate the opportunity to collaborate and look forward to the review and feedback from @germonprez, @sgoggins, @ElizabethN, and others. I'll wait for the topic approval and am ready to begin once the issue is assigned. Let me know if there's anything further you'd like me to provide in the meantime.
Thank you again!

In the meantime,  kindly go through other hacktoberfest issues to pick up anyone you like to work on.

Thank you for the suggestion! I'll definitely take a look at the other Hacktoberfest issues and find one that interests me. Looking forward to contributing!

Thank you for the suggestion! I'll definitely take a look at the other Hacktoberfest issues and find one that interests me. Looking forward to contributing!

Great tag me on the issue, I will assign it to you

Thanks for bringing this forward @Yeshajagad -- Could this be something that is part of this page: https://github.com/chaoss/community/blob/main/governance/data-policies.md
It may be easier to include a comment here than creating an entirely new page.
We also do have a License document that declares copyright issues as well.

Thank you for your feedback, @germonprez.
I appreciate the suggestion. Including this within the existing data-policies.md page sounds like a good approach to ensure consistency. I will review the current content in the data policies and the License document to identify where a brief comment about copyright issues could be added.
I'll share my thoughts soon, and if everything aligns well, I can propose the necessary changes via a pull request.
Thanks again!

Thank you for your feedback, @germonprez.
I appreciate the suggestion. Including this within the existing data-policies.md page sounds like a good approach to ensure consistency. I will review the current content in the data policies and the License document to identify where a brief comment about copyright issues could be added.
I'll share my thoughts soon, and if everything aligns well, I can propose the necessary changes via a pull request.
Thanks again!

@Yeshajagad I think I can go ahead and close this issue since you have been directed to the right repo.


Added FW rules for IPv6

Using ip6tables provider

This looks reasonable to me.  As we do not have CI for this module, have you been able to do a basic sanity test?  Note fir usage in tripleo-heat-templates, this may not truly be needed, as they set manage_fw to false (thanks @EmilienM for pointing that out).  That said, this seems still generally useful for when we do want to have the firewall managed.

I've tested against a pacemaker deployment (3 nodes) initially with a dual ipv4/ipv6 stack. And then disabling the ipv4 (which I need for provisioning), leaving only the ipv6 stack. The cluster, configured for ipv6, works fine, the FW rules are created properly.


Meeting url outated?

Seems like the google meet room has changed?
https://github.com/scikit-build/scikit-build-core/blob/f461bb139589871a0c8f2cba823f9d8965066824/README.md?plain=1#L18

I believe it got canceled an hour or so before the meeting today and that causes the Google meet link to report that. (Why it was canceled, I'm not sure, I was ready for one.)

Ahh, I wonder if it's because @thewtex left Kitware and he was the owner of the meeting?

Hmm, probably also good to have a link for the google calendar event of the meeting, or something to track the status

We should set up a room, I think. I created on at https://meet.google.com/kmh-snuy-dhv that we could use.

I'll join that one for a bit if you'd like to briefly meet (@jcfr if you are around, too)
If the room doesn't go away after we use it, we could make it the standard room.

Stuck trying to authenticate, be back in a bit.

Unclear is Google Meet updated their ID or not ... in the meantime, the correct reference has been set in https://github.com/scikit-build/scikit-build-core/pull/931
See also https://meet.google.com/qbx-nkkm-mup


QA: Map Search API down

Time category: https://its-pro.ucsd.edu/browse/MA-78
MapSearchService v2 in QA is not working. Investigate and resolve
This is done when:

[x] MapSearchService v2 API in QA has been synced from Prod
[x] MapSearchService v2 API in QA returns expected results which mirror Prod results for the same query

Resources:
QA API: https://us-west-2.console.aws.amazon.com/apigateway/home?region=us-west-2#/apis/vkil1id5r5/resources/l3bfnoe6i3
Prod API: https://us-west-2.console.aws.amazon.com/apigateway/home?region=us-west-2#/apis/xgu9qa7gx4/resources/yj647q

resolved


Add Seti theme

Coverage remained the same at 60.979% when pulling eef94af176a5cecbab1caa1f03bfb7e1d0435612 on aeschli/seti-theme into 538cd9ded5a471fbb20920f01eff9c8df4b83ee9 on release/1.5.

Also delivered to master


Move to SUIT org

Have you considered this?
The more generic ones like form, module and card could be very useful alongside the other core components as they aren't overly opinionated and I'm sure projects ends up recreating something similar.
Wasn't sure if that would disrupt your lerna setup though.
Otherwise including a link in the suit README will be a nice addition.

Yep I just used Lerna because it makes it easy to manage the packages.
As I started to use them on a real project I found some bugs and things that I would like to fix
For example:

now I agree that Form should be FormField - I might rewrite it and release v2 or make a separate component
Module it is a bit overkill I'd rather use spacing utilites (I will open source them).
Haven't used Card but it should work fine.

Interesting, okay that makes sense.
I suppose Module is in the same bracket as spacing utils. Maybe it's too tricky to have a one size fits all component for spacing. I remember the spacing classes got deprecated for a similar reason.

Module requires a lot of boilerplate, you need a UL-like structure even when you just want to add space between two elements.
I am using a scale n=1...6 of spacing utils with the following formula
.u-margin0 { margin: 0 !important }
.u-marginN { margin: calc(var(--u-margin) * 2^(n-1))  !important }

and it works great.

... not to mention that have responsive companion which makes it super easy to tweak things at different screen sizes.

it seems that this guy read my mind and make a framework and a talk https://twitter.com/giuseppegurgone/status/778843098852036608 :D
I am "tired" of writing css when I am asked to build features.


[WATCH@LIVE]UPDATES!!] NZ All Blacks vs AU Wallabies Rugby LIVE Broadcast ON Tv Channel 21 September 2024

Where To Watch Wallabies vs All Blacks Live stream FREE Bledisloe Cup 2024 The 2024 Bledisloe Cup series between the Wallabies and the All Blacks is set to be an exciting showdown! The series will be played across two Tests in September as part of The Rugby Championship.
🔴📺Link📲👉 Wallabies vs All Blacks Rugby Live
🔴📺Link📲👉 The Rugby Championship 2024 Live
🔴📺Link📲👉 Bledisloe Cup 2024 Live
Wallabies vs All Blacks is one of rugby's fiercest and most storied rivalries, and the 2024 live encounter promises another chapter of high-octane drama. This trans-Tasman clash brings together Australia’s Wallabies and New Zealand’s All Blacks in a battle for supremacy, with both teams looking to make a statement. The All Blacks, renowned for their precision, skill, and relentless attack, will look to maintain their historical edge over the Wallabies. Meanwhile, the Wallabies, with their trademark grit and determination, are eager to turn the tide and claim a significant victory on home soil or in enemy territory.
Expect powerful scrums, blistering counterattacks, and nail-biting moments as both teams give their all for national pride. With passionate fans on both sides, Wallabies vs All Blacks LIVE is more than just a game—it’s an electric sporting spectacle that encapsulates the best of international rugby
First Test: September 21 at Accor Stadium, Sydney
Second Test: September 28 at Sky Stadium, Wellington
Both teams have had a challenging season so far, with the Wallabies splitting results in Argentina and the All Blacks losing both Tests against the Springboks. However, the All Blacks have received some good news with veteran lock Patrick Tuipulotu rejoining the squad after recovering from injury.
The Teams:
All Blacks (New Zealand): These rugby legends have won 10 out of the 12 Rugby Championships held so far. They’re like the Gandalfs of the rugby world — wise, experienced, and capable of conjuring magic on the field.
Wallabies (Australia): Under the guidance of their new coach, Kiwi Joe Schmidt, the Wallabies are ready to roar. They’ve got nothing to lose, and that can make them a dangerous opponent.
Springboks (South Africa): The World Cup winners are flexing their muscles, aiming to dethrone the All Blacks.
Pumas (Argentina): Don’t underestimate them — especially when they’re playing at home. Argentina can surprise anyone.
The Schedule and Results So Far:
The teams face off twice in consecutive weeks to minimize travel fatigue.
Notable matches:
Wallabies vs. South Africa: The Aussies fell to the Springboks, but they’re hungry for redemption.
All Blacks vs. Argentina: A high-scoring thriller that saw the Pumas upset the All Blacks.
South Africa vs. All Blacks: The Springboks edged out New Zealand in a nail-biter.
Wallabies vs. All Blacks (upcoming): The Bledisloe Cup is on the line — Australia’s chance to break a long drought.
Argentina vs. South Africa (upcoming): Expect fireworks as these two clash.
Current Standings:
South Africa leads the pack with a perfect record.
Argentina is holding their own.
New Zealand is uncharacteristically trailing.
Australia, well, they’re still finding their rhythm.
Fun Fact:
New Zealand’s dominance in the Rugby Championship has been staggering. They’ve only missed out on victory in World Cup years when teams played each other just once.
So, mark your calendar for the upcoming showdowns: Wallabies vs. All Blacks and Argentina vs. South Africa. And remember, rugby isn’t just about points — it’s about heart, grit, and the thrill of the chase!


USDT omni_send wrong

I'm a newcomer to the USDT.I encountered some problems in the test network and I was looking for help.
1.Using "omni_send" to prompt "the sender has failed to balance" errors, how do you add the USDT currency to the new address on the test network?
2.The above methods can't succeed, then try another solution: https://github.com/omnilayer/omnicore/wiki/use ws-transaction - apito - create - a - simple - send ws-transaction;
Found through practice, you can send the transaction normally, but no USDT currency in my new address, only the BTC (test), BTC to deduct fees, to complete the transfer process, but I still can't query the destination address USDT balance, maybe that is the wrong way, then what should I do?
I'm looking forward to your help.Thank you very much.

I cannot find a Property USDT in the properties list here and here. Probably you need to check the name of the property you need.
Here, how I check a balance
curl -H 'Content-Type: application/json' --user 'user:password' -X POST --data-binary '{"jsonrpc": "2.0", "id":"omni_test", "method": "omni_getbalance", "params": ["address", property_id]}' http://<IP_ADDRESS>:14873

Hey @hhy464591454,
you can list available tokens with omni_listproperties. To create a token to test with, you could use omni_sendissuancefixed.

@hhy464591454  hi gays, can u tell me how solve this question? thks very much.


ValueError: The following model_kwargs are not used by the model: ['max_seq_length'] (note: typos in the generate arguments will also show up in this list)

i am saved merged model by using below code
model1 = AutoModelForCausalLM.from_pretrained(model_dir, torch_dtype=torch.float16)  # or torch_dtype="auto"
when I am using it facing error
code:
FastLanguageModel.for_inference(model1) # Enable native 2x faster inference
inputs = tokenizer1(
[
alpaca_prompt.format(
"Continue the fibonnaci sequence.", # instruction
"1, 1, 2, 3, 5, 8", # input
"", # output - leave this blank for generation!
)
], return_tensors = "pt")
outputs = model1.generate(**inputs, max_new_tokens = 64, use_cache = True, max_seq_length=200)
tokenizer1.batch_decode(outputs)
Error: ValueError: The following model_kwargs are not used by the model: ['max_seq_length'] (note: typos in the generate arguments will also show up in this list
Kindly help me out .

@devendrapal5755 I think you should use max_new_tokens = 64! Sorry on the delay!


Update urlProtocol

Currently all versions of ADS have the urlProtocol set to azuredatastudio. This is set every time ADS launches - which means that if you type azuredatastudio:// into the address bar it'll try to open up the latest version of ADS that you had opened. This is not only confusing, but can lead to weird bugs such as :

User has both stable and insiders installed
They open up ADS insiders so now azuredatastudio:// links will open up the insiders version of the app
They then close and uninstall ADS Insiders
Now azuredatastudio:// links are broken until they open up ADS stable to re-register the protocol handler

I'm fixing it here to use separate urlProtocols based on the quality level :
dev - azuredatastudio-oss (based on VS Code one here)
stable - azuredatastudio
insiders - azuredatastudio-insiders
saw - azuredatastudio-saw
rc1 - azuredatastudio-rc1
@MsSQLGirl @qifahs Do you see any issues with changing this for any of your stuff?

so the actual changes are in the distro repo?

The non-dev build ones are, yes. They look the same as the change here though for those respective files.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 41.704%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
26434

Relevant Lines:
57517

💛  - Coveralls

where are we using this feature?

Notebooks, resource deployment (for deploying Arc controllers from the portal) and opening TSGs from the portal are the 3 main ones I know of.

Apologies for the late reply. The proposed changes should be fine.
Thanks!


Used invalid characters

Describe the bug
Used invalid characters: hue.0.Lisa’s_room.allOn changed to hue.0.Lisa_s_room.allOn
Expected behavior
Be able to use apostrophe in name.
Screenshots & Logfiles
hue.0 | 2024-06-14 15:08:14.009 | warn | Used invalid characters: hue.0.Lisa’s_room.allOn changed to hue.0.Lisa_s_room.allOn
CPU: 11,87 %
RAM: 36 %
Uptime: 1h
Available: 6.0.4
Installed: 6.0.4
Events: ⇥28 / ↦13
Platform: linux
operating system: linux
Architecture: arm64
CPUs: 4
Speed: 1800 MHz
Model: Cortex-A72
RAM: 7.6 GB
System uptime: 4 d. 05:38:08
Node.js: v20.14.0
time:<PHONE_NUMBER>648
timeOffset: -120
NPM: 10.7.0
adapters count: 624
Disk size: 234.2 GB
Disk free: 207.9 GB
Active instances: 28
Path: /opt/iobroker/
Uptime: 00:54:39
alive: true
_nodeCurrent: 20.14.0
_nodeNewest: 20.14.0
_nodeNewestNext: 18.20.3
_npmCurrent: 10.7.0
_npmNewest: 10.7.0
_npmNewestNext: 10.7.0

Object Ids are limited to a restricted characterset. An ' is not allowed. Please use a-z A-Z 0-9 and _ only. Some more Charakters are tolerates but not recommended.
This is the intended behavior for all Adapters.

Reopen as Adapter should eventually do this cleanup to avoid warning

Thanks for the clarification.
The name comes from the Hue app, therefore my question.
I’ll rename it to a “friendly” name.
So this would be an ioBroker “issue” if I understand correctly?

Normally the adapter should filter illegal characters and replace them by ie. an _. In this case this did not work and the adpater code did the replacemnet and issued a warning. The result is - besides the warning - identical.
You may rename it at hue uapp or live with the slightly modified object id. Both should work.

Object Ids are limited to a restricted characterset. An ' is not allowed. Please use a-z A-Z 0-9 and _ only. Some more Charakters are tolerates but not recommended.
This is the intended behavior for all Adapters.

I have been using the hue adapter since around 2017 and since that time it has been no problem using German umlauts ( think Küche, Gästezimmer etc). It took me a while to realize that recently the hue adapter is not able to handle this and instead completely ignores it.
The object names with umlauts can be assigned in the Hue app and are also correctly adopted and displayed in the IOBroker object tree. However, when I use them in scripts, for example, nothing happens (anymore).
When building the object tree, a consistent behavior in the use of names would be desirable, e.g. to mark ignored characters with a question mark. Even better, however, would be support for the complete local character set.
Thank you.

Geman Umlute are allowed at state NAMES. Only state IDs are restricted to a Set of Charakters. Thats in effect since a long time but core now enforces this. Thats nothing thexadapter can Influenza.

For human readable infornarionbthe NAME attribute is available.

Okay. The reasons for older non unicode-capable connection systems are understandable. But if the object ID is derived identically from the HUE-app-naming, then 1. you have to remember in the Hue app that 2. IObroker script may not use the object ID because 3. an unknown connection system might have problems with it.
Sorry, but our lives could be a bit simpler at this point :)
And it's just as well that the Hue Alexa skill can - still - deal with the term "Schalte Kuechenspuele/Haustuer/Gaestezimmer ein".
Thank you for your support!
Translated with DeepL.com (free version)

But if the object ID is derived identically from the HUE-app-naming,

Thats a bug in the adapter - thats why this issue still exists.
Ids should be dereived from stable information (like mac address) if available. If they are taken from external sources they must be convertsed by the adapter to local naming rules.

But if the object ID is derived identically from the HUE-app-naming,

Thats a bug in the adapter - thats why this issue still exists. Ids should be dereived from stable information (like mac address) if available. If they are taken from external sources they must be convertsed by the adapter to local naming rules.

Sounds good in theory. In practice, for example, I currently manage 65 physical Hue devices (lamps, plugs, switches, motion detectors) and countless virtual devices (scenes, rooms, zones, etc.). If the IOBroker adapter introduces an additional layer of abstraction because of IT theory, then my management practice is made more difficult in favor of exactly what, what goal?
Searching for additional device information in the Hue app is almost impossible, especially when it comes to virtual devices such as scenes, rooms, etc.  It only works with names and these can be pretty arbitrary (spaces, umlauts, special characters etc.).
The current adapter creates a new entry when a Hue device is renamed, for example, without removing the old one from the object tree. I can live with this “bug” much better than with the fact that devices cannot be switched if there are umlauts in the name. Funnily enough, retrieving device information and statuses also works with umlauts.
Translated with DeepL.com (free version)


Error in task displayed in My Inbox

I'm receiving an error when clicking on the task in My Inbox (step 12 of Execute and Monitor Workflow) in Exercise 1.
An error occurred.  Could not load data.  Please reach out to your help desk.

Found the issue in the dev tools.

On the CapexApproval_00 form, I had listed the CAPEX field as a String instead of a Float.


feat(storage): Store blob content-type in extended attributes

After the discussion in #116, this stores the blob content types
in extended attributes when using the filesystem backend.
If the underlying filesystem doesn't support extended attributes,
storing blobs won't work; also, if extended attributes get removed,
blobs won't be served anymore. We can relax this behavior if
needed (i.e. log errors but still accept to store or serve blobs).
However, since the Docker Engine (and possibly other container
engines) won't accept to pull images from a registry that doesn't
use correct content types for manifest files, it could be argued
that it's better to give a hard fail. (Otherwise, the container
engine gives cryptic error messages like "missing signature key".)
I can change that behavior (and log errors but still store/serve
blobs to the filesystem) if you think it's better.
One last thing: I have very little knowledge of Go dependencies
and Nix, so I hope I did the right thing here to add the dependency
on pkg/xattr; but please double-check as it's my first time doing this :sweat_smile:

Welp, the CI system doesn't support xattrs. :upside_down_face:
Should we try to change the CI system, or?

@jpetazzo I did some quick tinkering by manually spinning up a ubuntu:latest container on my machine:
❯ docker run --mount type=tmpfs,destination=/foo -it ubuntu:latest
root@152f82d17d81:/# apt update && apt install attr &> /dev/null
[…]
root@152f82d17d81:/# touch /foo/bar;setfattr -n "user.mime_type" -v "value" /foo/bar
setfattr: /foo/bar: Operation not supported
root@152f82d17d81:/# touch /root/bar;setfattr -n "user.mime_type" -v "value" /root/bar

Apparently, user xattrs on tmpfs aren't supported (see tmpfs(5)), and for other filesystems, the answer is "it depends" (xattr(7)).
I tinkered around with the CI a bit, and luckily the CI itself is backed by something supporting xattrs, it's just tmpfs doesn't support it.
I made https://github.com/google/nixery/commit/d2c007060c32882e12a2b0c044fd6dda7db6ab17.patch, which fixes CI, as can be seen in https://github.com/flokli/nixery/runs/2861162544.
Feel free to cherry-pick this on top.

I pushed that CI change to a PR at https://github.com/google/nixery/pull/128. If that is merged, this PR should become green after a rebase.

Rebasing this PR on master should now resolve the CI issue, thanks to @flokli!

@jpetazzo can you press the rebase button?

Thanks for the work on the CI! 👍🏻
I would have been happy to try and fix it, but I saw that there had been changes in the CI recently so I wasn't sure what was the current state.
Looks like everything passes now, yay! 🎉
... And sorry for the slower turnaround on my end :)

... And sorry for the slower turnaround on my end :)

I feel like I should write that sentence :sweat_smile: Thank you for the contribution!
I guess we don't need 7db252f36a68d875429a25e06d88fbfc804d84fd anymore after this and can probably revert it once merged?

I guess we don't need 7db252f anymore after this and can probably revert it once merged?

Indeed! Would you like me to submit a PR that does both (i.e. with these commits + reverting the other one)?

Feel free to send a separate CL reverting the old commit. Thanks for your contribution! :)


Backward compatible support of new jsdom versions

In my test, it is not compatible for the jsdom 3.x. So I limit it to version 2.x, also have a good performance.


ADR: Don't rely on third party dependencies for the filtered list

closes #12

I think you nicely described the situation.
Only the title is a bit confusing for me. As far as I've understood, the new component still depends on the the mwc-list and therefore the users of oscd-filtered-list too. The main changes is that they no longer have to directly install and import mwc-list.
So my suggestion for the title would be something along lines:

"Encapsulate mwc-list instead of extending it"

I think you nicely described the situation. Only the title is a bit confusing for me. As far as I've understood, the new component still depends on the the mwc-list and therefore the users of oscd-filtered-list too. The main changes is that they no longer have to directly install and import mwc-list.
So my suggestion for the title would be something along lines:

"Encapsulate mwc-list instead of extending it"

Thank you, Ill change the title to something less confusing!

Feel free to share the new ADR in the Slack channel! It will help to get accepted


Make Schedules user friendly again

Feature Request
Feature Information:
Version 2.4 changed Time-zone and scheduling which makes AMP use UTC internally. I understand why this change was made.
However what this change does is introduce what I consider extreme levels of user unfriendliness to the scheduler. Since my servers time zone is set to GMT+2 I now have to convert that to UTC time and set that as the scheduler time instead of before where it used the servers time and bam stuff happened when I wanted stuff to happen and not later.
In short this update broke my timed schedules since it shifted them all by 2 hours.
I am requesting that something be done to make this more user friendly like an automatic conversion of sorts or anything at all so that I and others do not have to make a mental note every time that the scheduler is running on UTC time and that we should convert the time we want to set the thing up at into UTC before inputting it....

I confirm:

[x] that I have searched for an existing feature request matching the description.

The update hasn't changed anything for me on debian 11.
The scheduler is set to 3am, and it executes the trigger at 5am CEST.
root@amp:~# dpkg-reconfigure tzdata

Current default time zone: 'Europe/Berlin'
Local time is now:      Mon Sep 12 11:42:49 CEST 2022.
Universal Time is now:  Mon Sep 12 09:42:49 UTC 2022.

Best would be a conversion on the GUI from UTC to my local time zone or based on a per-user setting.

Fully agree this should be reviewed again. Any software I use converts it to local time zone in the GUI.

Why was this change made?

The need for the change is justified. It was causing issues having system time differ from UTC. But it causes this issue if it isn't translated in any way for users.

Agreed, my opinion is that UTC should be used internally, and translated to the local time zone in the GUI for the end user.  Asking a normal user to convert to UTC is only going to be a long term pain.

Bumping this consequential fix

Bumping from Tokyo - I'm loving AMP, just migrated from MineOS, but having to manually convert all my scheduled tasks for my MC servers is complete PITA.

Adding my 2 cents here that the schedule should use local time - anything shown in the GUI should use local TZ.

bumping this also. There has to be a way for AMP to convert UTC time based on the region the user wants. Would be ideal.

Agreed. Hadn't noticed that my scheduled tasks were happening 7 hours before they were meant to until it just interrupted some people due to a shutdown message.

I noticed this too. Please make the timezone consistent with the user's system (even better, browser) time.

Would be cool to be able to select my time zone and it use that for schedules

I'm not sure I understand the reasoning behind the idea that the UI displays the correct local time a scheduled backup was taken but to set up that schedule you must do the mental math for UTC -/+ to have it execute when needed.
Please, if you have any empathy for us users at all consider moving this up just a bit higher in the priority list.

Just to clarify the "why", this was done to fix previous inconsistencies initially. By making it all UTC on the back end it removed those issues. But yes, it does cause user experience issues.

To dump this into 2024- I will attest, all of the schedules being in UTC, is quite annoying, especially when you are hosting a server for people who are all in the same time zone.
Oh, good, let me set this schedule to run at midnight. Ok, done. (no warnings, no messages, or anything about time-zones.)
The next day- why is my server still running, I set a schedule to shut it down at midnight.
/ends up here.

Almost halfway through 2024 and I'm requesting this feature as well. Please give us the ability to set a time zone for the server and use local time for the scheduler. This UTC stuff is not user-friendly at all!

+1 plz fix it

Another bump for this to get fixed please!

 Because AMP does everything in UTC, if the users time-zone changes (as it does for people with daylight savings) - setting up a schedule at 5AM say could result in it actually firing at 4AM or 6AM depending on when in the year you set it up, and to top it off the label you set wouldn't make sense either.
So what I've done instead is simply add a server time-zone setting to AMPs UI. Changing this will not change your existing schedules so if you've built your schedules for UTC then you'll need to adjust them if you change this setting.

But this does mean you can at least set it to your time zone and it will make sense.  Because AMP does everything in UTC, if the users time-zone changes (as it does for people with daylight savings) - setting up a schedule at 5AM say could result in it actually firing at 4AM or 6AM depending on when in the year you set it up, and to top it off the label you set wouldn't make sense either.
So what I've done instead is simply add a server time-zone setting to AMPs UI. Changing this will not change your existing schedules so if you've built your schedules for UTC then you'll need to adjust them if you change this setting.

But this does mean you can at least set it to your time zone and it will make sense. Just not for users in other time zones necessarily.

Not sure if something happened since you added this, but the time zone setting doesn't work: https://discourse.cubecoders.com/t/some-issues-with-rust-on-amp/19595


Reload ndb will cause EADDRINUSE in web application

Steps to reproduce

Use ndb run an application listening a port.
command + r (OSX) in ndb
EADDRINUSE

ndb version: latest
Platform / OS version: OSX
Node.js version: 8.11.3

What is the expected result?
Release the port when reloading

Thanks for your report!
Ctrl + R on Linux/Windows and Cmd + R behave much better in published 1.0.16.


Bump supported version of camptocamp-systemd module

Currently, it produces following warning:
Warning: Module 'camptocamp-systemd' (v1.0.0) fails to meet some dependencies:
  'choria-mcollective_choria' (v0.3.1) requires 'camptocamp-systemd' (>= 0.3.0 < 1.0.0)

However, I didn't notice any problems with version 1.0.0.

I am supposing this is down to some other modules you use who have competing dependencies (they need 1)? A clean install at present work for me?
Not against updating anyway so lets do it

I just like to keep all modules up to date :) And I'm kind of person who is annoyed by warnings like this.

yeah fair enough, its a pain in the arse, I try to check out all depedencies every release or 3 but I bet we are about to see a load of releases on modules
thanks as always!


Documentation: Create screenshots and image files

Store image files in separate issues with names

Work in Progress. Close for now.


Base Unit for Bitcoin?

I was looking to add satoshi unit but unfortunately there's no base unit for bitcoin. If I'm not mistaken there's unit for Bitcoin in Numi.

I added it. it is BTC. Feel free to use this:
satoshi.js:
numi.addUnit({
    "id": "satoshi",
    "phrases": "satoshi, satoshis, sato, sat, bit",
    "baseUnitId": "BTC",
    "format" : "sat",
    "ratio" : 0.00000001,
});


Add FreeBSD as a host type

In the web UI, only macOS, Ubuntu, CentOS, and MS Windows host types are shown on the right pane. It would be nice to add FreeBSD to this list.

This looks to be a dupe of #1909 - and from a deleted user. @zwass close this in favor of that more active one?


Add test cases for all Header Component and empty state components

Description
Add test case for following components:-

AddressDetailHeader
AddressSapceHeader
ConnectionDetailHeader
EmptyAddresses
EmptyConnections

@riccardo-forina , made the changes as requested. Please let me know if any other change is required.


Auto keys

Can you please add auto key completion to the script?

soon

Can you please add auto key completion to the script?

https://github.com/masterking32/MasterHamsterKombatBot/discussions/45


chore(Filter/Pi): add piMap_map_pi

... and use it to prove IsOpenMap.piMap.

[x] depends on: #15694
[x] depends on: #17482

Thanks!
bors d+


tpcds-tools remove sf100 cascades specific control for global config

Proposed changes
remove sf100 cascades specific control for global config, which is only for q72
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


temporarily remove failing tests

Related issue
re https://github.com/kyverno/policies/pull/182#issuecomment-985583728

Milestone of this PR

What type of PR is this

Proposed Changes

Proof Manifests

Checklist

[ ] I have read the contributing guidelines.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] My PR contains new or altered behavior to Kyverno and
[ ] I have read the PR documentation guide and followed the process including adding proof manifests to this PR.

Further Comments

How about just rename test.yaml to tmp.yaml or something?

How about just rename test.yaml to tmp.yaml or something?

renaming was still throwing errors. once you've merged this i'll raise another to revert it

I just tested it by renaming test.yaml to asdf.yaml and the test was skipped.

oh, i'd done _test.yaml which it was still matching, I can do that if you'd
rather
On Fri, 3 Dec 2021 at 15:34, Chip Zoller @.***> wrote:

I just tested it by renaming test.yaml to asdf.yaml and the test was
skipped.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kyverno/policies/pull/185#issuecomment-985616007, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFOS4GZFVRQ5TE6IXSAMOTUPDPQXANCNFSM5JJ4ZNBA
.

--
Chris Nesbitt-Smith
Mobile:<PHONE_NUMBER>8

Yes, please, that way no files have to be restored, we just flip the name back to test.yaml when the upstream release is cut. We just need a reminder in the form of an issue to do so (which can be assigned to me).

done

Test pass, thanks. Don't forget that issue :)

reverting PR: https://github.com/kyverno/policies/pull/186


Quickstart tutorial application url does not work with Java 17

So, I couldn't get the quickstart tutorial to run at all until I changed '-runee: JavaSE-1.8' to '-runee: JavaSE-17'. After I did that, and the 'mvn bnd-indexer:index bnd-indexer:index@test-index bnd-resolver:resolve' and 'mvn verify', I was able to run it, but when I try and connect to 'http://localhost:8080/quickstart/index.html' it shows 'HTTP ERROR 404 Problem accessing /quickstart/index.html. Reason: Not Found'. http://localhost:8080/rest/upper/lower however does seem to be running, and loads 'LOWER'. Does the project simply not work with Java 17 and I need to use java 11? Or is there a way to make it work? :/

hey @Angular-Angel I'm sorry to hear you're having troubles. So in order to test this quickly I usually have multiple JVMs installed and because maven is super friendly it will adhere to what you have set for JAVA_HOME which makes testing different versions really simple:
export JAVA_HOME=/usr/lib/jvm/<vendor>8
mvn -version
mvn clean verify

Can you try doing above with JDK 8 as well as for Java 17 and return the result?
Chances are if you are using stock project as configured will give you an error since it's using super old bnd-maven-plugin. Try upping to 6.1.0 (in root of examples repo). (meanwhile, I'll send a PR to update the bnd version and any other small changes required.)

Hmm, let me see. I'm using netbeans, but I can tell it to use java 8...

Okay, so, outputs:
(base) angle@Sager:~/Documents/Programming/Java/osgi.enroute/examples/quickstart$ export JAVA_HOME=/usr/lib/jvm/java-8-openjdk-amd64
(base) angle@Sager:~/Documents/Programming/Java/osgi.enroute/examples/quickstart$ mvn -version
Apache Maven 3.8.3 (ff8e977a158738155dc465c6a97ffaf31982d739)
Maven home: /opt/apache-maven-3.8.3
Java version: 1.8.0_312, vendor: Private Build, runtime: /usr/lib/jvm/java-8-openjdk-amd64/jre
Default locale: en_US, platform encoding: UTF-8
OS name: "linux", version: "5.4.0-99-generic", arch: "amd64", family: "unix"
(base) angle@Sager:~/Documents/Programming/Java/osgi.enroute/examples/quickstart$ mvn clean verify
[INFO] Scanning for projects...
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Build Order:
[INFO]
[INFO] quickstart                                                         [pom]
[INFO] rest                                                               [jar]
[INFO] app                                                                [jar]
[INFO]
[INFO] ----------< org.osgi.enroute.examples.quickstart:quickstart >-----------
[INFO] Building quickstart 0.0.2-SNAPSHOT                                 [1/3]
[INFO] --------------------------------[ pom ]---------------------------------
[INFO]
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ quickstart ---
[INFO]
[INFO] -------------< org.osgi.enroute.examples.quickstart:rest >--------------
[INFO] Building rest 0.0.2-SNAPSHOT                                       [2/3]
[INFO] --------------------------------[ jar ]---------------------------------
[INFO]
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ rest ---
[INFO] Deleting /home/angle/Documents/Programming/Java/osgi.enroute/examples/quickstart/rest/target
[INFO]
[INFO] --- maven-resources-plugin:2.6:resources (default-resources) @ rest ---
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 4 resources
[INFO]
[INFO] --- maven-compiler-plugin:3.1:compile (default-compile) @ rest ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 1 source file to /home/angle/Documents/Programming/Java/osgi.enroute/examples/quickstart/rest/target/classes
[INFO]
[INFO] --- bnd-maven-plugin:5.1.2:bnd-process (default) @ rest ---
[INFO]
[INFO] --- maven-resources-plugin:2.6:testResources (default-testResources) @ rest ---
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory /home/angle/Documents/Programming/Java/osgi.enroute/examples/quickstart/rest/src/test/resources
[INFO]
[INFO] --- maven-compiler-plugin:3.1:testCompile (default-testCompile) @ rest ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 1 source file to /home/angle/Documents/Programming/Java/osgi.enroute/examples/quickstart/rest/target/test-classes
[INFO]
[INFO] --- maven-surefire-plugin:2.12.4:test (default-test) @ rest ---
[INFO] Surefire report directory: /home/angle/Documents/Programming/Java/osgi.enroute/examples/quickstart/rest/target/surefire-reports

-------------------------------------------------------
 T E S T S
-------------------------------------------------------
Running org.osgi.enroute.examples.quickstart.rest.UpperTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.002 sec

Results :

Tests run: 1, Failures: 0, Errors: 0, Skipped: 0

[INFO]
[INFO] --- maven-jar-plugin:3.0.2:jar (default-jar) @ rest ---
[INFO] Building jar: /home/angle/Documents/Programming/Java/osgi.enroute/examples/quickstart/rest/target/rest-0.0.2-SNAPSHOT.jar
[INFO]
[INFO] --------------< org.osgi.enroute.examples.quickstart:app >--------------
[INFO] Building app 0.0.2-SNAPSHOT                                        [3/3]
[INFO] --------------------------------[ jar ]---------------------------------
[INFO]
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ app ---
[INFO] Deleting /home/angle/Documents/Programming/Java/osgi.enroute/examples/quickstart/app/target
[INFO]
[INFO] --- maven-resources-plugin:2.6:resources (default-resources) @ app ---
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 1 resource
[INFO]
[INFO] --- maven-compiler-plugin:3.1:compile (default-compile) @ app ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 1 source file to /home/angle/Documents/Programming/Java/osgi.enroute/examples/quickstart/app/target/classes
[INFO]
[INFO] --- bnd-maven-plugin:5.1.2:bnd-process (default) @ app ---
[INFO]
[INFO] --- maven-resources-plugin:2.6:testResources (default-testResources) @ app ---
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory /home/angle/Documents/Programming/Java/osgi.enroute/examples/quickstart/app/src/test/resources
[INFO]
[INFO] --- maven-compiler-plugin:3.1:testCompile (default-testCompile) @ app ---
[INFO] No sources to compile
[INFO]
[INFO] --- maven-surefire-plugin:2.12.4:test (default-test) @ app ---
[INFO] No tests to run.
[INFO]
[INFO] --- maven-jar-plugin:3.0.2:jar (default-jar) @ app ---
[INFO] Building jar: /home/angle/Documents/Programming/Java/osgi.enroute/examples/quickstart/app/target/app-0.0.2-SNAPSHOT.jar
[INFO]
[INFO] --- bnd-indexer-maven-plugin:5.1.2:index (index) @ app ---
[INFO] The configured deployment repository OSGi Snapshots has the same id as one of the remote artifact repositories. It is assumed that these repositories are the same.
[INFO]
[INFO] --- bnd-indexer-maven-plugin:5.1.2:index (test-index) @ app ---
[INFO] The configured deployment repository OSGi Snapshots has the same id as one of the remote artifact repositories. It is assumed that these repositories are the same.
[INFO]
[INFO] --- bnd-export-maven-plugin:5.1.2:export (default) @ app ---
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary for quickstart 0.0.2-SNAPSHOT:
[INFO]
[INFO] quickstart ......................................... SUCCESS [  0.237 s]
[INFO] rest ............................................... SUCCESS [  2.819 s]
[INFO] app ................................................ SUCCESS [  2.680 s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  5.996 s
[INFO] Finished at: 2022-02-11T16:04:52-07:00
[INFO] ------------------------------------------------------------------------

For Java 8, for java 17:
(base) angle@Sager:~/Documents/Programming/Java/osgi.enroute/examples/quickstart$ mvn -version
Apache Maven 3.8.3 (ff8e977a158738155dc465c6a97ffaf31982d739)
Maven home: /opt/apache-maven-3.8.3
Java version: 17.0.1, vendor: Private Build, runtime: /usr/lib/jvm/java-17-openjdk-amd64
Default locale: en_US, platform encoding: UTF-8
OS name: "linux", version: "5.4.0-99-generic", arch: "amd64", family: "unix"
(base) angle@Sager:~/Documents/Programming/Java/osgi.enroute/examples/quickstart$ mvn clean verify
[INFO] Scanning for projects...
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Build Order:
[INFO]
[INFO] quickstart                                                         [pom]
[INFO] rest                                                               [jar]
[INFO] app                                                                [jar]
[INFO]
[INFO] ----------< org.osgi.enroute.examples.quickstart:quickstart >-----------
[INFO] Building quickstart 0.0.2-SNAPSHOT                                 [1/3]
[INFO] --------------------------------[ pom ]---------------------------------
[INFO]
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ quickstart ---
[INFO]
[INFO] -------------< org.osgi.enroute.examples.quickstart:rest >--------------
[INFO] Building rest 0.0.2-SNAPSHOT                                       [2/3]
[INFO] --------------------------------[ jar ]---------------------------------
[INFO]
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ rest ---
[INFO] Deleting /home/angle/Documents/Programming/Java/osgi.enroute/examples/quickstart/rest/target
[INFO]
[INFO] --- maven-resources-plugin:2.6:resources (default-resources) @ rest ---
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 4 resources
[INFO]
[INFO] --- maven-compiler-plugin:3.1:compile (default-compile) @ rest ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 1 source file to /home/angle/Documents/Programming/Java/osgi.enroute/examples/quickstart/rest/target/classes
[INFO]
[INFO] --- bnd-maven-plugin:5.1.2:bnd-process (default) @ rest ---
[INFO]
[INFO] --- maven-resources-plugin:2.6:testResources (default-testResources) @ rest ---
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory /home/angle/Documents/Programming/Java/osgi.enroute/examples/quickstart/rest/src/test/resources
[INFO]
[INFO] --- maven-compiler-plugin:3.1:testCompile (default-testCompile) @ rest ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 1 source file to /home/angle/Documents/Programming/Java/osgi.enroute/examples/quickstart/rest/target/test-classes
[INFO]
[INFO] --- maven-surefire-plugin:2.12.4:test (default-test) @ rest ---
[INFO] Surefire report directory: /home/angle/Documents/Programming/Java/osgi.enroute/examples/quickstart/rest/target/surefire-reports

-------------------------------------------------------
 T E S T S
-------------------------------------------------------
Running org.osgi.enroute.examples.quickstart.rest.UpperTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.001 sec

Results :

Tests run: 1, Failures: 0, Errors: 0, Skipped: 0

[INFO]
[INFO] --- maven-jar-plugin:3.0.2:jar (default-jar) @ rest ---
[INFO] Building jar: /home/angle/Documents/Programming/Java/osgi.enroute/examples/quickstart/rest/target/rest-0.0.2-SNAPSHOT.jar
[INFO]
[INFO] --------------< org.osgi.enroute.examples.quickstart:app >--------------
[INFO] Building app 0.0.2-SNAPSHOT                                        [3/3]
[INFO] --------------------------------[ jar ]---------------------------------
[INFO]
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ app ---
[INFO] Deleting /home/angle/Documents/Programming/Java/osgi.enroute/examples/quickstart/app/target
[INFO]
[INFO] --- maven-resources-plugin:2.6:resources (default-resources) @ app ---
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 1 resource
[INFO]
[INFO] --- maven-compiler-plugin:3.1:compile (default-compile) @ app ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 1 source file to /home/angle/Documents/Programming/Java/osgi.enroute/examples/quickstart/app/target/classes
[INFO]
[INFO] --- bnd-maven-plugin:5.1.2:bnd-process (default) @ app ---
[INFO]
[INFO] --- maven-resources-plugin:2.6:testResources (default-testResources) @ app ---
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory /home/angle/Documents/Programming/Java/osgi.enroute/examples/quickstart/app/src/test/resources
[INFO]
[INFO] --- maven-compiler-plugin:3.1:testCompile (default-testCompile) @ app ---
[INFO] No sources to compile
[INFO]
[INFO] --- maven-surefire-plugin:2.12.4:test (default-test) @ app ---
[INFO] No tests to run.
[INFO]
[INFO] --- maven-jar-plugin:3.0.2:jar (default-jar) @ app ---
[INFO] Building jar: /home/angle/Documents/Programming/Java/osgi.enroute/examples/quickstart/app/target/app-0.0.2-SNAPSHOT.jar
[INFO]
[INFO] --- bnd-indexer-maven-plugin:5.1.2:index (index) @ app ---
[INFO] The configured deployment repository OSGi Snapshots has the same id as one of the remote artifact repositories. It is assumed that these repositories are the same.
[INFO]
[INFO] --- bnd-indexer-maven-plugin:5.1.2:index (test-index) @ app ---
[INFO] The configured deployment repository OSGi Snapshots has the same id as one of the remote artifact repositories. It is assumed that these repositories are the same.
[INFO]
[INFO] --- bnd-export-maven-plugin:5.1.2:export (default) @ app ---
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary for quickstart 0.0.2-SNAPSHOT:
[INFO]
[INFO] quickstart ......................................... SUCCESS [  0.087 s]
[INFO] rest ............................................... SUCCESS [  2.569 s]
[INFO] app ................................................ SUCCESS [  3.867 s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  6.630 s
[INFO] Finished at: 2022-02-11T16:06:27-07:00
[INFO] ------------------------------------------------------------------------

I also tried setting the bnd run to reference java 1.8, recompiled, and reran it like that, and then I couldn't get the app or the rest endpoint working.

Oh, changing <bnd.version> to 6.1.0 doesn't seem to change anything either.

ok, so further testing something is clearly broken somewhere. Even after I make it all work with proper resolutions and no errors, the resource aspect of Upper (@HttpWhiteboardResource(pattern="/quickstart/index.html", prefix="/static/index.html")) never gets tracked by the Http Whiteboard (felix.http.jetty).
Somewhere along the line something broke and I need to investigate. Sadly right now I have to run. I'll try to dig in over the weekend. @Angular-Angel sorry about this.

a'right... so going back to the start and just add this to the Upper.java class
@org.osgi.service.http.whiteboard.propertytypes.HttpWhiteboardContextSelect("(osgi.http.whiteboard.context.name=org.osgi.service.http)")

and let me know if things start to work.

OK, so I've confirmed again after sleep and coffee that this was indeed the issue.
For some reason I do not fully comprehend somewhere along the line a difference in the Http Whiteboard (or perhaps the JAX-RS Whiteboard) makes that the resource part of Upper (@HttpWhiteboardResource(pattern="/quickstart/*", prefix="static")) is bound the default context which provides a ServletContext (backed by a ServletContextHelper) which has no bundle associated and therefore cannot local the static resources. By adding the suggested line above, we force Upper's resource to bind to another existing context which creates a per bundle ServletContext so that the static resource is found. I'm puzzled frankly as to how this may have previously worked.

So there were two issues for which I can send a fix:

the bnd version was incompatible with Java 17
the broken resource binding on any Java version

Hey, nice! I'll take a look at these on monday, see if I can get the thing working.

Well, I cleaned and tried again with the latest version, and it behaves exactly the same. I'll wait for that pull request to go through and then try again.

I merged the PR which closed the issue but please if you continue to have issue after retest on HEAD we can reopen this to perform more investigations.

Yup, I can confirm, it works now!

Great news! thanks for the report.


Game modes: Multicolor as joker

Great job so far! Super enjoyable.
One thing missing:
The physical game instructions recommends playing with the Multicolor cards as jokers. That allows two more levels of difficulty:
Easiest, where there is no sixth pile to build, but the multicolored cards can be used for any other color. However, when giving hints, they are of ALL other colors, and can not be named alone. That is, if a player has two blue cards, two multicolored cards and a yellow card, the possible hints are: "these four cards are blue", or "these three cards are yellow".
Hardest: same as above for hints, but the multicolored cards can only be played in their own pile.

Hi! Thanks for the feedback!
We're actually going to release the "hard" multicolor version soon. We called it the "rainbow" variant.(https://github.com/bstnfrmry/hanabi/pull/96).
I'll probably merge tomorrow. In the meantime you can play it on the testing URL here: https://hanabi-git-rainbow.hanabi.now.sh/
The "easy" version should be pretty easy to implement. I might do it next week.

@rubchacon it's now live 🎉

Hey Bastien!
The "Rainbow" Mode is perfect 😊. It's really hard, but that is to be
expected.
I agree that the other mode, let's call it "Rainbow easy" should be easy to
implement, it's basically just taking away the rainbow stack, leaving
everything the same.
I'll be happy to beta test any advances and I'll keep making suggestions,
as long as they are welcome. And I'd love to play with you guys in one
game, if we can swing the hours! (I'm in Germany).
Thanks for the heads up, and stay safe and healthy!
Ruben
On Sat, 18 Apr 2020, 14:20 Bastien Formery<EMAIL_ADDRESS>wrote:

@rubchacon https://github.com/rubchacon it's now live 🎉
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bstnfrmry/hanabi/issues/99#issuecomment-615857477,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANDQKCUEJIIVVXTH54GQ2SDRNGLJRANCNFSM4MGRE4ZQ
.

Oh.. I left something out of the previous email: rainbow easy is like
rainbow without the stack, PLUS the rainbow cards go in ANY other stack.
Wanted to repeat it so I'm not misleading.
Cheers
Ruben
On Sun, 19 Apr 2020, 18:20 Ruben Chacon<EMAIL_ADDRESS>wrote:

Hey Bastien!
The "Rainbow" Mode is perfect 😊. It's really hard, but that is to be
expected.
I agree that the other mode, let's call it "Rainbow easy" should be easy
to implement, it's basically just taking away the rainbow stack, leaving
everything the same.
I'll be happy to beta test any advances and I'll keep making suggestions,
as long as they are welcome. And I'd love to play with you guys in one
game, if we can swing the hours! (I'm in Germany).
Thanks for the heads up, and stay safe and healthy!
Ruben
On Sat, 18 Apr 2020, 14:20 Bastien Formery<EMAIL_ADDRESS>wrote:

@rubchacon https://github.com/rubchacon it's now live 🎉
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bstnfrmry/hanabi/issues/99#issuecomment-615857477,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANDQKCUEJIIVVXTH54GQ2SDRNGLJRANCNFSM4MGRE4ZQ
.

Thanks for the feedback!
Feel free to join our Discord server so we can chat about some improvements and maybe play a few games 😉


Trace only contains a single span?

I've implemented spring-cloud-sleuth-otel-autoconfigure into a project I'm working on (REST api), and while I do see a trace being sent to my local collector, said trace only contains a single span that lasts the duration of the rest call. I'm not sure if I need to add an annotation somewhere, or maybe if i have a conflicting dependency? I'm hoping to see a trace with more detailed span information.
Relevant sections of my pom.xml file:
	<parent>
		<groupId>org.springframework.boot</groupId>
		<artifactId>spring-boot-starter-parent</artifactId>
		<version>2.4.13</version>
		<relativePath/>
	</parent>

...

	<dependencies>
		<dependency>
			<groupId>org.springframework.cloud</groupId>
			<artifactId>spring-cloud-starter-sleuth</artifactId>
			<exclusions>
				<exclusion>
					<groupId>org.springframework.cloud</groupId>
					<artifactId>spring-cloud-sleuth-brave</artifactId>
				</exclusion>
			</exclusions>
		</dependency>

		<dependency>
			<groupId>org.springframework.cloud</groupId>
			<artifactId>spring-cloud-sleuth-otel-autoconfigure</artifactId>
		</dependency>

		<dependency>
			<groupId>io.opentelemetry</groupId>
			<artifactId>opentelemetry-extension-annotations</artifactId>
		</dependency>

		<dependency>
			<groupId>io.opentelemetry</groupId>
			<artifactId>opentelemetry-exporter-otlp</artifactId>
		</dependency>

		<dependency>
			<groupId>io.grpc</groupId>
			<artifactId>grpc-okhttp</artifactId>
			<version>1.47.0</version>
		</dependency>
	</dependencies>

...

	<repositories>
		<repository>
			<id>spring-snapshots</id>
			<url>https://repo.spring.io/snapshot</url>
			<snapshots><enabled>true</enabled></snapshots>
		</repository>
		<repository>
			<id>spring-milestones</id>
			<url>https://repo.spring.io/milestone</url>
		</repository>
	</repositories>

	<pluginRepositories>
		<pluginRepository>
			<id>spring-snapshots</id>
			<url>https://repo.spring.io/snapshot</url>
		</pluginRepository>
		<pluginRepository>
			<id>spring-milestones</id>
			<url>https://repo.spring.io/milestone</url>
		</pluginRepository>
	</pluginRepositories>

	<dependencyManagement>
		<dependencies>
			<dependency>
				<groupId>org.springframework.cloud</groupId>
				<artifactId>spring-cloud-dependencies</artifactId>
				<!-- Provide the latest stable Spring Cloud release train version (e.g. 2020.0.0) -->
				<version>2020.0.5</version>
				<type>pom</type>
				<scope>import</scope>
			</dependency>
			<dependency>
				<groupId>org.springframework.cloud</groupId>
				<artifactId>spring-cloud-sleuth-otel-dependencies</artifactId>
				<version>1.0.0-M12</version>
				<!-- <version>1.0.0-M13</version> -->
				<scope>import</scope>
				<type>pom</type>
			</dependency>
		</dependencies>
	</dependencyManagement>

I've also tried annotating my controller's @RequestMapping functions with @WithSpan, as well as a few other downstream functions that it calls.
Here is the output I see in my local jaeger ui:

Update: I'm going to test using the opentelemetry-exporter-zipkin exporter. A peer mentioned seeing similar behavior in the past and tracing it back to the exporter being used.

Thanks for the response!
This app is basically a huge calculator; its running many of calculations (there are probably 10 different calculator classes that every POST request runs through), pulling values from a local DB, etc, The request taking 1.6 seconds from start to finish is a fairly average response time from this API in its current state.
If only the 'big' functions were added as spans (i know this isn't how it works, but just for context, ie. each of the main .calculate() functions and the 'convert from DTO to domain object function), id expect to see about 30~ spans.

As for my local collector / jaeger instance, i referenced the opentelemetry 'getting started' page (https://opentelemetry.io/docs/collector/getting-started/#demo).
Looking at the jaeger ui as well as the zipkin ui show the same thing (1 trace with 1 span)

One more update, when i edit my pom.xml file as follows:
		<!--
			<dependency>
				<groupId>io.opentelemetry</groupId>
				<artifactId>opentelemetry-exporter-otlp</artifactId>
			</dependency>
		-->
		<dependency>
			<groupId>org.springframework.cloud</groupId>
			<artifactId>spring-cloud-sleuth-zipkin</artifactId>
		</dependency>

		<dependency>
			<groupId>io.opentelemetry</groupId>
			<artifactId>opentelemetry-exporter-zipkin</artifactId>
		</dependency>

I still only get a trace with a single span sent to my local zipkin collector. The further I go, the more i feel like this is something I've misconfigured. haha

We would need to see some code I guess. Can you show us a sample that replicates this issue?

@marcingrzejszczak I have the same situation tbh. Simply I have 2 controllers and one calls another. And the called controller having a GET request with WebClient to get the content of Google's home page. I also called
tracer.nextSpan()?.tag("abc", "def")
but still the same. It is being shown like 2 different trace with single span(each belong to one trace)
When I put a breakpoint into io.opentelemetry.exporter.otlp.trace.OtlpGrpcSpanExporter#export, I have seen that each span collection input includes single span. I started to think maybe it is related with webclient usage.


HTTP Code should return the response rather than implicitly populate a parameter of the func call

https://github.com/c650/irc-bot-framework/pull/13


Special Characters being removed and counts are off

Characters outside the normal ascii set are being deleted in the tree. Like the fancy apostrophe in this: "Attorney’s"

This is not the problem. The problem is that the NLP ENGINE is not in Unicode and is not compiling for Unicode.


[Draft]Initial pass at py3 compatibility.

@deadbits, can you take a look at this PR and try to run the CLI to see if things are working in Python 3.6? I can run the CLI, but can't start a session.

Hey @cmmorrow, I can take a look this weekend and get this merged in if that works!


Draft: Async terminal.

Parallel runs not supported yet
Errors not supported yet.
Old logs and launched actions IDs stored in sessionStorage of browser.

[ ] Attempt (with incorrent env value) failed but didn't say why:

[ ] Would be great for this icon to flash when a new action starts (only for a few sec, so that user know where to put his attention)

[ ] Would be great to have more sexy styles in column form:

[ ] Would be great for user to have ability to resize width of column with mouse
[ ] Since "flow" UI is working, I suggest to make it default home page

to have link to access standard list in menu (which will not be used much)


add contributor avatars to each section

see

as example

I oppose - it does not value, does it?
It's nice for WJAX, but I'll remove it afterwards...
let's concentrate on stuff that's adding value for stakeholders :-)

ok - we have a docToolchain demo for the JAX - this should then be enough.
Regarding the value - there is not much value for the reader but I guess there is some value for the contributors...


use https to avoid host key verification problem

Using ssh might result in host key verification failure unless specifically configured.
Host key verification failed.
fatal: Could not read from remote repository.

Also, this makes it consistent with the duckdb submodule (above) where https is used.

This would be useful for my work environment where cloning using the git protocol from github is not allowed.

thanks @stephaniewang526!


Have URLs periodically checked

I'd love to have something that periodically checks the urls on my courses even if I haven't filed a pull request and then if can alert me to broken ones that would be perfect.
I know this is possible with GitHub actions, I'd just need to set it up.

I'll make this a priority too because it should be a fairly quick add. should....

That was a lie. I did not make this a priority. But now this would be a good issue for @howardbaek to do to get familiar with OTTR. So let’s chat about it.

This post suggests a GHA for periodically checking for broken links using the schedule event

The other thing that would be great for this, is to have it create a new branch and file a pull request that tells someone what the broken URLs are. That way there's already the branch they can check out to fix it.
Grab code from here:
https://github.com/jhudsl/OTTR_Template/blob/main/.github/workflows/pull_request.yml


[WORKAROUND] fix create namespace page broken on rancher 2.8.x

Summary
shell/models/steve-schema.ts was added in https://github.com/harvester/dashboard/pull/1208 which was not existed on Rancher v2.8 shell.
This PR add workaround if condition to avoid fall into below two throw new Error.
PR Checklist

Is this a multi-tenancy feature/bug?

[ ] Yes, the relevant RBAC changes are at:

Do we need to backport changes to the old Rancher UI, such as RKE1?

[ ] Yes, the relevant PR is at:

Are backend engineers aware of UI changes?

[ ] Yes, the backend owner is:

Related Issue #
https://github.com/harvester/harvester/issues/6953
Screenshot/Video

Hi @torchiaf ,
shell/models/steve-schema.ts is imported to rancher shell when importing harvester v1.4.0 on Rancher v2.8.x.
See you have better solution to overcome this problem.

Re-test work well on Rancher v2.9.2.


Hint on how to set config options

Using the script for the first time, it was not clear to me how to change the default config.

Thanks, I moved it up and have actually rewritten it to give a bit more insight on what is going on, see https://github.com/lukas2511/dehydrated#config


dotnet-sdk_7: 7.0.102 -> 7.0.201

Description of changes
Release notes: https://github.com/dotnet/core/blob/main/release-notes/7.0/7.0.3/7.0.3.md, https://github.com/dotnet/core/blob/main/release-notes/7.0/7.0.3/7.0.201.md
Fixes: CVE-2023-21808

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 219205 run on x86_64-linux 1

9 packages built:

ArchiSteamFarm
BeatSaberModManager
denaro
dotnet-aspnetcore_7 (dotnetCorePackages.aspnetcore_7_0)
dotnet-runtime_7 (dotnetCorePackages.runtime_7_0)
dotnet-sdk_7 (dotnetCorePackages.sdk_7_0)
pinta
ryujinx
wasabibackend

Result of nixpkgs-review pr 219205 run on x86_64-linux 1

9 packages built:

ArchiSteamFarm
BeatSaberModManager
denaro
dotnet-aspnetcore_7 (dotnetCorePackages.aspnetcore_7_0)
dotnet-runtime_7 (dotnetCorePackages.runtime_7_0)
dotnet-sdk_7 (dotnetCorePackages.sdk_7_0)
pinta
ryujinx
wasabibackend

Thanks for bumping BSMM deps btw. I happened to see this PR and made a TODO to check BSMM but you already took care of that :)
Feel free to ping me next time.

Thanks for bumping BSMM deps btw. I happened to see this PR merged and made a TODO to check BSMM but you already took care of that :)
Feel free to ping me next time.

Do you know why this package requires manual dependency updates after dotnet update? The depedencies updated should be included implicitly by buildDotnetModule.

No idea whatsoever. I don't have any DotNet experience. I just want to play Beat Saber with mods ;)

The depedencies updated should be included implicitly by buildDotnetModule.

Yes, but BeatSaberModManager is a .NET 7 project so we build it with dotnet-sdk_7. However, it depends on a .NET 6 library (BSIPA-Linux).
Because of this BSMM depends on some packages twice... For example, it depends on Microsoft.NETCore.App.Ref 6.0.14 and 7.0.3.
This problem should be solvable by improving buildDotnetModule. In theory setting dotnet-sdk = with dotnetCorePackages; combinePackages [ sdk_6_0 sdk_7_0 ] should work and do the right thing, but it currently doesn't because the thing returned by dotnetCorePackages.combinePackages (a wrapper around buildEnv) lacks the neccessary metadata for buildDotnetModule to work...
An alternative would be to patch this line (from net6.0 to net7.0) in patchPhase, but it's a bit unsafe because there are breaking changes between major .NET versions.

The depedencies updated should be included implicitly by buildDotnetModule.

Yes, but BeatSaberModManager is a .NET 7 project so we build it with dotnet-sdk_7. However, it depends on a .NET 6 library (BSIPA-Linux).
Because of this BSMM depends on some packages twice... For example, it depends on Microsoft.NETCore.App.Ref 6.0.14 and 7.0.3.
This problem should be solvable by improving buildDotnetModule. In theory setting dotnet-sdk = with dotnetCorePackages; combinePackages [ sdk_6_0 sdk_7_0 ] should work and do the right thing, but it currently doesn't because the thing returned by dotnetCorePackages.combinePackages (a wrapper around buildEnv) lacks the neccessary metadata for buildDotnetModule to work...
An alternative would be to patch this line (from net6.0 to net7.0) in patchPhase, but it's a bit unsafe because there are breaking changes between major .NET versions.
Or patch the nuget-to-nix script that generates deps.nix to ignore the version when checking if it should download a package. So that it wouldn't attempt to download version 6 of a package if and only if the same version or higher is already part of dotnet-sdk.

Thanks for the info, I'll attempt a fix for that.


Fix "Greater" false positive

This PR:

Adds test case demonstrating the false positive

Notes:
Testifylint is returning a false positive for:
empty: use assert.NotEmpty (testifylint)
assert.Greater(t, value, 1)
^

This false positive occurs with v0.2.1 which is used in golangci-lint. Using the current version does not result in this issue


Clamp translate values

Translate values only range from a normalized -1.0 to 1.0, need to clamp as you can currently add anything you want.

Clamped.


Core media

@kspurgin Review required for this one

@kevin-kariuki -- merging in my fixes to the mapper code slightly shifted which part of the code is throwing an error on this test. Before the fixes, it was:
Failure/Error: v.each_with_index{ |val, i| groups[i][:data][f] = val }
     NoMethodError:
       undefined method `[]' for nil:NilClass

After the fixes, it is:
     Failure/Error: groups[i][:data][f] = val

     NoMethodError:
       undefined method `[]' for nil:NilClass

PR #31 has the necessary test data changes that, in combination with the mapper code fixes, make the tests pass.


Fix e2e test with correct sample-service port

Fixed e2e test: changed sample-service port from 8888 to 9001

E2E test running at https://github.com/GoogleCloudPlatform/solutions-template/actions/runs/4207882052

/e2e-test

cancelling this PR


Bump numpy version

Description

Bump numpy version 1.17 because nan parameters of nan_to_num is introduced in this version.
Bug introduced in #1360
Type of change

[x] Bug-fix (non-breaking change which fixes an issue)

References

https://numpy.org/doc/stable/reference/generated/numpy.nan_to_num.html
https://github.com/napari/napari/blob/ea0fc9ca29d730808b1a66a8710679ee0fbe7ab5/napari/components/layerlist.py#L230-L231
https://travis-ci.com/github/4DNucleome/PartSeg/jobs/380612109 line 663
How has this been tested?

[ ] example: the test suite for my feature covers cases x, y, and z
[ ] example: all tests pass with my change

Final checklist:

[x] My PR is the minimum possible work for the desired functionality

Gah. Thank you for spotting this @Czaki.
According to NEP-29, we should continue to support 1.16 until January. So I think the correct fix is to bump to 1.16 and write our own nan_to_num.
We should also add a "minimum requirements" build to our test matrix to avoid these issues in the future. We do this with an env var in scikit-image, see here:
https://github.com/scikit-image/scikit-image/blob/73a8dbf0128e8832b15d86c88c9304bc1280aa71/tools/travis/before_install.sh#L60-L64

import os
from configparser import ConfigParser

config = ConfigParser()
config.read(os.path.join(os.path.dirname(__file__), "..", "setup.cfg"))

minimum_requirements = config["options"]["install_requires"].replace(">=", "==")

with open(os.path.join(os.path.dirname(__file__), "minimal-req.txt"), "w") as f_p:
    f_p.write(minimum_requirements)
    f_p.write("\n")

I tes nowt such code in my project to generate minimum requirements during test base on setup.cfg

Ok, thanks for the fix in #1613, agreed testing minimum versions sounds good, and I'm very open to different ways to do it in our CI, but that script from @Czaki looks like a good place to start

new version of script (with python dependecies)
import os
from configparser import ConfigParser

config = ConfigParser()
config.read(os.path.join(os.path.dirname(__file__), "..", "setup.cfg"))

minimum_requirements = config["options"]["install_requires"].replace(">=", "==")
minimum_qt = config["options.extras_require"]["pyqt"].replace(">=", "==")

with open(os.path.join(os.path.dirname(__file__), "minimal-req.txt"), "w") as f_p:
    f_p.write(minimum_requirements)
    f_p.write(minimum_qt)
    f_p.write("\n")

This can now be closed as #1613 and #1617 will supersede it


not able to download without without watermark i dont know how to use this thru github please help

Let me know how to use the updated version of this app! I love this extension and its really helpful
also not able to download videos in full hd Thanks

The new version should be published on both Firefox and Chrome now. I pushed it through yesterday to Chrome, but it usually takes them a day or two to review it, but it should be reviewed and available now.
Also, regarding HD formats. It isn't available through the API anymore, so I can't do much about that.
Anyways, for the future, loading an addon from its source code can be done like this:
On Chrome you can go to extensions, enable Developer mode in the top-right corner and then click Load unpacked, then selecting the directory containing the source code of this repository.
On Firefox, you also go to the extensions, then debug addons (through the cogwheel menu), then Load Temporary Addon, and again selecting the directory or the location of the manifest.json of this source.

Hey really thanks for the Sixem, i just wanted one more help is there any way that when i download only the USERNAME is their on the file and not the whole username - caption. It would be awesome if you can help in this also is there any way we can support you?


[refactor] Configure and use the zero-monkey-paching-mode in RSpec

This PR changes an RSpec thing: Prefix describe with RSpec.
This allows us to use the recommended RSpec zero-monkey-patching mode.
What I did to try this out

brew install phantomjs
brew install imagemagick
bundle exec rspec

The tests pass locally.
(Bonus: See #3166 - these instructions are now added as a README addition.)

Thanks 👍


pull request for e2e test at 01/10/2017 19:15:10

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[ENHANCEMENT] Revisit the max persistent partition size settings

Is your enhancement related to a problem? Please describe.
The persistent partition's maximum of the size is 100GB (See https://github.com/harvester/harvester-installer/blob/57171a58585b3dc76752b69dbe6d3faf498f00db/pkg/config/cos.go#L721).
This setting is too tight for increasing local images (especially upgrade path) and could lead to garbage collection easily.
And after introducing the user workload to bare metal cluster, this can go worse.
Describe the solution you'd like

We need to increase the persistent partition size.
Describe alternatives you've considered

Additional context

Here is the partition layout of a 2T disk:
nvme0n1     259:0    0   1.8T  0 disk
├─nvme0n1p1 259:1    0    64M  0 part
├─nvme0n1p2 259:2    0    50M  0 part /oem
├─nvme0n1p3 259:3    0     8G  0 part
├─nvme0n1p4 259:4    0    15G  0 part /run/initramfs/cos-state
├─nvme0n1p5 259:5    0   100G  0 part /var/lib/longhorn
│                                     /var/crash
│                                     /var/lib/third-party
│                                     /var/lib/cni
│                                     /var/lib/wicked
│                                     /var/lib/kubelet
│                                     /var/lib/rancher
│                                     /var/log
│                                     /usr/libexec
│                                     /root
│                                     /opt
│                                     /home
│                                     /etc/pki/trust/anchors
│                                     /etc/cni
│                                     /etc/iscsi
│                                     /etc/ssh
│                                     /etc/rancher
│                                     /etc/systemd
│                                     /usr/local
└─nvme0n1p6 259:6    0   1.7T  0 part /var/lib/harvester/defaultdisk

It looks the new ISO not built yet, the latest one is v1.2-3ea43d86-head which not include this fix.

The backport PR is merged after the version. @masteryyh we need to confirm backport BR is in then move the issue to ready-for-testing.

It looks the new ISO not built yet, the latest one is v1.2-3ea43d86-head which not include this fix.

@WuJun2016, @bk201 it looks false alarm, the installation stuck on fetching config.yaml because I reduce the size < 150Gi, but I think we should able to enhance it to not stuck but showing some error message or fail the installation.
Verified this feature has been implemented.

Test Information

Environment: qemu/KVM single node
Harvester Version: v1.2.0-rc2
ui-source Option: Auto

Verify Steps

Install Harvester with any nodes with configuration:

install:
  persistent_partition_size: 155Gi
# or `persistentPartitionSize`, 150 is the minimal size, otherwise the install will stuck

Login to node, execute lsblk, the partition size for /usr/local should equals to the size defined in configuration.


Avatar popcorn

Have two peers, running the same account/avatar. Then quit one and re-join. You'll see two peer avatars at origin. Quit and re-join. You'll see three peer avatars at origin. This goes on to infinity.

Fixed with https://github.com/webaverse/app/commit/14febae26f1022bd4cb3c4cba862b4c18b77aa50


Flamethrowers don't work on Ghasts

Issue description:
D: A player of mine was given a flamethrower and nearly died! Are they fire resitant?
Steps to reproduce:

Git flamethrower
Am for the nose!

Nope lol oof sorry

They love to swim in lava as well

kinky


Update Ubuntu download URL and SHA256 to 14.04.4

The 14.04.3 URLs are 404s now so bin/build fails

Thanks for the PR @dajobe


able to replace extended characters (\n) like notepad++

Issue Type: Feature Request
able to replace extended characters (\n) like notepad++
VS Code version: Code - Insiders 1.57.0-insider (fe1eafc80c07608de52a2f9c99fee103dea77f5e, 2021-06-04T12:21:40.077Z)
OS version: Windows_NT x64 10.0.19042

/needsMoreInfo
How are you truing to do that?
You should be able to do that by enabling RegEx mode for search/replace

@vscode-triage-bot  sorry. i was just using the "Report Issue" right on the Visual Studio Code, i have no idea that there are some guidelines.
@IllusionMH i do not think it is working by enabling RegEx mode.
for example, i want to replace "\r\n" to , (comma).
it works in reverse but not in the above case

@tins820 Are you trying this in the Search view or in Editor find functionality?

VS Code editor normalizes line endings and doesn't keep separate CR characters in files with CRLF EOLs.
Just use \n in Find widget (in editor) to replace it to , - this will replace both characters CRLF.
Find in files will match both characters separately as it doesn't do normalization and searches on disk (if file is not in open tab)

@tins820 FYI - please check the comment

Are you trying this in the Search view or in Editor find functionality?


codegen conditional on hasmethod

Currently passing everything except precompilation. Here's the error:
caching with precompilation: Test Failed at /home/chad/git/KeywordCalls/test/runtests.jl:86
  Expression: Precompile2.test() == (a = 2, b = 1)
   Evaluated: (b = 1, a = 2) == (a = 2, b = 1)
Stacktrace:
 [1] macro expansion
   @ ~/git/KeywordCalls/test/runtests.jl:86 [inlined]
 [2] macro expansion
   @ /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.6/Test/src/Test.jl:1151 [inlined]
 [3] top-level scope
   @ ~/git/KeywordCalls/test/runtests.jl:86
Test Summary:               | Fail  Total
caching with precompilation |    1      1
ERROR: LoadError: Some tests did not pass: 0 passed, 1 failed, 0 errored, 0 broken.
in expression starting at /home/chad/git/KeywordCalls/test/runtests.jl:85
ERROR: Package KeywordCalls errored during testing

Think it's working, what do you think @mileslucas ?

Just to be clear, this doesn't yet address #17 , but only #16

Great point, I've updated the docstrings


Outdated Requires

Hi, it's me again...
Proble with install ton, but rel for this package
composer require olifanton/interop

Problem 1
    - olifanton/interop[0.1.0, ..., 0.1.1, 1.0.0, ..., 1.4.0] require brick/math ^0.10 -> found brick/math[0.10.0, 0.10.1, 0.10.2] but the package is fixed to 0.12.1 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
    - Root composer.json requires olifanton/interop * -> satisfiable by olifanton/interop[0.1.0, 0.1.1, 1.0.0, ..., 1.4.0].

Fixed in version 1.4.1


Mobile > Badge: Add user, default and a generic icon

@LucasBrazi06  Does the new style still requires a generic icon ?

@LucasBrazi06 Does the new style still requires a generic icon ?

Yep.
Btw, the ID is no longer displayed, only visualID

@LucasBrazi06 Does the new style still requires a generic icon ?

Yep.
Btw, the ID is no longer displayed, only visualID

I am afraid the icon will take too much space, here it is still small

I am afraid the icon will take too much space, here it is still small

Yes you're right.
Have you checked the design on other apps ?

I am afraid the icon will take too much space, here it is still small

Yes you're right.
Have you checked the design on other apps ?

Chargemap has a single pass, Smoov has a really poor design, I am trying to figure something out

ChargePoint and Shell

Well, one common point is that fonts are smaller, so I tried something with reduced fonts and I think it looks better :)

The icon I have chosen is very similar to the one in Shell Recharge app

Mine is better :-P

Mine is better :-P

Where did you get it ?

Where did you get it ?

I made it :)


Mr/spinnakerenvs

What this PR does / why we need it:
Enables the values.yaml to deliver additional Environment Variables to the halyard container
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Pls review
@viglesiasce
@dwardu89
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] DCO signed
[ ] Chart Version bumped
[ ] Variables are documented in the README.md

/assign @dwardu89
Pls review

/assign @paulczar

/assign @davidkarlsen

Does it work?


Update ver.md

Updated to note that ver does not work in powershell and provided an example that shows how to get the ver info from PowerShell.

Thanks, Thomas!


Cannot access storage file '/var/run/kubevirt-ephemeral-disks/disk-data/containerdisk/disk.qcow2'

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:
/kind bug

Below are the steps followed for creating kubevirt vm :-

Deployed K8s cluster successfully with one master and one worker in KVM-based host.
Later deployed Kubevirt service successfully.

What happened:
While creating kubevirt vm with the help of user guide, it’s getting failed due to below error
kubectl describe vmi testvm
Normal   SuccessfulCreate  87s                 virtualmachine-controller  Created virtual machine pod virt-launcher-testvm-h74ww
Warning  SyncFailed        34s (x14 over 76s)  virt-handler               server error. command SyncVMI failed: "LibvirtError(Code=38, Domain=18, Message='Cannot access storage file '/var/run/kubevirt-ephemeral-disks/disk-data/containerdisk/disk.qcow2' (as uid:107, gid:107): Permission denied')"
kubectl describe pod virt-launcher-testvm-h74ww

kubevirt
What you expected to happen:
I expected the VM to start.
How to reproduce it (as minimally and precisely as possible):
just deploy k8s cluster and kubevirt
Anything else we need to know?:
Environment:

KubeVirt version (use virtctl version): 0.41.3

Kubernetes version (use kubectl version): 1.22

VM or VMI specifications:
apiVersion: kubevirt.io/v1alpha3
kind: VirtualMachine
metadata:
name: testvm
spec:
running: false
template:
metadata:
labels:
kubevirt.io/size: small
kubevirt.io/domain: testvm
spec:
domain:
devices:
disks:
- name: containerdisk
disk:
bus: virtio
- name: cloudinitdisk
disk:
bus: virtio
interfaces:
- name: default
bridge: {}
resources:
requests:
memory: 64M
networks:
- name: default
pod: {}
volumes:
- name: containerdisk
containerDisk:
image: quay.io/kubevirt/cirros-container-disk-demo
- name: cloudinitdisk
cloudInitNoCloud:
userDataBase64: SGkuXG4=

Cloud provider or hardware configuration: no cloud provider

OS (e.g. from /etc/os-release):NAME="EulerOS"
VERSION="2.0 (SP10x86_64)"
ID="euleros"
VERSION_ID="2.0"
PRETTY_NAME="EulerOS 2.0 (SP10x86_64)"
ANSI_COLOR="0;31"

Kernel (e.g. uname -a):4.18.0-<IP_ADDRESS>.h596.eulerosv2r10.x86_64

Install tools:

Others:

Hi, thanks for this report.
First, I'd like to say that we don't test KubeVirt on k8s 1.22 just yet.
I'm also not familiar with the EulerOS. Can you please tell me what is the default mask on this os?

Interesting...
Perhaps you could check what are the permission on the /var/run/kubevirt-ephemeral-disks/disk-data/containerdisk directory and its content?
It should be possible to leave the virt-launcher container alive after qemu has failed to execute. To enable this, please add the kubevirt.io/keep-launcher-alive-after-failure annotation to the VMI yaml.
Then you will be able to exec it to the virt-launcher compute container.

The permission of /var/run/kubevirt-ephemeral-disks/disk-data/containerdisk directory is 600, and the group and user in qemu.conf is "qemu". I change  mod the permission of /var/run/kubevirt-ephemeral-disks/disk-data/containerdisk directory to 777, but it doesn't work. And I can restart libvirtd in the virt-launcher container.

Can anyone know how to solve this problem?

Interesting...
Perhaps you could check what are the permission on the /var/run/kubevirt-ephemeral-disks/disk-data/containerdisk directory and its content?
It should be possible to leave the virt-launcher container alive after qemu has failed to execute. To enable this, please add the kubevirt.io/keep-launcher-alive-after-failure annotation to the VMI yaml.
Then you will be able to exec it to the virt-launcher compute container.

The permission of /var/run/kubevirt-ephemeral-disks/disk-data/containerdisk directory is 600, and the group and user in qemu.conf is "qemu". I change mod the permission of /var/run/kubevirt-ephemeral-disks/disk-data/containerdisk directory to 777, but it doesn't work. And I can't restart libvirtd in the virt-launcher container.

Did you have a chance to check the permissions and ownership of the disk?

Did you have a chance to check the permissions and ownership of the disk?

This may have already been checked but did you create the containerdisk like described here:
ADD --chown=107:107 fedora25.qcow2 /disk/

Note that the ADD sets the file ownership to the typical qemu uid/gid.


UI bugbash: sql-migration fix assessment db selection

This PR fixes

adds error handing and an error banner for assessment errors
fix issue refreshing assessment results for runtime error and database selection changes
fix issue with assessed and selected database counters on sku recommendation page
show loader while assessments are re-run
add context check to improve error handling for vm and mi asset queries

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 41.718%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
26419

Relevant Lines:
57464

💛  - Coveralls